RFR: 8311805: Clean up ScrollPane: drop redundant initialiser, mark scroller final

Alexander Zvegintsev azvegint at openjdk.org
Tue Jul 11 17:16:02 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 azvegint (Reviewer).

-------------

PR Review: https://git.openjdk.org/jdk/pull/14827#pullrequestreview-1524752861



More information about the client-libs-dev mailing list