RFR: JDK-8036818: DateTimeFormatter withResolverFields() fails to accept null
Chris Hegarty
chris.hegarty at oracle.com
Mon Mar 17 13:58:09 UTC 2014
> On 17 Mar 2014, at 11:17, Stephen Colebourne <scolebourne at joda.org> wrote:
>
> To confirm, this counts as a review "yes"?
Yes. Sorry if this wasn't clear.
-Chris.
> Stephen
>
>
>> On 12 March 2014 14:27, Chris Hegarty <chris.hegarty at oracle.com> wrote:
>> The change look ok to me too.
>>
>> There is a change in behavior here, but I don't expect it to be surprising (
>> no NPE where there once was ), so I think it should be fine for 8u-dev also.
>>
>> The TCK test changes however, may not be suitable for 8u. Though I'm not
>> sure how these tests feed from the OpenJDK repo into the actual TCK.
>>
>> -Chris.
>>
>>> On 12/03/14 13:54, roger riggs wrote:
>>>
>>> Looks fine, (not a reviewer).
>>>
>>> I can sponsor the fix when reviewed.
>>>
>>> Thanks, Roger
>>>
>>>
>>>> On 3/12/2014 6:45 AM, Stephen Colebourne wrote:
>>>>
>>>> This is a request for review of this bug:
>>>> https://bugs.openjdk.java.net/browse/JDK-8036818
>>>>
>>>> The method DateTimeFormatter withResolverFields() is supposed to
>>>> accept null. This is to allow a coy of the formatter to be returned
>>>> reset to the original state of having no resolver fields. The docs
>>>> say:
>>>> "@param resolverFields the new set of resolver fields, null if no fields"
>>>> which was written to indicate that resetting to null is permitted.
>>>>
>>>> The fix is to check for null and return a copy of the formatter. Note
>>>> that there are two variations of the method which need fixing.
>>>>
>>>> Proposed patch:
>>>> https://gist.github.com/jodastephen/9395197
>>>> The patch includes no spec changes.
>>>> The patch fixes the broken TCK tests.
>>>>
>>>> I need a reviewer and a committer please.
>>>> thanks
>>>> Stephen
>>
More information about the core-libs-dev
mailing list