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