Index: /trunk/scripts/SyncEditorLayerIndex.groovy
===================================================================
--- /trunk/scripts/SyncEditorLayerIndex.groovy	(revision 13826)
+++ /trunk/scripts/SyncEditorLayerIndex.groovy	(revision 13827)
@@ -346,5 +346,5 @@
                     shapes += sep + "</shape>\n"
                 }
-            } catch(IllegalArgumentException) {
+            } catch(IllegalArgumentException ignored) {
             }
             if(shapes) {
Index: /trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java	(revision 13826)
+++ /trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java	(revision 13827)
@@ -191,6 +191,6 @@
             formats.setToolTipText(tr("Select image format for WMS layer"));
 
-            if (!GraphicsEnvironment.isHeadless()) {
-                if (1 != new ExtendedDialog(Main.parent, tr("Select WMS layers"), tr("Add layers"), tr("Cancel")) { {
+            if (!GraphicsEnvironment.isHeadless() &&
+                1 != new ExtendedDialog(Main.parent, tr("Select WMS layers"), tr("Add layers"), tr("Cancel")) { {
                     final JScrollPane scrollPane = new JScrollPane(tree.getLayerTree());
                     scrollPane.setPreferredSize(new Dimension(400, 400));
@@ -199,7 +199,6 @@
                     panel.add(formats, GBC.eol().fill(GBC.HORIZONTAL));
                     setContent(panel);
-                } }.showDialog().getValue()) {
-                    return null;
-                }
+            } }.showDialog().getValue()) {
+                return null;
             }
 
Index: /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java	(revision 13826)
+++ /trunk/src/org/openstreetmap/josm/io/imagery/WMSImagery.java	(revision 13827)
@@ -466,13 +466,8 @@
                 }
                 if (tagEquals(QN_LAYER, reader.getName())) {
-
                     parseLayer(reader, ret);
                 }
-                if (tagEquals(QN_EX_GEOGRAPHIC_BBOX, reader.getName())) {
-                    if (ret.getBounds() == null) {
-                        Bounds bbox = parseExGeographic(reader);
-                        ret.setBounds(bbox);
-                    }
-
+                if (tagEquals(QN_EX_GEOGRAPHIC_BBOX, reader.getName()) && ret.getBounds() == null) {
+                    ret.setBounds(parseExGeographic(reader));
                 }
                 if (tagEquals(QN_BOUNDINGBOX, reader.getName())) {
@@ -484,13 +479,9 @@
                     }
                     if (ret.getBounds() == null && conv != null) {
-                        Bounds bbox = parseBoundingBox(reader, conv);
-                        ret.setBounds(bbox);
+                        ret.setBounds(parseBoundingBox(reader, conv));
                     }
                 }
-                if (tagEquals(QN_LATLONBOUNDINGBOX, reader.getName()) && belowWMS130()) {
-                    if (ret.getBounds() == null) {
-                        Bounds bbox = parseBoundingBox(reader, null);
-                        ret.setBounds(bbox);
-                    }
+                if (tagEquals(QN_LATLONBOUNDINGBOX, reader.getName()) && belowWMS130() && ret.getBounds() == null) {
+                    ret.setBounds(parseBoundingBox(reader, null));
                 }
             }
Index: /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddWayHandler.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddWayHandler.java	(revision 13826)
+++ /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddWayHandler.java	(revision 13827)
@@ -94,5 +94,5 @@
     protected void validateRequest() throws RequestHandlerBadRequestException {
         allCoordinates.clear();
-        for (String coordinatesString : (args != null ? args.get("way") : "").split(";\\s*")) {
+        for (String coordinatesString : splitArg("way", SPLITTER_SEMIC)) {
             String[] coordinates = coordinatesString.split(",\\s*", 2);
             if (coordinates.length < 2) {
Index: /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java	(revision 13826)
+++ /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java	(revision 13827)
@@ -98,5 +98,5 @@
     protected void validateRequest() throws RequestHandlerBadRequestException {
         ps.clear();
-        for (String i : (args != null ? args.get("objects") : "").split(",\\s*")) {
+        for (String i : splitArg("objects", SPLITTER_COMMA)) {
             if (!i.isEmpty()) {
                 try {
Index: /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java
===================================================================
--- /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java	(revision 13826)
+++ /trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java	(revision 13827)
@@ -14,4 +14,5 @@
 import java.util.Map;
 import java.util.Set;
+import java.util.regex.Pattern;
 
 import javax.swing.JLabel;
@@ -36,4 +37,7 @@
     public static final String loadInNewLayerKey = "remotecontrol.new-layer";
     public static final boolean loadInNewLayerDefault = false;
+
+    protected static final Pattern SPLITTER_COMMA = Pattern.compile(",\\s*");
+    protected static final Pattern SPLITTER_SEMIC = Pattern.compile(";\\s*");
 
     /** past confirmations */
@@ -215,4 +219,8 @@
     protected void parseArgs() throws URISyntaxException {
         this.args = getRequestParameter(new URI(this.request));
+    }
+
+    protected final String[] splitArg(String arg, Pattern splitter) {
+        return splitter.split(args != null ? args.get(arg) : "");
     }