On Thu, 19 Jun 2025 08:25:53 GMT, Andrey Turbanov <aturba...@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

Reply via email to