RFR: 4668290: unclear spec for Polygon.bounds field [v3]
SWinxy
duke at openjdk.org
Tue Dec 6 01:18:24 UTC 2022
> 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:
Use javadoc code syntax for null
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/10417/files
- new: https://git.openjdk.org/jdk/pull/10417/files/7d2e36a6..9b6f7d91
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=10417&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=10417&range=01-02
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/10417.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/10417/head:pull/10417
PR: https://git.openjdk.org/jdk/pull/10417
More information about the client-libs-dev
mailing list