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

Reply via email to