Looks good to me. -Sundar
On 3/24/2016 9:42 PM, Naoto Sato wrote: > And the fix is here. Please review this as well. > > http://cr.openjdk.java.net/~naoto/8152704/webrev.00/ > > Naoto > > On 3/24/16 8:51 AM, Naoto Sato wrote: >> Hi Mandy, Sundar, Alan, >> >> Thank you for your reviews. >> >> On 3/23/16 11:49 PM, Alan Bateman wrote: >>> On 23/03/2016 22:47, Naoto Sato wrote: >>>> Hello, >>>> >>>> Please review the fix to the following bug: >>>> >>>> https://bugs.openjdk.java.net/browse/JDK-8152143 >>>> >>>> The fix is located at: >>>> >>>> http://cr.openjdk.java.net/~naoto/8152143/webrev.02/ >>> One little nit that I didn't notice before is "eg: en,ja,*-IN" where I >>> assume should be "e.g." >> >> I copied the part from another plugin. As I've already pushed the above >> change, I created an issue specific for >> this(https://bugs.openjdk.java.net/browse/JDK-8152704), and will fix >> them shortly. >> >> Naoto