Changeset 8291 in josm for trunk/src/org/openstreetmap/josm/tools

Timestamp:

2015-04-29T01:44:01+02:00 (10 years ago)

Author:

Don-vip

Message:

fix squid:RedundantThrowsDeclarationCheck + consistent Javadoc for exceptions

Location:

trunk/src/org/openstreetmap/josm/tools

Files:

• CheckParameterUtil.java (modified) (1 diff)
• OpenBrowser.java (modified) (2 diffs)
• OsmUrlToBounds.java (modified) (2 diffs)
• Utils.java (modified) (2 diffs)
• WindowGeometry.java (modified) (1 diff)

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

@@ -106 +106 @@
      * @param id  the primitive  id
      * @param parameterName the name of the parameter to be checked
-     * @throws IllegalArgumentException thrown if id is null
-     * @throws IllegalArgumentException thrown if id.getType() != NODE
+     * @throws IllegalArgumentException if id is null
+     * @throws IllegalArgumentException if id.getType() != NODE
      */
     public static void ensureValidNodeId(PrimitiveId id, String parameterName) throws IllegalArgumentException {

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

@@ -34 +34 @@
      * @param uri The URI to display
      * @return <code>null</code> for success or a string in case of an error.
-     * @throws IllegalStateException thrown if no platform is set to which opening the URL can be dispatched,
+     * @throws IllegalStateException if no platform is set to which opening the URL can be dispatched,
      * {@link Main#platform}
      */
@@ -81 +81 @@
      * @param url The URL to display
      * @return <code>null</code> for success or a string in case of an error.
-     * @throws IllegalStateException thrown if no platform is set to which opening the URL can be dispatched,
+     * @throws IllegalStateException if no platform is set to which opening the URL can be dispatched,
      * {@link Main#platform}
      */

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

@@ -22 +22 @@
     }
 
-    public static Bounds parse(String url) throws IllegalArgumentException {
+    public static Bounds parse(String url) {
         try {
             // a percent sign indicates an encoded URL (RFC 1738).
@@ -83 +83 @@
      * @return Bounds if hashurl, {@code null} otherwise
      */
-    private static Bounds parseHashURLs(String url) throws IllegalArgumentException {
+    private static Bounds parseHashURLs(String url) {
         int startIndex = url.indexOf("#map=");
         if (startIndex == -1) return null;

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

@@ -353 +353 @@
      * @param out The destination file
      * @return the path to the target file
-     * @throws java.io.IOException If any I/O error occurs
-     * @throws IllegalArgumentException If {@code in} or {@code out} is {@code null}
+     * @throws IOException if any I/O error occurs
+     * @throws IllegalArgumentException if {@code in} or {@code out} is {@code null}
      * @since 7003
      */
@@ -367 +367 @@
      * @param in The source directory
      * @param out The destination directory
-     * @throws IOException If any I/O error ooccurs
-     * @throws IllegalArgumentException If {@code in} or {@code out} is {@code null}
+     * @throws IOException if any I/O error ooccurs
+     * @throws IllegalArgumentException if {@code in} or {@code out} is {@code null}
      * @since 7835
      */

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

@@ -222 +222 @@
      *
      * @param preferenceKey the preference key
-     * @throws WindowGeometryException thrown if no such key exist or if the preference value has
+     * @throws WindowGeometryException if no such key exist or if the preference value has
      * an illegal format
      */