Changeset 13207 in josm for trunk/src/org/openstreetmap/josm/gui

Timestamp:

2017-12-17T01:25:46+01:00 (6 years ago)

Author:

Don-vip

Message:

enable PMD rule PreserveStackTrace + add missing jars to run new PMD rule designer

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• mappaint/RenderingCLI.java (modified) (10 diffs)
• oauth/OsmOAuthAuthorizationClient.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/mappaint/RenderingCLI.java

@@ -231 +231 @@
                 } catch (NumberFormatException nfe) {
                     throw new IllegalArgumentException(
-                            tr("Expected integer number for option {0}, but got ''{1}''", "--zoom", getopt.getOptarg()));
+                            tr("Expected integer number for option {0}, but got ''{1}''", "--zoom", getopt.getOptarg()), nfe);
                 }
                 if (argZoom < 0)
@@ -242 +242 @@
                         argBounds = new Bounds(getopt.getOptarg(), ",", Bounds.ParseMethod.LEFT_BOTTOM_RIGHT_TOP, false);
                     } catch (IllegalArgumentException iae) { // NOPMD
-                        throw new IllegalArgumentException(tr("Unable to parse {0} parameter: {1}", "--bounds", iae.getMessage()));
+                        throw new IllegalArgumentException(tr("Unable to parse {0} parameter: {1}", "--bounds", iae.getMessage()), iae);
                     }
                 }
@@ -268 +268 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected floating point number for option {0}, but got ''{1}''", "--scale", getopt.getOptarg()));
+                                tr("Expected floating point number for option {0}, but got ''{1}''", "--scale", getopt.getOptarg()), nfe);
                     }
                     break;
@@ -282 +282 @@
                         argAnchor = new LatLon(lat, lon);
                     } catch (IllegalArgumentException iae) { // NOPMD
-                        throw new IllegalArgumentException(tr("In option {0}: {1}", "--anchor", iae.getMessage()));
+                        throw new IllegalArgumentException(tr("In option {0}: {1}", "--anchor", iae.getMessage()), iae);
                     }
                     break;
@@ -290 +290 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected floating point number for option {0}, but got ''{1}''", "--width-m", getopt.getOptarg()));
+                                tr("Expected floating point number for option {0}, but got ''{1}''", "--width-m", getopt.getOptarg()), nfe);
                     }
                     if (argWidthM <= 0) throw new IllegalArgumentException(
@@ -300 +300 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected floating point number for option {0}, but got ''{1}''", "--height-m", getopt.getOptarg()));
+                                tr("Expected floating point number for option {0}, but got ''{1}''", "--height-m", getopt.getOptarg()), nfe);
                     }
                     if (argHeightM <= 0) throw new IllegalArgumentException(
@@ -310 +310 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected integer number for option {0}, but got ''{1}''", "--width-px", getopt.getOptarg()));
+                                tr("Expected integer number for option {0}, but got ''{1}''", "--width-px", getopt.getOptarg()), nfe);
                     }
                     if (argWidthPx <= 0) throw new IllegalArgumentException(
@@ -320 +320 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected integer number for option {0}, but got ''{1}''", "--height-px", getopt.getOptarg()));
+                                tr("Expected integer number for option {0}, but got ''{1}''", "--height-px", getopt.getOptarg()), nfe);
                     }
                     if (argHeightPx <= 0) throw new IllegalArgumentException(
@@ -333 +333 @@
                     } catch (NumberFormatException nfe) {
                         throw new IllegalArgumentException(
-                                tr("Expected integer number for option {0}, but got ''{1}''", "--max-image-size", getopt.getOptarg()));
+                                tr("Expected integer number for option {0}, but got ''{1}''", "--max-image-size", getopt.getOptarg()), nfe);
                     }
                     if (argMaxImageSize < 0) throw new IllegalArgumentException(
@@ -551 +551 @@
             return OsmReader.parseDataSet(Files.newInputStream(Paths.get(argInput)), null);
         } catch (IllegalDataException e) {
-            throw new IllegalDataException(tr("In .osm data file ''{0}'' - ", argInput) + e.getMessage());
+            throw new IllegalDataException(tr("In .osm data file ''{0}'' - ", argInput) + e.getMessage(), e);
         }
     }

trunk/src/org/openstreetmap/josm/gui/oauth/OsmOAuthAuthorizationClient.java

@@ -328 +328 @@
                 throw new OsmOAuthAuthorizationException(tr("Failed to authenticate user ''{0}'' with password ''***'' as OAuth user",
                         userName));
-        } catch (OsmOAuthAuthorizationException e) {
-            Logging.debug(e);
-            throw new OsmLoginFailedException(e.getCause());
-        } catch (IOException e) {
+        } catch (OsmOAuthAuthorizationException | IOException e) {
             throw new OsmLoginFailedException(e);
         } finally {