Changeset 12866 in josm for trunk


Ignore:
Timestamp:
2017-09-15T01:35:48+02:00 (2 years ago)
Author:
Don-vip
Message:

SonarQube:

  • squid:S4144 - Methods should not have identical implementations
  • squid:S4165 - Assignments should not be redundant
Location:
trunk/src/org/openstreetmap/josm
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/osm/RelationMemberData.java

    r12190 r12866  
    8383    @Override
    8484    public OsmPrimitiveType getType() {
    85         return memberType;
     85        return getMemberType();
    8686    }
    8787
     
    9191    @Override
    9292    public long getUniqueId() {
    93         return memberId;
     93        return getMemberId();
    9494    }
    9595
  • trunk/src/org/openstreetmap/josm/gui/MapStatus.java

    r12863 r12866  
    671671                 @Override
    672672                 public void mouseDragged(MouseEvent e) {
    673                     l.setBackground(SystemColor.info);
    674                     l.setForeground(SystemColor.infoText);
     673                     mouseMoved(e);
    675674                 }
    676675            });
  • trunk/src/org/openstreetmap/josm/gui/ScrollViewport.java

    r12391 r12866  
    7070
    7171        @Override
    72         public void mouseExited(MouseEvent arg0) {
     72        public void mouseExited(MouseEvent e) {
     73            mouseReleased(e);
     74        }
     75
     76        @Override
     77        public void mouseReleased(MouseEvent e) {
    7378            ScrollViewport.this.scrollDirection = NO_SCROLL;
    7479            timer.stop();
     
    7681
    7782        @Override
    78         public void mouseReleased(MouseEvent arg0) {
    79             ScrollViewport.this.scrollDirection = NO_SCROLL;
    80             timer.stop();
    81         }
    82 
    83         @Override public void mousePressed(MouseEvent arg0) {
     83        public void mousePressed(MouseEvent e) {
    8484            ScrollViewport.this.scrollDirection = direction;
    8585            scroll();
    8686            timer.restart();
    8787        }
    88 
    8988    }
    9089
  • trunk/src/org/openstreetmap/josm/gui/dialogs/CommandStackDialog.java

    r12718 r12866  
    208208            @Override
    209209            public void treeNodesInserted(TreeModelEvent e) {
    210                 updater.updateEnabledState();
    211                 updateTitle();
     210                treeNodesChanged(e);
    212211            }
    213212
    214213            @Override
    215214            public void treeNodesRemoved(TreeModelEvent e) {
    216                 updater.updateEnabledState();
    217                 updateTitle();
     215                treeNodesChanged(e);
    218216            }
    219217
    220218            @Override
    221219            public void treeStructureChanged(TreeModelEvent e) {
    222                 updater.updateEnabledState();
    223                 updateTitle();
     220                treeNodesChanged(e);
    224221            }
    225222        });
  • trunk/src/org/openstreetmap/josm/gui/io/ActionFlagsTableCell.java

    r11534 r12866  
    101101    @Override
    102102    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {
    103         updatePanel((SaveLayerInfo) value);
    104         updateCheckboxes(value);
    105         return this;
     103        return getTableCellEditorComponent(table, value, isSelected, row, column);
    106104    }
    107105
  • trunk/src/org/openstreetmap/josm/gui/widgets/ButtonColumn.java

    r11615 r12866  
    8686            editButton.setIcon(null);
    8787        }
    88         this.editorValue = value;
    8988        return editButton;
    9089    }
Note: See TracChangeset for help on using the changeset viewer.