On Wed, 26 Nov 2025 16:41:56 GMT, Alexey Ivanov <[email protected]> wrote:
>> Phil Race has updated the pull request with a new target base due to a merge
>> or a rebase. The incremental webrev excludes the unrelated changes brought
>> in by the merge/rebase. The pull request contains four additional commits
>> since the last revision:
>>
>> - 4617681
>> - Merge remote-tracking branch 'upstream/master' into bimg
>> Merge master
>> - 4617681
>> - 4617681
>
> src/java.desktop/share/classes/java/awt/image/BufferedImage.java line 343:
>
>> 341: if (lsz > Integer.MAX_VALUE) {
>> 342: throw new IllegalArgumentException(
>> 343: "width " + width + " height " + height + "
>> overflow int");
>
> Suggestion:
>
> "width " + width + "* height " + height + " overflow
> int");
>
> Is the asterisk to denote multiplication missing?
done
> src/java.desktop/share/classes/java/awt/image/BufferedImage.java line 672:
>
>> 670: * {@code raster} is incompatible with {@code cm}
>> 671: * @throws IllegalArgumentException if
>> 672: * {@code raster} {@code minX} or {@code minY} is not zero
>
> Suggestion:
>
> * {@code raster}, {@code minX} or {@code minY} is not zero
>
> Is the comma missing?
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4211917825
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4211919162
