On Thu, 5 Oct 2023 10:31:31 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

>> See the [JBS issue](https://bugs.openjdk.org/browse/JDK-8317515) for the 
>> details.
>
> Raffaello Giulietti has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Avoid localized integers in radix-out-of-range exception messages.

src/java.base/share/classes/java/lang/Integer.java line 767:

> 765:             /* Use MIN_VALUE + x < MIN_VALUE + y as unsigned x < y 
> comparison */
> 766:             while (i < len && (digit = digit(s.charAt(i++), radix)) >= 0
> 767:                     && (inRange = MIN_VALUE + result < MIN_VALUE + 
> multmax

`compareUnsigned(result, multmax) < 0` would be better here.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16050#discussion_r1348196974

Reply via email to