On Wed, 15 Apr 2026 14:56:24 GMT, Alexey Ivanov <[email protected]> wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8377568
>
> src/java.desktop/share/classes/java/awt/image/DataBufferFloat.java line 118:
>
>> 116: * @throws NullPointerException if {@code dataArray} is {@code
>> null}.
>> 117: * @throws IllegalArgumentException if {@code size} is less than or
>> equal
>> 118: * to zero or is greater than the length of {@code
>> dataArray}.
>
> Suggestion:
>
> * @throws IllegalArgumentException if {@code size} is less than or equal
> * to zero, or is greater than the length of {@code dataArray}.
>
> A comma is missing here.
comma added
> src/java.desktop/share/classes/java/awt/image/DataBufferInt.java line 121:
>
>> 119: * @throws NullPointerException if {@code dataArray} is {@code
>> null}.
>> 120: * @throws IllegalArgumentException if {@code size} is less than or
>> equal
>> 121: * to zero or is greater than the length of {@code
>> dataArray}.
>
> Suggestion:
>
> * @throws IllegalArgumentException if {@code size} is less than or equal
> * to zero, or is greater than the length of {@code dataArray}.
comma added
> src/java.desktop/share/classes/java/awt/image/DataBufferUShort.java line 122:
>
>> 120: * @throws NullPointerException if {@code dataArray} is {@code
>> null}.
>> 121: * @throws IllegalArgumentException if {@code size} is less than or
>> equal
>> 122: * to zero or is greater than the length of {@code
>> dataArray}.
>
> Suggestion:
>
> * @throws IllegalArgumentException if {@code size} is less than or equal
> * to zero, or is greater than the length of {@code dataArray}.
comma added
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3096032093
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3096038827
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3096041526