[10] RFR 8179086: java.time.temporal.ValueRange has poor hashCode()

[Roger Riggs]

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.
>