Changeset 12279 in josm for trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

Timestamp:

2017-05-30T15:12:36+02:00 (7 years ago)

Author:

Don-vip

Message:

sonar - squid:S3878 - Arrays should not be created for varargs parameters

File:

• trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

@@ -61 +61 @@
 
     protected static boolean confirmChangeDirectionOfWays() {
-        ExtendedDialog ed = new ExtendedDialog(Main.parent,
+        return new ExtendedDialog(Main.parent,
                 tr("Change directions?"),
-                new String[] {tr("Reverse and Combine"), tr("Cancel")});
-        ed.setButtonIcons(new String[] {"wayflip", "cancel"});
-        ed.setContent(tr("The ways can not be combined in their current directions.  "
-                + "Do you want to reverse some of them?"));
-        ed.toggleEnable("combineway-reverse");
-        ed.showDialog();
-        return ed.getValue() == 1;
+                tr("Reverse and Combine"), tr("Cancel"))
+            .setButtonIcons("wayflip", "cancel")
+            .setContent(tr("The ways can not be combined in their current directions.  "
+                + "Do you want to reverse some of them?"))
+            .toggleEnable("combineway-reverse")
+            .showDialog()
+            .getValue() == 1;
     }