Re: [9] RFR 8027289: [Windows zh_CN] NumberFormat: Incorrect sequence of loading currency symbol

2014-02-17 Thread Erik Joelsson
My understanding is that these changes are needed. From a build perspective, while making complicated make logic even more complicated, it's really not that bad imho. So OK from me. /Erik On 2014-02-17 17:39, Naoto Sato wrote: Any other comments? If there is no strong opinion against pushing t

Re: RFR: JDK-8035067: Move jdk/src/share/classes/java/util/CurrencyData.properties to jdk/make/data

2014-02-17 Thread Naoto Sato
+1 Naoto On 2/17/2014 6:40 AM, Alan Bateman wrote: On 17/02/2014 14:36, Erik Joelsson wrote: Hello, I found another data file that belongs in make/data. Bug: https://bugs.openjdk.java.net/browse/JDK-8035067 Webrev: http://cr.openjdk.java.net/~erikj/8035067/webrev.jdk.01/ This looks okay to

Re: [9] RFR 8027289: [Windows zh_CN] NumberFormat: Incorrect sequence of loading currency symbol

2014-02-17 Thread Naoto Sato
Any other comments? If there is no strong opinion against pushing this change, I just want to push it to the repo. Naoto On 2/12/14, 4:43 PM, Naoto Sato wrote: On 2/12/14, 4:07 PM, Masayoshi Okutsu wrote: The problem in the bug report is that the currency symbol is taken from the HOST locale

Re: RFR: JDK-8035067: Move jdk/src/share/classes/java/util/CurrencyData.properties to jdk/make/data

2014-02-17 Thread Alan Bateman
On 17/02/2014 14:36, Erik Joelsson wrote: Hello, I found another data file that belongs in make/data. Bug: https://bugs.openjdk.java.net/browse/JDK-8035067 Webrev: http://cr.openjdk.java.net/~erikj/8035067/webrev.jdk.01/ This looks okay to me. Also cc'ing i18n-dev as this is where the currency