[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