RFR: 4668290: unclear spec for Polygon.bounds field [v4]
Phil Race
prr at openjdk.org
Tue Dec 6 21:50:06 UTC 2022
On Tue, 6 Dec 2022 21:33:33 GMT, SWinxy <duke at openjdk.org> wrote:
>> New documentation replaces the `This value can be null.`:
>>
>> When created, {@link #invalidate() invalidated}, or {@link #reset() reset},
>> this becomes {@code null}. To get out of the null state,
>> {@link #getBounds()} called with {@link #npoints} being greater than
>> {@code 0} will transfer into a non-null {@link Rectangle}.
>>
>>
>> I think I've got the reason for why it can be `null` correct, but you never know.
>>
>> In javax.swing.text.html.Map, I've replaced setting the field to null with the equivalent #invalidate(). This may mean we can make the field private in the future.
>
> SWinxy has updated the pull request incrementally with one additional commit since the last revision:
>
> dash
Need the CSR
-------------
Marked as reviewed by prr (Reviewer).
PR: https://git.openjdk.org/jdk/pull/10417
More information about the client-libs-dev
mailing list