Changeset 13468 in josm for trunk/src/org

Timestamp:

2018-02-26T20:03:44+01:00 (6 years ago)

Author:

Don-vip

Message:

fix #15992 - fix codestyle issues

Location:

trunk/src/org/openstreetmap/josm/tools

Files:

• PlatformHookWindows.java (modified) (2 diffs)
• Utils.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java

@@ -708 +708 @@
                 Matcher m = Pattern.compile("(\\d+)\\.(\\d+)(\\.\\d+.*)?").matcher(version);
                 if (m.matches()) {
-                    int maj = Integer.valueOf(m.group(1));
-                    int min = Integer.valueOf(m.group(2));
+                    int maj = Integer.parseInt(m.group(1));
+                    int min = Integer.parseInt(m.group(2));
                     return (maj == 4 && min >= 5) || maj > 4;
                 }
@@ -726 +726 @@
     public static int getPowerShellVersion() {
         try {
-            return Integer.valueOf(Utils.execOutput(Arrays.asList(
+            return Integer.parseInt(Utils.execOutput(Arrays.asList(
                     "powershell", "-Command", "$PSVersionTable.PSVersion.Major"), 2, TimeUnit.SECONDS));
         } catch (NumberFormatException | IOException | ExecutionException | InterruptedException e) {

trunk/src/org/openstreetmap/josm/tools/Utils.java

@@ -869 +869 @@
             }
             String msg = all != null ? all.toString() : null;
-            if (!p.waitFor(timeout, unit)) {
-                throw new ExecutionException(msg, null);
-            }
-            if (p.exitValue() != 0) {
+            if (!p.waitFor(timeout, unit) || p.exitValue() != 0) {
                 throw new ExecutionException(msg, null);
             }