+1
Naoto
On 10/7/16 4:08 PM, Xueming Shen wrote:
On 10/07/2016 03:52 PM, Naoto Sato wrote:
The test case now looks good. Using biconsumer makes it more readable.
I see the additional paragraph with regard to IllegalArgumentException
in each method description, however, it contradicts the
On 10/07/2016 03:52 PM, Naoto Sato wrote:
The test case now looks good. Using biconsumer makes it more readable.
I see the additional paragraph with regard to IllegalArgumentException in each
method description, however, it contradicts the @throws clause. Need to add
@throws
The test case now looks good. Using biconsumer makes it more readable.
I see the additional paragraph with regard to IllegalArgumentException
in each method description, however, it contradicts the @throws clause.
Need to add @throws IllegalArgumentException for each method?
Naoto
On
thanks! updated, with biconsumer as well.
http://cr.openjdk.java.net/~sherman/8166261/webrev/
On 10/05/2016 10:25 AM, Naoto Sato wrote:
Looks good to me.
The test case could use IntStream.rangeClosed(Character.MIN_RADIX,
Character.MAX_RADIX) for the good radixes, instead of hard coding
Looks good to me.
The test case could use IntStream.rangeClosed(Character.MIN_RADIX,
Character.MAX_RADIX) for the good radixes, instead of hard coding ints.
Naoto
On 10/5/16 8:53 AM, Xueming Shen wrote:
Hi
Please help review
issue: https://bugs.openjdk.java.net/browse/JDK-8166261
webre:
Hi
Please help review
issue: https://bugs.openjdk.java.net/browse/JDK-8166261
webre: http://cr.openjdk.java.net/~sherman/8166261/webrev
The radix sanity check are missing from
hasNextByte/Short/Int/Long/BigInteger().
The only method we are doing now is useRadix(). The proposed change here