On Mon, 23 Mar 2026 19:47:54 GMT, Alexey Ivanov <[email protected]> wrote:

>> 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.

hmm. no unanimity here. Let's go with the status quo.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r2977841528

Reply via email to