Re: [8] Request for review: 8029239: jdk8 l10n resource file translation update - localenames

2013-12-04 Thread Masayoshi Okutsu
Thanks for making the \u changes. It's much easier to review. I checked LocaleNames_ja.properties with JIS X 0412 (Japanese version of ISO 639) and JIS X 0304 (Japanese version of ISO 3166). Many changes corrected wrong names, but there are still questionable ones, and some are inconsisten

Re: [8] Request for review: 8029239: jdk8 l10n resource file translation update - localenames

2013-12-04 Thread Michael Fang
Thanks Masayoshi for the feedback. I looked at the existing LocaleNames_xx.properties files and it contains lower case \u notation for most of sections of the files, but newer sections as ISO 639.2 language code and ISO 15924 script code were in upper case since they were added by the curr

Re: [8] Request for review: 8029239: jdk8 l10n resource file translation update - localenames

2013-12-04 Thread Masayoshi Okutsu
I still don't like the unnecessary case changes for the \u notation, which makes code review very difficult (even UTF-8 ones). I noticed the following change uses '=' (the second one) as a word delimiter. This looks strange to me. src/share/classes/sun/util/resources/ja/LocaleNames_ja.pro