+1
Naoto
On 8/9/16 10:31 AM, Rachna Goel wrote:
Hi Naoto,
Thanks for the review.
Please have a look at updated patch:
http://cr.openjdk.java.net/~rgoel/jdk-8163350/webrev.03/
Thanks,
Rachna
On 8/9/16 9:39 PM, Naoto Sato wrote:
Hi Rachna,
I should've caught it at the internal review, b
Hi Naoto,
Thanks for the review.
Please have a look at updated patch:
http://cr.openjdk.java.net/~rgoel/jdk-8163350/webrev.03/
Thanks,
Rachna
On 8/9/16 9:39 PM, Naoto Sato wrote:
Hi Rachna,
I should've caught it at the internal review, but here are some
cosmetic comments to the test cas
Hi Rachna,
I should've caught it at the internal review, but here are some cosmetic
comments to the test case.
- I'd prefer a blank line between import statements and the main method.
- The argument to main() should be (String[] args)
- Indentation is incorrect at line 39
- Needs parens for th
Hi,
Please review fix for JDK-8163350.
Bug: https://bugs.openjdk.java.net/browse/JDK-8163350
Webrev: http://cr.openjdk.java.net/~rgoel/jdk-8163350/webrev.02/
Fix: If user specifies either CLDR or COMPAT via System Property,
FALLBACK should not be implicitly included.
It is meant to be inclu
