[10] RFR 8179086: java.time.temporal.ValueRange has poor hashCode()
Ivan Gerasimov
ivan.gerasimov at oracle.com
Fri Apr 21 18:42:14 UTC 2017
Thank you Roger!
Pushed.
On 4/21/17 11:07 AM, Roger Riggs wrote:
> Hi Ivan,
>
> Looks fine, I don't expect many ValueRanges in hash maps.
> JDK 10 is a fine place to put it.
>
> Thanks, Roger
>
>
> On 4/21/2017 1:55 PM, Ivan Gerasimov wrote:
>> Hello!
>>
>> It was noticed that ValueRange.hashCode() returns zero for many values.
>> This is due to lack of some parentheses to set correct precedence.
>>
>> Would you please help review the trivial fix?
>>
>> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8179086
>> WEBREV: http://cr.openjdk.java.net/~igerasim/8179086/00/webrev/
>>
>> The fix is applicable to jdk 9, but I'm not sure it's that critical.
>> We can later backport it to an update release.
>>
>
>
--
With kind regards,
Ivan Gerasimov
More information about the core-libs-dev
mailing list