RFR:JDK-8148949:DateTimeFormatter pattern letters 'A','n','N'

[nadeesh tv]

Hi all,
Thanks Stephen for the comments.
Please see the updated webrev 
http://cr.openjdk.java.net/~ntv/8148949/webrev.02/

Regards,
Nadeesh


On 4/28/2016 7:58 PM, Stephen Colebourne wrote:
> 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.tv at 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
>>

-- 
Thanks and Regards,
Nadeesh TV