On 02/07/2014 11:32 AM, Paul Sandoz wrote:
On Feb 6, 2014, at 6:40 PM, Xueming Shen <[email protected]> wrote:
Paul,
toUpperCaseEx is overridden in CharacterData00/Latin1. Those two are
under gensrc/java/lang. It might be possible to combine them some day
(need to dig out some decade long history and probably there is compability
concern...), but definitely is beyond the scope of this "improvement" :-)
I was not suggesting combining, it was more a spin off to pondering of why
Character.toUpperCase was not used (mirroring the use of toLowerCase). Anyway,
this is minor.
Are we good enough to go? :-) While it took much longer than I would have
expected, but I'm happy with the latest result.
http://cr.openjdk.java.net/~sherman/8032012/webrev/
+1 from me.
Paul.
I'm happy with the latest code too.
Rémi