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.
