On Mon, 23 Mar 2026 05:52:55 GMT, Jayathirth D V <[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/DataBufferByte.java line 214:
>
>> 212: * @throws IllegalArgumentException if {@code size} is less than or
>> equal to zero.
>> 213: * @throws NullPointerException if {@code dataArray} is {@code
>> null}.
>> 214: * @throws NullPointerException if any bank of {@code dataArray} is
>> {@code null}.
>
> We can merge NPE spec into single comment:
> `@throws NullPointerException if {@code dataArray} or any of its bank is
> {@code null}.`
>
> Similar observation is made in other classes also.
It may be clearer to have them separate.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r2977217873