Xueming Shen wrote:
:
UNICODE_CHARACTER_CLASS is clear and straightforward. I am OK with it.
The webrev, ccc and api docs have been updated accordingly.
Yes, I still need a reviewer for the implementation changes. Tom has
helped review
the doc (and the definition of those properties).
I've gone through the implementation changes. Looks good and I didn't
see anything obviously wrong. A couple of minor comments:
In UnicodeProp.forName it might be more efficient to eliminate the
containsKey and instead check whether get returns null. Also, should
this only be ignoring IAE rather than Exception?
In Pattern.java L912 I guess you didn't mean to change "string" to
"stringg". Another one at L1317 - "if UNICODE_CHARACTER_CLASS presents"
- I assume this should be "present".
You might want to check the headers on the new tests - looks like you've
included the header that has the classpath exception, also they claim to
have been written years ago. UnicodeProp is also confused about its
birthdate.
-Alan
