JDK 9 RFR of 8030814: Long.parseUnsignedLong should throw exception on too large input

Brian Burkhalter brian.burkhalter at oracle.com
Thu Jan 9 15:32:31 UTC 2014


On Jan 9, 2014, at 1:48 AM, Paul Sandoz wrote:

> This generally looks good, rather clever trick, i prefer it to using a cache.
> 
> I agree with Joe some comments are required as it is not immediately obvious how this works.

I have some simpler comments almost done being redacted.

> The additional tests seem adequate (overflow of the overflow), it's easy to go overboard e.g. you could test:
> 
>  BigInteger b = quotient.multiply(BigInteger.valueOf(radix + N));
> 
> for some range of N according to the radix under test. However, it might be useful to accurately test boundary conditions.

Working on that also.

Thanks,

Brian


More information about the core-libs-dev mailing list