Changeset 17518 in josm for trunk/src/org

Timestamp:

2021-02-22T00:07:10+01:00 (3 years ago)

Author:

Don-vip

Message:

spotbugs - SF_SWITCH_NO_DEFAULT

File:

• trunk/src/org/openstreetmap/josm/gui/history/NodeListTableCellRenderer.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/history/NodeListTableCellRenderer.java

@@ -18 +18 @@
 /**
  * A {@link TableCellRenderer} for the {@link NodeListViewer}.
- * 
+ *
  * Renders information about a node when comparing the node list of two
  * historical versions of a way.
@@ -24 +24 @@
 public class NodeListTableCellRenderer extends JLabel implements TableCellRenderer {
 
+    /** Color of selected background */
     public static final Color BGCOLOR_SELECTED = new Color(143, 170, 255);
 
@@ -63 +64 @@
     public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus,
             int row, int column) {
-
         if (value == null) return this;
         TwoColumnDiff.Item item = (TwoColumnDiff.Item) value;
         GuiHelper.setBackgroundReadable(this, item.state.getColor(isSelected, hasFocus));
-        switch(column) {
-        case NodeListTableColumnModel.INDEX_COLUMN:
+        if (column == NodeListTableColumnModel.INDEX_COLUMN) {
             renderIndex((DiffTableModel) table.getModel(), row);
-            break;
-        case NodeListTableColumnModel.NODE_COLUMN:
+        } else if (column == NodeListTableColumnModel.NODE_COLUMN) {
             renderNode(item);
-            break;
         }
         return this;