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

Reply via email to