Changeset 8855 in josm for trunk/src/org/openstreetmap/josm/gui/tagging

Timestamp:

2015-10-10T21:01:42+02:00 (9 years ago)

Author:

Don-vip

Message:

sonar - Unused private method should be removed
sonar - Unused protected methods should be removed
sonar - Sections of code should not be "commented out"
sonar - Empty statements should be removed
sonar - squid:S1172 - Unused method parameters should be removed
sonar - squid:S1481 - Unused local variables should be removed

Location:

trunk/src/org/openstreetmap/josm/gui/tagging

Files:

• TagCellRenderer.java (modified) (1 diff)
• TaggingPresetItems.java (modified) (2 diffs)
• ac/AutoCompletionList.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/tagging/TagCellRenderer.java

@@ -69 +69 @@
     }
 
-    protected TagEditorModel getModel(JTable table) {
-        return (TagEditorModel) table.getModel();
-    }
-
     /**
      * replies the cell renderer component for a specific cell

trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPresetItems.java

@@ -233 +233 @@
         }
 
-        public boolean addToPanel(JPanel p, Collection<OsmPrimitive> sel) {
+        public boolean addToPanel(JPanel p) {
             String cstring;
             if (count > 0 && !required) {
@@ -477 +477 @@
                 proles.add(new JLabel(tr("elements")), GBC.eol());
                 for (Role i : roles) {
-                    i.addToPanel(proles, sel);
+                    i.addToPanel(proles);
                 }
                 p.add(proles, GBC.eol());

trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionList.java

@@ -231 +231 @@
         if (filter == null) {
             // Collections.copy throws an exception "Source does not fit in dest"
-            // Collections.copy(filtered, list);
             filtered.ensureCapacity(list.size());
             for (AutoCompletionListItem item: list) {