Hi Nishit,

One nit in the test. At line 71, probably the original intention of "str" is to list the accept languages in alphabetic order. Would you change it?

I have not looked into actual tags in the test. Did you modify them manually? If so, it would be desirable to make it automated.

Naoto

On 11/16/17 3:47 AM, Nishit Jain wrote:
Hi,

Please review the fix for JDK-8191404

Bug: https://bugs.openjdk.java.net/browse/JDK-8191404
Webrev: http://cr.openjdk.java.net/~nishjain/8191404/webrev.00/

Fix: Updated the LSR data with the version: 2017-08-15

Regards,
Nishit Jain

Reply via email to