Ignore:
Timestamp:
2016-12-08T00:54:43+01:00 (2 years ago)
Author:
Don-vip
Message:

findbugs - SIC_INNER_SHOULD_BE_STATIC_ANON

Location:
trunk/src/org/openstreetmap/josm/gui/io
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/io/BasicUploadSettingsPanel.java

    r11288 r11366  
    131131    public void setHistoryComboBoxDownFocusTraversalHandler(final Action handler, final HistoryComboBox hcb) {
    132132        hcb.getEditor().addActionListener(handler);
    133         hcb.getEditorComponent().addKeyListener(
    134                 new KeyAdapter() {
    135                     @Override
    136                     public void keyTyped(KeyEvent e) {
    137                         if (e.getKeyCode() == KeyEvent.VK_TAB) {
    138                             handler.actionPerformed(new ActionEvent(hcb, 0, "focusDown"));
    139                         }
    140                     }
    141                 }
    142         );
     133        hcb.getEditorComponent().addKeyListener(new HistoryComboBoxKeyAdapter(hcb, handler));
    143134    }
    144135
     
    184175    }
    185176
     177    static final class HistoryComboBoxKeyAdapter extends KeyAdapter {
     178        private final HistoryComboBox hcb;
     179        private final Action handler;
     180
     181        HistoryComboBoxKeyAdapter(HistoryComboBox hcb, Action handler) {
     182            this.hcb = hcb;
     183            this.handler = handler;
     184        }
     185
     186        @Override
     187        public void keyTyped(KeyEvent e) {
     188            if (e.getKeyCode() == KeyEvent.VK_TAB) {
     189                handler.actionPerformed(new ActionEvent(hcb, 0, "focusDown"));
     190            }
     191        }
     192    }
     193
    186194    /**
    187195     * Updates the changeset comment model upon changes in the input field.
  • trunk/src/org/openstreetmap/josm/gui/io/RecentlyOpenedFilesMenu.java

    r10428 r11366  
    6060
    6161        for (final String file : fileHistory) {
    62             add(new AbstractAction() {
    63                 {
    64                     putValue(NAME, file);
    65                     putValue("help", ht("/Action/OpenRecent"));
    66                     putValue("toolbar", Boolean.FALSE);
    67                 }
    68                 @Override
    69                 public void actionPerformed(ActionEvent e) {
    70                     File f = new File(file);
    71                     OpenFileTask task = new OpenFileTask(Collections.singletonList(f), null);
    72                     task.setRecordHistory(true);
    73                     Main.worker.submit(task);
    74                 }
    75             });
     62            add(new OpenRecentAction(file));
    7663        }
    7764        add(new JSeparator());
     
    8269        clearItem.setEnabled(!fileHistory.isEmpty());
    8370        add(clearItem);
     71    }
     72
     73    static final class OpenRecentAction extends AbstractAction {
     74        private final String file;
     75
     76        OpenRecentAction(String file) {
     77            this.file = file;
     78            putValue(NAME, file);
     79            putValue("help", ht("/Action/OpenRecent"));
     80            putValue("toolbar", Boolean.FALSE);
     81        }
     82
     83        @Override
     84        public void actionPerformed(ActionEvent e) {
     85            OpenFileTask task = new OpenFileTask(Collections.singletonList(new File(file)), null);
     86            task.setRecordHistory(true);
     87            Main.worker.submit(task);
     88        }
    8489    }
    8590
  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersDialog.java

    r11323 r11366  
    225225
    226226    private static class LayerListWarningMessagePanel extends JPanel {
     227        static final class LayerCellRenderer implements ListCellRenderer<SaveLayerInfo> {
     228            private final DefaultListCellRenderer def = new DefaultListCellRenderer();
     229
     230            @Override
     231            public Component getListCellRendererComponent(JList<? extends SaveLayerInfo> list, SaveLayerInfo info, int index,
     232                    boolean isSelected, boolean cellHasFocus) {
     233                def.setIcon(info.getLayer().getIcon());
     234                def.setText(info.getName());
     235                return def;
     236            }
     237        }
     238
    227239        private final JLabel lblMessage = new JLabel();
    228240        private final JList<SaveLayerInfo> lstLayers = new JList<>();
    229241
    230242        LayerListWarningMessagePanel(String msg, List<SaveLayerInfo> infos) {
     243            super(new GridBagLayout());
    231244            build();
    232245            lblMessage.setText(msg);
     
    235248
    236249        protected void build() {
    237             setLayout(new GridBagLayout());
    238250            GridBagConstraints gc = new GridBagConstraints();
    239251            gc.gridx = 0;
     
    244256            add(lblMessage, gc);
    245257            lblMessage.setHorizontalAlignment(JLabel.LEFT);
    246             lstLayers.setCellRenderer(
    247                     new ListCellRenderer<SaveLayerInfo>() {
    248                         private final DefaultListCellRenderer def = new DefaultListCellRenderer();
    249                         @Override
    250                         public Component getListCellRendererComponent(JList<? extends SaveLayerInfo> list, SaveLayerInfo info, int index,
    251                                 boolean isSelected, boolean cellHasFocus) {
    252                             def.setIcon(info.getLayer().getIcon());
    253                             def.setText(info.getName());
    254                             return def;
    255                         }
    256                     }
    257             );
     258            lstLayers.setCellRenderer(new LayerCellRenderer());
    258259            gc.gridx = 0;
    259260            gc.gridy = 1;
  • trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

    r11293 r11366  
    5555import org.openstreetmap.josm.tools.ImageProvider.ImageSizes;
    5656import org.openstreetmap.josm.tools.InputMapUtils;
     57import org.openstreetmap.josm.tools.MultiLineFlowLayout;
    5758import org.openstreetmap.josm.tools.Utils;
    5859import org.openstreetmap.josm.tools.WindowGeometry;
    59 import org.openstreetmap.josm.tools.MultiLineFlowLayout;
    6060
    6161/**
     
    135135
    136136        // a tabbed pane with configuration panels in the lower half
    137         tpConfigPanels = new JTabbedPane() {
    138             @Override
    139             public Dimension getPreferredSize() {
    140                 // make sure the tabbed pane never grabs more space than necessary
    141                 return super.getMinimumSize();
    142             }
    143         };
     137        tpConfigPanels = new CompactTabbedPane();
    144138
    145139        pnlBasicUploadSettings = new BasicUploadSettingsPanel(changesetCommentModel, changesetSourceModel);
     
    414408    }
    415409
     410    static final class CompactTabbedPane extends JTabbedPane {
     411        @Override
     412        public Dimension getPreferredSize() {
     413            // make sure the tabbed pane never grabs more space than necessary
     414            return super.getMinimumSize();
     415        }
     416    }
     417
    416418    /**
    417419     * Handles an upload.
  • trunk/src/org/openstreetmap/josm/gui/io/UploadSelectionDialog.java

    r10791 r11366  
    1010import java.awt.event.WindowAdapter;
    1111import java.awt.event.WindowEvent;
     12import java.io.Serializable;
    1213import java.util.ArrayList;
    1314import java.util.Collection;
     
    196197
    197198    static class OsmPrimitiveListModel extends AbstractListModel<OsmPrimitive> {
     199        static final class OsmPrimitiveComparator implements Comparator<OsmPrimitive>, Serializable {
     200            private final DefaultNameFormatter formatter = DefaultNameFormatter.getInstance();
     201
     202            @Override
     203            public int compare(OsmPrimitive o1, OsmPrimitive o2) {
     204                int ret = OsmPrimitiveType.from(o1).compareTo(OsmPrimitiveType.from(o2));
     205                if (ret != 0)
     206                    return ret;
     207                return o1.getDisplayName(formatter).compareTo(o1.getDisplayName(formatter));
     208            }
     209        }
     210
    198211        private transient List<OsmPrimitive> data;
    199212
    200213        protected void sort() {
    201             if (data == null)
    202                 return;
    203             data.sort(new Comparator<OsmPrimitive>() {
    204                     private DefaultNameFormatter formatter = DefaultNameFormatter.getInstance();
    205                     @Override
    206                     public int compare(OsmPrimitive o1, OsmPrimitive o2) {
    207                         int ret = OsmPrimitiveType.from(o1).compareTo(OsmPrimitiveType.from(o2));
    208                         if (ret != 0)
    209                             return ret;
    210                         return o1.getDisplayName(formatter).compareTo(o1.getDisplayName(formatter));
    211                     }
    212                 }
    213             );
     214            if (data != null)
     215                data.sort(new OsmPrimitiveComparator());
    214216        }
    215217
Note: See TracChangeset for help on using the changeset viewer.