Thank you Paul for your review!
On 13.07.2015 12:57, Paul Sandoz wrote:
On Jul 12, 2015, at 6:32 PM, Ivan Gerasimov <[email protected]> wrote:
Hello!
There's a minor issue in the current implementation of Pattern.quote() with
possible numeric overflow when calculating initial capacity of StringBuilder.
With the fix, some slight optimizations were done and a few typos were fixed.
Would you please help review the fix?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8131034
WEBREV: http://cr.openjdk.java.net/~igerasim/8131034/00/webrev/
Looks ok.
/**
- * The pattern is converted to normalizedD form and then a pure group
+ * The pattern is converted to normalized form and then a pure group
* is constructed to match canonical equivalences of the characters.
*/
The "D" in "normalizedD form" may well be short hand for "canonical decomposition"
(Normalizer.Form.NFD) as in "normalized canonical decomposition form".
Hm. Still looks like a typo to me.
A few lines below normalizedPattern has no extra D.
Another normalization form, NFKD, also ends with D, so normalizedD
wouldn't uniquely identify NFD.
grep didn't find other occurrences of normalizedD abbreviation in JDK :)
Sincerely yours,
Ivan
Paul.
Sincerely yours,
Ivan
