Hi Claes,
thanks for review, timeout is "int" so even if you set max(2147483647)
value that int data type can hold, there will not be any overflow. If
you try to set very large number like "0x7fffffffffffffff" to int data
type you will get compile time error(integer number too large:
7fffffffffffffff).
Thanks,
Vyom
On Tuesday 09 May 2017 11:20 PM, Claes Redestad wrote:
Hi,
doesn't this need to consider numerical overflows, e.g., what happens
if someone sets a timeout value near 0x7fffffffffffffffL before and
after this change?
/Claes
On 2017-05-09 11:55, Vyom Tewari wrote:
Hi ,
Please review the code change for below issue.
Webrev :
http://cr.openjdk.java.net/~vtewari/8179905/webrev0.0/index.html
BugId : https://bugs.openjdk.java.net/browse/JDK-8179905
This issue is duplicate of "JDK-8165437", where pushed code change
failed on 32 bit platforms so we revert back code changes as part of
"JDK-8179602".
Please find below is the webrev that was pushed as part of
"JDK-8165437".
Webrev :
http://cr.openjdk.java.net/~vtewari/8165437/webrev0.7/index.html
Thanks,
Vyom
