JDK 9 RFR of 8030814: Long.parseUnsignedLong should throw exception on too large input
joe.darcy at oracle.com
Fri Jan 3 20:23:43 UTC 2014
I think more explanation of how the new code works needs to be included
in the code (and the commented out code should be deleted).
For the test, I would expect something with a bit simpler structure, but
perhaps I don't fully understand the boundary cases of the new code.
On 01/03/2014 11:00 AM, Brian Burkhalter wrote:
> Issue: https://bugs.openjdk.java.net/browse/JDK-8030814
> webrev: http://cr.openjdk.java.net/~bpb/8030814/webrev.2/
> This review request follows from the discussion of last month in this thread:
> The contributed patch before my minor tweaking of it is here
> with a detailed explanation of its logic here
> I added to the java/lang/Long/Unsigned JTREG test the case from the issue report as well as some other cases which exercise both sides of the A v B overflow test.
More information about the core-libs-dev