Re: Venezuela Currency has been changed to BS.F. now

2012-08-08 Thread Yong Jeffrey Huang
Hi Daven, The change looks good to me. When the fix is pushed to open JDK repository, please notify me, so that I can update the CR status. thanks, Yong On 2012/8/9 11:21, Deven You wrote: Hi Jeffrey, Thanks for your review. I have updated the webrev[1] according to your comments. To M

Re: Venezuela Currency has been changed to BS.F. now

2012-08-08 Thread Deven You
Hi Jeffrey, Thanks for your review. I have updated the webrev[1] according to your comments. To Masayoshi, After Jeffrey confirm this version of webrev please give your opinion as Jeffrey mentioned we need a reviewer. And then I will ask Jonathan to help me commit this patch. Thanks a lo

Re: [8] Review request for JEP 127: Improve Locale Data Packaging and Adopt Unicode CLDR Data

2012-08-08 Thread Naoto Sato
On 8/7/12 2:57 AM, Erik Joelsson wrote: See inline On 2012-07-13 22:20, Kelly O'Hair wrote: Something seems strange here: http://cr.openjdk.java.net/~naoto/6336885/webrev.00/make/java/java/localegen.sh.sdiff.html It's like someone was avoiding overall quotes, but using them to add spaces someh

Re: [8] Review request for JEP 127: Improve Locale Data Packaging and Adopt Unicode CLDR Data

2012-08-08 Thread Erik Joelsson
See inline On 2012-07-13 22:20, Kelly O'Hair wrote: Something seems strange here: http://cr.openjdk.java.net/~naoto/6336885/webrev.00/make/java/java/localegen.sh.sdiff.html It's like someone was avoiding overall quotes, but using them to add spaces somehow... I sure would like to get rid o