Changeset 10173 in josm


Ignore:
Timestamp:
2016-05-10T14:54:38+02:00 (9 years ago)
Author:
Don-vip
Message:

sonar - squid:S1186 - Methods should not be empty

Location:
trunk/src/org/openstreetmap/josm
Files:
39 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/ImageryMenu.java

    r9261 r10173  
    121121            @Override
    122122            public void menuDeselected(MenuEvent e) {
     123                // Do nothing
    123124            }
    124125
    125126            @Override
    126127            public void menuCanceled(MenuEvent e) {
     128                // Do nothing
    127129            }
    128130        });
    … …  
    238240    @Override
    239241    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
     242        // Do nothing
    240243    }
    241244

  • trunk/src/org/openstreetmap/josm/gui/MainMenu.java

    r9443 r10173  
    3333import org.openstreetmap.josm.actions.ChangesetManagerToggleAction;
    3434import org.openstreetmap.josm.actions.CloseChangesetAction;
    35 import org.openstreetmap.josm.actions.DeleteLayerAction;
    3635import org.openstreetmap.josm.actions.CombineWayAction;
    3736import org.openstreetmap.josm.actions.CopyAction;
    … …  
    4039import org.openstreetmap.josm.actions.CreateMultipolygonAction;
    4140import org.openstreetmap.josm.actions.DeleteAction;
     41import org.openstreetmap.josm.actions.DeleteLayerAction;
    4242import org.openstreetmap.josm.actions.DialogsToggleAction;
    4343import org.openstreetmap.josm.actions.DistributeAction;
    … …  
    403403    public static final MenuListener menuSeparatorHandler = new MenuListener() {
    404404        @Override
    405         public void menuCanceled(MenuEvent e) {}
     405        public void menuCanceled(MenuEvent e) {
     406            // Do nothing
     407        }
    406408
    407409        @Override
    408         public void menuDeselected(MenuEvent e) {}
     410        public void menuDeselected(MenuEvent e) {
     411            // Do nothing
     412        }
    409413
    410414        @Override

  • trunk/src/org/openstreetmap/josm/gui/MapFrame.java

    r9997 r10173  
    220220                    @Override
    221221                    public void setBorder(Border b) {
     222                        // Do nothing
    222223                    }
    223224                };
    … …  
    546547
    547548                @Override
    548                 public void popupMenuCanceled(PopupMenuEvent e) {}
     549                public void popupMenuCanceled(PopupMenuEvent e) {
     550                    // Do nothing
     551                }
    549552            });
    550553
    … …  
    777780
    778781    @Override
    779     public void layerAdded(Layer newLayer) { }
     782    public void layerAdded(Layer newLayer) {
     783        // Do nothing
     784    }
    780785
    781786    @Override

  • trunk/src/org/openstreetmap/josm/gui/PleaseWaitDialog.java

    r10035 r10173  
    88import java.awt.GridBagLayout;
    99import java.awt.event.ActionListener;
     10import java.awt.event.ComponentAdapter;
    1011import java.awt.event.ComponentEvent;
    11 import java.awt.event.ComponentListener;
    1212
    1313import javax.swing.BorderFactory;
    … …  
    6565        setCustomText("");
    6666        setLocationRelativeTo(getParent());
    67         addComponentListener(new ComponentListener() {
    68             @Override
    69             public void componentHidden(ComponentEvent e) {}
    70 
    71             @Override
    72             public void componentMoved(ComponentEvent e) {}
    73 
    74             @Override
    75             public void componentShown(ComponentEvent e) {}
    76 
     67        addComponentListener(new ComponentAdapter() {
    7768            @Override
    7869            public void componentResized(ComponentEvent ev) {

  • trunk/src/org/openstreetmap/josm/gui/SelectionManager.java

    r10031 r10173  
    453453
    454454    @Override
    455     public void mouseClicked(MouseEvent e) {}
    456 
    457     @Override
    458     public void mouseEntered(MouseEvent e) {}
    459 
    460     @Override
    461     public void mouseExited(MouseEvent e) {}
    462 
    463     @Override
    464     public void mouseMoved(MouseEvent e) {}
     455    public void mouseClicked(MouseEvent e) {
     456        // Do nothing
     457    }
     458
     459    @Override
     460    public void mouseEntered(MouseEvent e) {
     461        // Do nothing
     462    }
     463
     464    @Override
     465    public void mouseExited(MouseEvent e) {
     466        // Do nothing
     467    }
     468
     469    @Override
     470    public void mouseMoved(MouseEvent e) {
     471        // Do nothing
     472    }
    465473}

  • trunk/src/org/openstreetmap/josm/gui/SplashScreen.java

    r10121 r10173  
    310310        @Override
    311311        public void setTicks(int ticks) {
     312            // Not used
    312313        }
    313314

  • trunk/src/org/openstreetmap/josm/gui/dialogs/OsmIdSelectionDialog.java

    r10044 r10173  
    251251    }
    252252
    253     @Override public void windowOpened(WindowEvent e) {}
    254 
    255     @Override public void windowClosing(WindowEvent e) {}
    256 
    257     @Override public void windowIconified(WindowEvent e) {}
    258 
    259     @Override public void windowDeiconified(WindowEvent e) {}
    260 
    261     @Override public void windowActivated(WindowEvent e) {}
    262 
    263     @Override public void windowDeactivated(WindowEvent e) {}
     253    @Override public void windowOpened(WindowEvent e) {
     254        // Do nothing
     255    }
     256
     257    @Override public void windowClosing(WindowEvent e) {
     258        // Do nothing
     259    }
     260
     261    @Override public void windowIconified(WindowEvent e) {
     262        // Do nothing
     263    }
     264
     265    @Override public void windowDeiconified(WindowEvent e) {
     266        // Do nothing
     267    }
     268
     269    @Override public void windowActivated(WindowEvent e) {
     270        // Do nothing
     271    }
     272
     273    @Override public void windowDeactivated(WindowEvent e) {
     274        // Do nothing
     275    }
    264276}

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java

    r10043 r10173  
    459459    /**
    460460     * Closes the detached dialog if this toggle dialog is currently displayed in a detached dialog.
    461      *
    462461     */
    463462    public void closeDetachedDialog() {
    … …  
    474473     */
    475474    public void showNotify() {
    476 
    477     }
    478 
    479     /**
    480      * Called when toggle dialog is hidden (collapsed, removed, MapFrame is removed, ...). Good place to unregister
    481      * listeners
     475        // Do nothing
     476    }
     477
     478    /**
     479     * Called when toggle dialog is hidden (collapsed, removed, MapFrame is removed, ...). Good place to unregister listeners
    482480     */
    483481    public void hideNotify() {
    484 
     482        // Do nothing
    485483    }
    486484
    487485    /**
    488486     * The title bar displayed in docked mode
    489      *
    490487     */
    491488    protected class TitleBar extends JPanel {
    … …  
    855852     */
    856853    protected void toggleButtonHook() {
     854        // Do nothing
    857855    }
    858856
    … …  
    865863     */
    866864    protected void stateChanged() {
     865        // Do nothing
    867866    }
    868867

  • trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

    r10035 r10173  
    477477
    478478    protected void buildMainPanelAboveDownloadSelections(JPanel pnl) {
     479        // Do nothing
    479480    }
    480481

  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserDialogManager.java

    r10055 r10173  
    153153    /* ----------------------------------------------------------------------------- */
    154154    @Override
    155     public void activeLayerChange(Layer oldLayer, Layer newLayer) {}
     155    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
     156        // Do nothing
     157    }
    156158
    157159    @Override
    158     public void layerAdded(Layer newLayer) {}
     160    public void layerAdded(Layer newLayer) {
     161        // Do nothing
     162    }
    159163
    160164    @Override

  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r10109 r10173  
    761761
    762762    @Override
    763     public void layerAdded(Layer newLayer) {}
    764 
    765     @Override
    766     public void layerRemoved(Layer oldLayer) {}
     763    public void layerAdded(Layer newLayer) {
     764        // Do nothing
     765    }
     766
     767    @Override
     768    public void layerRemoved(Layer oldLayer) {
     769        // Do nothing
     770    }
    767771
    768772    /**

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

    r9685 r10173  
    1010import java.awt.event.FocusAdapter;
    1111import java.awt.event.FocusEvent;
     12import java.awt.event.KeyAdapter;
    1213import java.awt.event.KeyEvent;
    13 import java.awt.event.KeyListener;
    1414import java.util.Arrays;
    1515import java.util.Collections;
    … …  
    135135        hcb.getEditor().addActionListener(handler);
    136136        hcb.getEditorComponent().addKeyListener(
    137                 new KeyListener() {
     137                new KeyAdapter() {
    138138                    @Override
    139139                    public void keyTyped(KeyEvent e) {
    … …  
    141141                            handler.actionPerformed(new ActionEvent(hcb, 0, "focusDown"));
    142142                        }
    143                     }
    144 
    145                     @Override
    146                     public void keyReleased(KeyEvent e) {
    147                         // Do nothing
    148                     }
    149 
    150                     @Override
    151                     public void keyPressed(KeyEvent e) {
    152                         // Do nothing
    153143                    }
    154144                }

  • trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java

    r9776 r10173  
    1414import java.awt.event.FocusAdapter;
    1515import java.awt.event.FocusEvent;
     16import java.awt.event.KeyAdapter;
    1617import java.awt.event.KeyEvent;
    17 import java.awt.event.KeyListener;
    1818import java.awt.event.WindowAdapter;
    1919import java.awt.event.WindowEvent;
    … …  
    344344     *   If both text fields contain characters, submits the form by calling owner's {@link OKAction}.
    345345     */
    346     private static class TFKeyListener implements KeyListener {
     346    private static class TFKeyListener extends KeyAdapter {
    347347        protected CredentialDialog owner; // owner Dependency Injection to call OKAction
    348348        protected JTextField currentTF;
    … …  
    371371            }
    372372        }
    373 
    374         @Override
    375         public void keyReleased(KeyEvent e) {
    376         }
    377 
    378         @Override
    379         public void keyTyped(KeyEvent e) {
    380         }
    381373    }
    382374

  • trunk/src/org/openstreetmap/josm/gui/io/DownloadFileTask.java

    r9645 r10173  
    8585
    8686    @Override
    87     protected void finish() {}
     87    protected void finish() {
     88        // Do nothing
     89    }
    8890
    8991    /**

  • trunk/src/org/openstreetmap/josm/gui/io/RecentlyOpenedFilesMenu.java

    r8836 r10173  
    2727    private ClearAction clearAction;
    2828
     29    /**
     30     * Constructs a new {@code RecentlyOpenedFilesMenu}.
     31     */
    2932    public RecentlyOpenedFilesMenu() {
    3033        super(tr("Open Recent"));
    … …  
    4245            @Override
    4346            public void menuDeselected(MenuEvent e) {
     47                // Do nothing
    4448            }
    4549
    4650            @Override
    4751            public void menuCanceled(MenuEvent e) {
     52                // Do nothing
    4853            }
    4954        });

  • trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java

    r9841 r10173  
    1212import java.awt.event.ActionEvent;
    1313import java.awt.event.ActionListener;
     14import java.awt.event.FocusAdapter;
    1415import java.awt.event.FocusEvent;
    15 import java.awt.event.FocusListener;
    1616import java.awt.event.ItemEvent;
    1717import java.awt.event.ItemListener;
    … …  
    389389    }
    390390
    391     static class TextFieldFocusHandler implements FocusListener {
     391    static class TextFieldFocusHandler extends FocusAdapter {
    392392        @Override
    393393        public void focusGained(FocusEvent e) {
    … …  
    398398            }
    399399        }
    400 
    401         @Override
    402         public void focusLost(FocusEvent e) {}
    403400    }
    404401
    … …  
    467464    }
    468465
    469     class StrategyChangeListener implements ItemListener, FocusListener, ActionListener {
     466    class StrategyChangeListener extends FocusAdapter implements ItemListener, ActionListener {
    470467
    471468        protected void notifyStrategy() {
    … …  
    489486
    490487        @Override
    491         public void focusGained(FocusEvent arg0) {}
    492 
    493         @Override
    494488        public void focusLost(FocusEvent arg0) {
    495489            notifyStrategy();

  • trunk/src/org/openstreetmap/josm/gui/layer/ImageryLayer.java

    r10142 r10173  
    195195        @Override
    196196        public void actionPerformed(ActionEvent e) {
     197            // Do nothing
    197198        }
    198199

  • trunk/src/org/openstreetmap/josm/gui/layer/ValidatorLayer.java

    r10008 r10173  
    108108    @Override
    109109    public void mergeFrom(Layer from) {
     110        // Do nothing
    110111    }
    111112
    … …  
    122123    @Override
    123124    public void visitBoundingBox(BoundingXYVisitor v) {
     125        // Do nothing
    124126    }
    125127
    … …  
    142144    @Override
    143145    public void activeLayerChange(Layer oldLayer, Layer newLayer) {
     146        // Do nothing
    144147    }
    145148
    146149    @Override
    147150    public void layerAdded(Layer newLayer) {
     151        // Do nothing
    148152    }
    149153

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r10001 r10173  
    793793        @Override
    794794        public void changedUpdate(DocumentEvent ev) {
     795            // Do nothing
    795796        }
    796797

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r10031 r10173  
    900900            @Override
    901901            public void layerAdded(Layer newLayer) {
     902                // Do nothing
    902903            }
    903904

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java

    r9079 r10173  
    402402        @Override
    403403        public void mouseEntered(MouseEvent e) {
     404            // Do nothing
    404405        }
    405406
    406407        @Override
    407408        public void mouseExited(MouseEvent e) {
     409            // Do nothing
    408410        }
    409411
    410412        @Override
    411413        public void mouseMoved(MouseEvent e) {
     414            // Do nothing
    412415        }
    413416

  • trunk/src/org/openstreetmap/josm/gui/layer/gpx/DownloadAlongTrackAction.java

    r10001 r10173  
    119119            @Override
    120120            protected void finish() {
     121                // Do nothing
    121122            }
    122123

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java

    r9730 r10173  
    388388     */
    389389    public void actionPerformed(ActionEvent ev) {
     390        // Do nothing
    390391    }
    391392

  • trunk/src/org/openstreetmap/josm/gui/oauth/FullyAutomaticAuthorizationUI.java

    r10073 r10173  
    411411
    412412        @Override
    413         protected void finish() {}
     413        protected void finish() {
     414            // Do nothing
     415        }
    414416
    415417        protected void alertAuthorisationFailed(OsmOAuthAuthorizationException e) {

  • trunk/src/org/openstreetmap/josm/gui/oauth/OAuthAuthorizationWizard.java

    r10035 r10173  
    1313import java.awt.Insets;
    1414import java.awt.event.ActionEvent;
     15import java.awt.event.ComponentAdapter;
    1516import java.awt.event.ComponentEvent;
    16 import java.awt.event.ComponentListener;
    1717import java.awt.event.ItemEvent;
    1818import java.awt.event.ItemListener;
    … …  
    216216        spAuthorisationProcedureUI = GuiHelper.embedInVerticalScrollPane(new JPanel());
    217217        spAuthorisationProcedureUI.getVerticalScrollBar().addComponentListener(
    218                 new ComponentListener() {
     218                new ComponentAdapter() {
    219219                    @Override
    220220                    public void componentShown(ComponentEvent e) {
    … …  
    226226                        spAuthorisationProcedureUI.setBorder(null);
    227227                    }
    228 
    229                     @Override
    230                     public void componentResized(ComponentEvent e) {}
    231 
    232                     @Override
    233                     public void componentMoved(ComponentEvent e) {}
    234228                }
    235229        );

  • trunk/src/org/openstreetmap/josm/gui/oauth/TestAccessTokenTask.java

    r9309 r10173  
    7878
    7979    @Override
    80     protected void finish() {}
     80    protected void finish() {
     81        // Do nothing
     82    }
    8183
    8284    protected void sign(HttpClient con) throws OAuthException {

  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r9983 r10173  
    462462
    463463                @Override
    464                 public void popupMenuWillBecomeInvisible(PopupMenuEvent e) {}
    465 
    466                 @Override
    467                 public void popupMenuCanceled(PopupMenuEvent e) {}
     464                public void popupMenuWillBecomeInvisible(PopupMenuEvent e) {
     465                    // Do nothing
     466                }
     467
     468                @Override
     469                public void popupMenuCanceled(PopupMenuEvent e) {
     470                    // Do nothing
     471                }
    468472            });
    469473            add(remove);
    … …  
    820824            p.setLayout(new LayoutManager() {
    821825                @Override
    822                 public void addLayoutComponent(String name, Component comp) {}
    823 
    824                 @Override
    825                 public void removeLayoutComponent(Component comp) {}
     826                public void addLayoutComponent(String name, Component comp) {
     827                    // Do nothing
     828                }
     829
     830                @Override
     831                public void removeLayoutComponent(Component comp) {
     832                    // Do nothing
     833                }
    826834
    827835                @Override

  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/SingleProjectionChoice.java

    r8444 r10173  
    5454    @Override
    5555    public void setPreferences(Collection<String> args) {
     56        // Do nothing
    5657    }
    5758

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/ApiUrlTestTask.java

    r9309 r10173  
    143143
    144144    @Override
    145     protected void finish() {}
     145    protected void finish() {
     146        // Do nothing
     147    }
    146148
    147149    /**

  • trunk/src/org/openstreetmap/josm/gui/progress/ChildProgress.java

    r9531 r10173  
    3030    @Override
    3131    protected void doBeginTask() {
     32        // Do nothing
    3233    }
    3334

  • trunk/src/org/openstreetmap/josm/gui/progress/NullProgressMonitor.java

    r10121 r10173  
    1616
    1717    private NullProgressMonitor() {
    18 
     18        // Do nothing
    1919    }
    2020
    2121    @Override
    2222    public void addCancelListener(CancelListener listener) {
     23        // Do nothing
    2324    }
    2425
    … …  
    3940    @Override
    4041    public void cancel() {
     42        // Do nothing
    4143    }
    4244
    … …  
    4850    @Override
    4951    public void finishTask() {
    50     }
    51 
    52     public String getErrorMessage() {
    53         return null;
     52        // Do nothing
    5453    }
    5554
    … …  
    6867    @Override
    6968    public void invalidate() {
     69        // Do nothing
    7070    }
    7171
    … …  
    7777    @Override
    7878    public void removeCancelListener(CancelListener listener) {
     79        // Do nothing
    7980    }
    8081
    8182    @Override
    8283    public void setCustomText(String text) {
    83     }
    84 
    85     public void setErrorMessage(String message) {
     84        // Do nothing
    8685    }
    8786
    8887    @Override
    8988    public void setExtraText(String text) {
     89        // Do nothing
    9090    }
    9191
    9292    @Override
    9393    public void appendLogMessage(String message) {
    94     }
    95 
    96     public void setSilent(boolean value) {
     94        // Do nothing
    9795    }
    9896
    9997    @Override
    10098    public void setTicks(int ticks) {
     99        // Do nothing
    101100    }
    102101
    103102    @Override
    104103    public void setTicksCount(int ticks) {
     104        // Do nothing
    105105    }
    106106
    … …  
    114114    @Override
    115115    public void worked(int ticks) {
     116        // Do nothing
    116117    }
    117118
    … …  
    123124    @Override
    124125    public void setProgressTaskId(ProgressTaskId taskId) {
     126        // Do nothing
    125127    }
    126128

  • trunk/src/org/openstreetmap/josm/gui/progress/SwingRenderingProgressMonitor.java

    r9078 r10173  
    9494    @Override
    9595    public void setProgressTaskId(ProgressTaskId taskId) {
     96        // Do nothing
    9697    }
    9798

  • trunk/src/org/openstreetmap/josm/gui/widgets/MultiSplitPane.java

    r9970 r10173  
    384384
    385385        @Override
    386         public void keyReleased(KeyEvent e) { }
    387 
    388         @Override
    389         public void keyTyped(KeyEvent e) { }
     386        public void keyReleased(KeyEvent e) {
     387            // Do nothing
     388        }
     389
     390        @Override
     391        public void keyTyped(KeyEvent e) {
     392            // Do nothing
     393        }
    390394    }
    391395

  • trunk/src/org/openstreetmap/josm/gui/widgets/PopupMenuLauncher.java

    r8513 r10173  
    44import java.awt.Component;
    55import java.awt.Point;
     6import java.awt.event.FocusAdapter;
    67import java.awt.event.FocusEvent;
    7 import java.awt.event.FocusListener;
    88import java.awt.event.MouseAdapter;
    99import java.awt.event.MouseEvent;
    … …  
    9595    protected void checkFocusAndShowMenu(final Component component, final MouseEvent evt) {
    9696        if (component != null && component.isFocusable() && !component.hasFocus() && component.requestFocusInWindow()) {
    97             component.addFocusListener(new FocusListener() {
    98                 @Override
    99                 public void focusLost(FocusEvent e) {}
    100 
     97            component.addFocusListener(new FocusAdapter() {
    10198                @Override
    10299                public void focusGained(FocusEvent e) {

  • trunk/src/org/openstreetmap/josm/gui/widgets/QuadStateCheckBox.java

    r9978 r10173  
    9292    /** Do not let anyone add mouse listeners */
    9393    @Override
    94     public void addMouseListener(MouseListener l) { }
     94    public void addMouseListener(MouseListener l) {
     95        // Do nothing
     96    }
    9597
    9698    /**
    … …  
    211213
    212214        @Override
    213         public void setArmed(boolean b) { }
    214 
    215         @Override
    216         public void setSelected(boolean b) { }
    217 
    218         @Override
    219         public void setPressed(boolean b) { }
     215        public void setArmed(boolean b) {
     216            // Do nothing
     217        }
     218
     219        @Override
     220        public void setSelected(boolean b) {
     221            // Do nothing
     222        }
     223
     224        @Override
     225        public void setPressed(boolean b) {
     226            // Do nothing
     227        }
    220228
    221229        /** We disable focusing on the component when it is not enabled. */

  • trunk/src/org/openstreetmap/josm/io/GeoJSONWriter.java

    r10171 r10173  
    114114        @Override
    115115        public void visit(IRelation r) {
     116            // Do nothing
    116117        }
    117118    }

  • trunk/src/org/openstreetmap/josm/plugins/ReadLocalPluginInformationTask.java

    r10123 r10173  
    5858
    5959    @Override
    60     protected void finish() {}
     60    protected void finish() {
     61        // Do nothing
     62    }
    6163
    6264    protected void processJarFile(File f, String pluginName) throws PluginException {

  • trunk/src/org/openstreetmap/josm/plugins/ReadRemotePluginInformationTask.java

    r10043 r10173  
    9797
    9898    @Override
    99     protected void finish() {}
     99    protected void finish() {
     100        // Do nothing
     101    }
    100102
    101103    /**

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r10166 r10173  
    108108    @Override
    109109    public void afterPrefStartupHook() {
     110        // Do nothing
    110111    }
    111112
Note: See TracChangeset for help on using the changeset viewer.