On Wed, 26 Nov 2025 16:43:41 GMT, Alexey Ivanov <[email protected]> wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 4617681
>
> src/java.desktop/share/classes/java/awt/image/BufferedImage.java line 362:
>
>> 360: if ((spp != 1) && (lsz * spp > Integer.MAX_VALUE)) {
>> 361: throw new IllegalArgumentException(
>> 362: "width " + width + " height " + height + " * " +
>
> Suggestion:
>
> "width " + width + "* height " + height + " * " +
>
>
> Is the asterisk missing?
fixed
> src/java.desktop/share/classes/java/awt/image/BufferedImage.java line 677:
>
>> 675: * @see WritableRaster
>> 676: */
>> 677:
>
> Remove this blank too? So that the javadoc ends just above a constructor
> declaration as in other cases?
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4212114698
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4212126951
