I'd like to see the test cases in test_secondsPattern() check the result of the parse (by passing more arguments from data_secondsPattern)
Otherwise looks good. Stephen On 28 April 2016 at 14:12, nadeesh tv <nadeesh...@oracle.com> wrote: > Hi all, > Please see the updated webrev > http://cr.openjdk.java.net/~ntv/8148949/webrev.01/ > > Regards, > Nadeesh TV > > On 4/25/2016 8:08 PM, nadeesh tv wrote: >> >> HI all, >> Please review a fix for >> Bug ID - https://bugs.openjdk.java.net/browse/JDK-8148949 >> >> Issue - Pattern letters 'A' does not match the intent of LDML/CLDR >> >> Solution - Changed the definition of pattern letters 'A','n','N' >> >> Webrev - http://cr.openjdk.java.net/~ntv/8148949/webrev.00/ >> >> > > -- > Thanks and Regards, > Nadeesh TV >