Review: JDK 8 CR for Support Integer overflow
Roger Riggs
Roger.Riggs at oracle.com
Mon Feb 6 21:16:21 UTC 2012
Thanks for the review and comments:
The comments and suggestions are included in the updated webrev:
http://cr.openjdk.java.net/~rriggs/6708398.1
* Corrected error in multipleExact(long,long) with the special case
Long.MIN_VALUE * -1.
* Verified that retaining the optimization for small (2^31) arguments
is worthwhile,
not doing the divide saves about 1/2 on the execution time.
* Removed the negateExact methods since they don't pull their weight
in the API,
simple tests for MIN_VALUE and MAX_VALUE can be done by the
developer more efficiently.
* Simplified the arguments to the ArithmeticExceptions to be simple
strings since
debugging this kind of exception requires the source code.
* Expanded the comments in the implementation include descriptions and
references to the Hackers Delight where they are used.
* Updated the tests to include missing test cases
More comments, please
Thanks, Roger
More information about the core-libs-dev
mailing list