RFR: 8364808: Make BasicDesktopPaneUI.Actions.MOVE_RESIZE_INCREMENT static

Alexander Zvegintsev azvegint at openjdk.org
Wed Aug 6 12:49:04 UTC 2025


On Thu, 19 Jun 2025 08:25:53 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> Looks like no reasons for it to be instance field

Marked as reviewed by azvegint (Reviewer).

src/java.desktop/share/classes/javax/swing/plaf/basic/BasicDesktopPaneUI.java line 377:

> 375:         private static String NAVIGATE_NEXT = "navigateNext";
> 376:         private static String NAVIGATE_PREVIOUS = "navigatePrevious";
> 377:         private static final int MOVE_RESIZE_INCREMENT = 10;

It appears that all of the above strings in `Actions` class can be final.

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

PR Review: https://git.openjdk.org/jdk/pull/25893#pullrequestreview-3092374474
PR Review Comment: https://git.openjdk.org/jdk/pull/25893#discussion_r2257034353


More information about the client-libs-dev mailing list