On Thu, 4 Dec 2025 19:42:56 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 300:
>
>> 298: * This is at most {@code Integer.MAX_VALUE}.
>> 299: * The number of samples per-pixel for an {@code imageType} affect
>> 300: * the maximum. For example, if an image format uses bytes to store
>
> Suggestion:
>
> * The number of samples per-pixel for an {@code imageType} affects
> * the maximum. For example, if an image format uses bytes to store
>
>
> “The number … affects…”?
fixed (the wording was changed)
> src/java.desktop/share/classes/java/awt/image/BufferedImage.java line 340:
>
>> 338: if (width <= 0 || height <= 0) {
>> 339: throw new IllegalArgumentException(
>> 340: "width " + width + " height " + height + " must
>> both be > 0");
>
> Suggestion:
>
> "width " + width + "and height " + height + " must both
> be > 0");
>
> width *and* height?
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4212072088
PR Review Comment: https://git.openjdk.org/jdk/pull/27640#discussion_r4212066172
