Changeset 8291 in josm for trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

Timestamp:

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

Author:

Don-vip

Message:

fix squid:RedundantThrowsDeclarationCheck + consistent Javadoc for exceptions

File:

• trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java (modified) (9 diffs)

trunk/src/org/openstreetmap/josm/io/ChangesetQuery.java

@@ -32 +32 @@
      * @param query the query part
      * @return the query object
-     * @throws ChangesetQueryUrlException thrown if query doesn't consist of valid query parameters
-     *
+     * @throws ChangesetQueryUrlException if query doesn't consist of valid query parameters
      */
     public static ChangesetQuery buildFromUrlQuery(String query) throws ChangesetQueryUrlException{
@@ -67 +66 @@
      * @param uid the uid of the user. > 0 expected.
      * @return the query object with the applied restriction
-     * @throws IllegalArgumentException thrown if uid <= 0
+     * @throws IllegalArgumentException if uid <= 0
      * @see #forUser(String)
      */
-    public ChangesetQuery forUser(int uid) throws IllegalArgumentException{
+    public ChangesetQuery forUser(int uid) {
         if (uid <= 0)
             throw new IllegalArgumentException(MessageFormat.format("Parameter ''{0}'' > 0 expected. Got ''{1}''.", "uid", uid));
@@ -86 +85 @@
      * @param username the username. Must not be null.
      * @return the query object with the applied restriction
-     * @throws IllegalArgumentException thrown if username is null.
+     * @throws IllegalArgumentException if username is null.
      * @see #forUser(int)
      */
@@ -133 +132 @@
      *
      * @return the restricted changeset query
-     * @throws IllegalArgumentException thrown if either of the parameters isn't a valid longitude or
+     * @throws IllegalArgumentException if either of the parameters isn't a valid longitude or
      * latitude value
      */
-    public ChangesetQuery inBbox(double minLon, double minLat, double maxLon, double maxLat) throws IllegalArgumentException{
+    public ChangesetQuery inBbox(double minLon, double minLat, double maxLon, double maxLat) {
         if (!LatLon.isValidLon(minLon))
             throw new IllegalArgumentException(tr("Illegal longitude value for parameter ''{0}'', got {1}", "minLon", minLon));
@@ -156 +155 @@
      *
      * @return the restricted changeset query
-     * @throws IllegalArgumentException thrown if min is null
-     * @throws IllegalArgumentException thrown if max is null
+     * @throws IllegalArgumentException if min is null
+     * @throws IllegalArgumentException if max is null
      */
     public ChangesetQuery inBbox(LatLon min, LatLon max) {
@@ -171 +170 @@
      * @param bbox the bounding box. Must not be null.
      * @return the changeset query
-     * @throws IllegalArgumentException thrown if bbox is null.
-     */
-    public ChangesetQuery inBbox(Bounds bbox) throws IllegalArgumentException {
+     * @throws IllegalArgumentException if bbox is null.
+     */
+    public ChangesetQuery inBbox(Bounds bbox) {
         CheckParameterUtil.ensureParameterNotNull(bbox, "bbox");
         this.bounds = bbox;
@@ -185 +184 @@
      * @param d the date . Must not be null.
      * @return the restricted changeset query
-     * @throws IllegalArgumentException thrown if d is null
-     */
-    public ChangesetQuery closedAfter(Date d) throws IllegalArgumentException{
+     * @throws IllegalArgumentException if d is null
+     */
+    public ChangesetQuery closedAfter(Date d) {
         CheckParameterUtil.ensureParameterNotNull(d, "d");
         this.closedAfter = d;
@@ -201 +200 @@
      * @param createdBefore only reply changesets created before this date. Must not be null.
      * @return the restricted changeset query
-     * @throws IllegalArgumentException thrown if closedAfter is null
-     * @throws IllegalArgumentException thrown if createdBefore is null
-     */
-    public ChangesetQuery closedAfterAndCreatedBefore(Date closedAfter, Date createdBefore ) throws IllegalArgumentException {
+     * @throws IllegalArgumentException if closedAfter is null
+     * @throws IllegalArgumentException if createdBefore is null
+     */
+    public ChangesetQuery closedAfterAndCreatedBefore(Date closedAfter, Date createdBefore ) {
         CheckParameterUtil.ensureParameterNotNull(closedAfter, "closedAfter");
         CheckParameterUtil.ensureParameterNotNull(createdBefore, "createdBefore");
@@ -241 +240 @@
      * @param changesetIds the changeset ids
      * @return the query object with the applied restriction
-     * @throws IllegalArgumentException thrown if changesetIds is null.
+     * @throws IllegalArgumentException if changesetIds is null.
      */
     public ChangesetQuery forChangesetIds(Collection<Long> changesetIds) {