RFR: 8311805: Clean up ScrollPane: drop redundant initialiser, mark scroller final
Phil Race
prr at openjdk.org
Tue Jul 11 18:11:15 UTC 2023
On Tue, 11 Jul 2023 11:44:15 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
> A small clean-up of `awt.ScrollPane` code:
>
> 1. Remove the redundant initialiser for `vs` in `layout` and move the declaration to where it's used. (One less `Dimension` object is created.)
> 2. Mark the `scroller` field of the `PeerFixer` class as `final`.
>
> Perhaps, the `PeerFixer` class could be `final` too.
Marked as reviewed by prr (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/14827#pullrequestreview-1524866303
More information about the client-libs-dev
mailing list