On Thu, 7 Aug 2025 16:21:41 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Looks like no reasons for it to be instance field > > 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; Sorry, missed the comment. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25893#discussion_r2262151251
