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
