Changeset 7012 in josm for trunk/src/org/openstreetmap/josm/io/OsmChangeReader.java

Timestamp:

2014-04-27T15:35:47+02:00 (11 years ago)

Author:

Don-vip

Message:

see #8465 - use String switch/case where applicable

File:

• trunk/src/org/openstreetmap/josm/io/OsmChangeReader.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/io/OsmChangeReader.java

@@ -34 +34 @@
     @Override
     protected void parseRoot() throws XMLStreamException {
-        if (parser.getLocalName().equals("osmChange")) {
+        if ("osmChange".equals(parser.getLocalName())) {
             parseOsmChange();
         } else {
@@ -69 +69 @@
             if (event == XMLStreamConstants.START_ELEMENT) {
                 OsmPrimitive p = null;
-                if (parser.getLocalName().equals("node")) {
+                switch (parser.getLocalName()) {
+                case "node":
                     p = parseNode();
-                } else if (parser.getLocalName().equals("way")) {
+                    break;
+                case "way":
                     p = parseWay();
-                } else if (parser.getLocalName().equals("relation")) {
+                    break;
+                case "relation":
                     p = parseRelation();
-                } else {
+                    break;
+                default:
                     parseUnknown();
                 }
                 if (p != null && action != null) {
-                    if (action.equals("modify")) {
+                    if ("modify".equals(action)) {
                         p.setModified(true);
-                    } else if (action.equals("delete")) {
+                    } else if ("delete".equals(action)) {
                         p.setDeleted(true);
                     }