Changeset 12279 in josm for trunk/src/org/openstreetmap/josm/actions/SaveActionBase.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/SaveActionBase.java (modified) (1 diff)

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

@@ -207 +207 @@
     public static boolean confirmOverwrite(File file) {
         if (file == null || file.exists()) {
-            ExtendedDialog dialog = new ExtendedDialog(
+            return new ExtendedDialog(
                     Main.parent,
                     tr("Overwrite"),
-                    new String[] {tr("Overwrite"), tr("Cancel")}
-            );
-            dialog.setContent(tr("File exists. Overwrite?"));
-            dialog.setButtonIcons(new String[] {"save_as", "cancel"});
-            dialog.showDialog();
-            return dialog.getValue() == 1;
+                    tr("Overwrite"), tr("Cancel"))
+                .setContent(tr("File exists. Overwrite?"))
+                .setButtonIcons("save_as", "cancel")
+                .showDialog()
+                .getValue() == 1;
         }
         return true;