Re: Review request for IMF classes and Locale related classes

2011-12-05 Thread Stuart Marks
Looks good! Please proceed with the push. thanks. s'marks On 12/5/11 11:01 AM, Naoto Sato wrote: OK here is the revised one: http://cr.openjdk.java.net/~naoto/7117469/webrev.02/ I hope this is the final :-) Naoto On 12/2/11 5:41 PM, Stuart Marks wrote: The changes you made are good, than

Re: Review request for IMF classes and Locale related classes

2011-12-05 Thread Naoto Sato
OK here is the revised one: http://cr.openjdk.java.net/~naoto/7117469/webrev.02/ I hope this is the final :-) Naoto On 12/2/11 5:41 PM, Stuart Marks wrote: The changes you made are good, thanks. I did figure out how to deal with the static initializer in AllAvailableLocales: @SuppressWarnin