Re: [8] Code review request: 8000983 and 8003267

2012-12-06 Thread Naoto Sato
Here are my comments: - Bundle.java Line 277,297: Do we need two separate iteration loops here? Two iteration loops look like the same. Line 303: typo "paris" Line 606: typo "Someting" Line 728: Could use foreach here. - CLDRConverter.java Line 388: The FormatData bundle is now "OPEN", wh

Re: [8] Code review request: 8000983 and 8003267

2012-12-06 Thread Erik Joelsson
(resending to all recepients) I don't see any changes to the build except for java classes in the cldrconverter build tool. This shouldn't affect the new build, so it's ok with the build group. /Erik On 2012-12-06 06:06, Masayoshi Okutsu wrote: Resending in order to include build-...@openjdk