Looks good to me.
Thanks,
Rachna
On 1/7/19 12:36 PM, Dora Zhou wrote:
Hi Rachna,
Updated the webrev: http://cr.openjdk.java.net/~dzhou/8215913/webrev.01/
Regards,
Dora
On 2019/1/4 17:44, Rachna Goel wrote:
Hi Dora,
Kindly update copyright years in both files and add bug id in
Hi Rachna,
Updated the webrev: http://cr.openjdk.java.net/~dzhou/8215913/webrev.01/
Regards,
Dora
On 2019/1/4 17:44, Rachna Goel wrote:
Hi Dora,
Kindly update copyright years in both files and add bug id in
LocaleProvidersRun.java.
Other than that, it looks good to me.
Thanks,
Rachna
+1
Naoto
On 1/4/19 7:21 AM, Roger Riggs wrote:
+1
On 01/04/2019 04:44 AM, Rachna Goel wrote:
Hi Dora,
Kindly update copyright years in both files and add bug id in
LocaleProvidersRun.java.
Other than that, it looks good to me.
Thanks,
Rachna
On 1/4/19 7:58 AM, Dora Zhou wrote:
+1
On 01/04/2019 04:44 AM, Rachna Goel wrote:
Hi Dora,
Kindly update copyright years in both files and add bug id in
LocaleProvidersRun.java.
Other than that, it looks good to me.
Thanks,
Rachna
On 1/4/19 7:58 AM, Dora Zhou wrote:
Hello,
Please help review the fix for the test bug
Hi Dora,
Kindly update copyright years in both files and add bug id in
LocaleProvidersRun.java.
Other than that, it looks good to me.
Thanks,
Rachna
On 1/4/19 7:58 AM, Dora Zhou wrote:
Hello,
Please help review the fix for the test bug
java/util/Locale/LocaleProvidersRun.java failed on
Hello,
Please help review the fix for the test bug
java/util/Locale/LocaleProvidersRun.java failed on de_DE and ja_JP locale.
Bug: https://bugs.openjdk.java.net/browse/JDK-8215913
Webrev: http://cr.openjdk.java.net/~dzhou/8215913/webrev.00/
Thanks,
Dora