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 aivanov (Reviewer). src/java.desktop/share/classes/javax/swing/plaf/basic/BasicDesktopPaneUI.java line 33: > 31: import java.beans.*; > 32: > 33: import java.awt.event.ActionEvent; I suggest expanding all the wildcard imports: import java.awt.Component; import java.awt.Container; import java.awt.Dimension; import java.awt.FocusTraversalPolicy; import java.awt.Graphics; import java.awt.Insets; import java.awt.KeyboardFocusManager; import java.awt.Point; import java.awt.event.ActionEvent; import java.beans.PropertyChangeEvent; import java.beans.PropertyChangeListener; import java.beans.PropertyVetoException; import javax.swing.AbstractAction; import javax.swing.DefaultDesktopManager; import javax.swing.DesktopManager; import javax.swing.InputMap; import javax.swing.JComponent; import javax.swing.JDesktopPane; import javax.swing.JInternalFrame; import javax.swing.KeyStroke; import javax.swing.LookAndFeel; import javax.swing.SortingFocusTraversalPolicy; import javax.swing.SwingUtilities; import javax.swing.UIManager; import javax.swing.plaf.ComponentUI; import javax.swing.plaf.DesktopPaneUI; import javax.swing.plaf.UIResource; import sun.swing.DefaultLookup; import sun.swing.UIAction; ------------- PR Review: https://git.openjdk.org/jdk/pull/25893#pullrequestreview-3097887826 PR Review Comment: https://git.openjdk.org/jdk/pull/25893#discussion_r2260823380
