RFR:JDK-8148949:DateTimeFormatter pattern letters 'A','n','N'
Roger Riggs
Roger.Riggs at Oracle.com
Tue May 3 14:37:40 UTC 2016
Hi Nadeesh,
src/java.base/share/classes/java/time/format/DateTimeFormatterBuilder:1522-1524
Is the switch from SignStyle.NOT_NEGATIVE to NORMAL intentional?
The ValueRange of MilliOfDay for example is (0, 86400000-1), so negative
values would be out of range.
Similarly, NanoOfSecond and NanoOfDay are non-negative. (Otherwise,
there should be test cases for negative values).
Thanks, Roger
On 4/28/2016 4:04 PM, nadeesh tv wrote:
> 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
>>>
>
More information about the core-libs-dev
mailing list