Hi Rachna,

The fix looks OK to me. Here are two nits:

- The bug id in the comment should be 8170840, since it is a regression caused by the fix to 8071929.

- The JIRA issue should have "noreg-self" keyword, and the link to 8071929 would be helpful.

Naoto

On 12/7/16 4:03 AM, Rachna Goel wrote:
Hi,

Kindly review fix for JDK-8170840.

 Bug :https://bugs.openjdk.java.net/browse/JDK-8170840

Fix : http://cr.openjdk.java.net/~rgoel/JDK-8170840/webrev/

Fix is to correct no of country codes returned in
TestGetLangsAndCountries() method.

Thanks,
Rachna

Reply via email to