RFR: 6185110: Undefined behaviour of SampleModel for width, height < 0 [v6]
Phil Race
prr at openjdk.org
Tue Nov 25 22:59:07 UTC 2025
On Tue, 25 Nov 2025 20:23:54 GMT, Phil Race <prr at openjdk.org> wrote:
>> Also, can we specify {@code null} to give emphasis? I see it is be being done like that in many places in swing like for ex.
>> `javax/swing/plaf/basic/BasicLookAndFeel.java: * @throws NullPointerException if {@code table} is {@code null}`
>> and in some updated class in this PR too in some places
>
> I've changed "is null" to "is {@code null}" everywhere I could see in the javadoc in the files in this PR.
> I have not updated the CSR, and don't plan to, since this is not a spec issue .
> Can we specify the NPEs in the same line like what it is done in l675 to be consistent and also to be concise? `@throws NullPointerException if {@code obj} or {code data} is null.`
I think I've done this now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27754#discussion_r2561812192
More information about the client-libs-dev
mailing list