Ignore:
Timestamp:
2017-03-19T02:26:34+01:00 (2 years ago)
Author:
Don-vip
Message:

PMD - Strict Exceptions

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r11731 r11746  
    11271127                findBoundaryPolygonsStartingWith(discardedResult, traverser, result, startWay);
    11281128            }
    1129         } catch (Throwable t) {
     1129        } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException t) {
    11301130            throw BugReport.intercept(t).put("traverser", traverser);
    11311131        }
     
    11831183                }
    11841184            }
    1185         } catch (Throwable t) {
     1185        } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException t) {
    11861186            throw BugReport.intercept(t).put("path", path);
    11871187        }
  • trunk/src/org/openstreetmap/josm/actions/SessionLoadAction.java

    r11645 r11746  
    3131import org.openstreetmap.josm.io.session.SessionReader;
    3232import org.openstreetmap.josm.tools.CheckParameterUtil;
     33import org.openstreetmap.josm.tools.JosmRuntimeException;
    3334import org.openstreetmap.josm.tools.Utils;
    3435
     
    178179            } catch (IOException e) {
    179180                handleException(tr("IO Error"), e);
    180             } catch (RuntimeException e) {
     181            } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException e) {
    181182                cancel();
    182183                throw e;
  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r11650 r11746  
    269269        String reportHeader = getReportHeader();
    270270        text.append(reportHeader);
    271         try {
    272             Map<String, Setting<?>> settings = Main.pref.getAllSettings();
    273             Set<String> keys = new HashSet<>(settings.keySet());
    274             for (String key : keys) {
    275                 // Remove sensitive information from status report
    276                 if (key.startsWith("marker.show") || key.contains("username") || key.contains("password") || key.contains("access-token")) {
    277                     settings.remove(key);
    278                 }
    279             }
    280             for (Entry<String, Setting<?>> entry : settings.entrySet()) {
    281                 text.append(paramCleanup(entry.getKey()))
    282                     .append('=')
    283                     .append(paramCleanup(entry.getValue().getValue().toString())).append('\n');
    284             }
    285         } catch (Exception ex) {
    286             Main.error(ex);
     271        Map<String, Setting<?>> settings = Main.pref.getAllSettings();
     272        Set<String> keys = new HashSet<>(settings.keySet());
     273        for (String key : keys) {
     274            // Remove sensitive information from status report
     275            if (key.startsWith("marker.show") || key.contains("username") || key.contains("password") || key.contains("access-token")) {
     276                settings.remove(key);
     277            }
     278        }
     279        for (Entry<String, Setting<?>> entry : settings.entrySet()) {
     280            text.append(paramCleanup(entry.getKey()))
     281                .append('=')
     282                .append(paramCleanup(entry.getValue().getValue().toString())).append('\n');
    287283        }
    288284
  • trunk/src/org/openstreetmap/josm/actions/audio/AudioBackAction.java

    r6380 r11746  
    88import java.awt.event.ActionEvent;
    99import java.awt.event.KeyEvent;
     10import java.io.IOException;
    1011
    1112import org.openstreetmap.josm.Main;
     
    3839            else
    3940                MarkerLayer.playAudio();
    40         } catch (Exception ex) {
     41        } catch (IOException | InterruptedException ex) {
    4142            AudioPlayer.audioMalfunction(ex);
    4243        }
  • trunk/src/org/openstreetmap/josm/actions/audio/AudioFastSlowAction.java

    r8444 r11746  
    33
    44import java.awt.event.ActionEvent;
     5import java.io.IOException;
    56
    67import org.openstreetmap.josm.Main;
     
    4142            if (AudioPlayer.playing() || AudioPlayer.paused())
    4243                AudioPlayer.play(AudioPlayer.url(), AudioPlayer.position(), speed * multiplier);
    43         } catch (Exception ex) {
     44        } catch (IOException | InterruptedException ex) {
    4445            AudioPlayer.audioMalfunction(ex);
    4546        }
  • trunk/src/org/openstreetmap/josm/actions/audio/AudioFwdAction.java

    r6380 r11746  
    77import java.awt.event.ActionEvent;
    88import java.awt.event.KeyEvent;
     9import java.io.IOException;
    910
    1011import org.openstreetmap.josm.Main;
     
    3637            else
    3738                MarkerLayer.playAudio();
    38         } catch (Exception ex) {
     39        } catch (IOException | InterruptedException ex) {
    3940            AudioPlayer.audioMalfunction(ex);
    4041        }
  • trunk/src/org/openstreetmap/josm/actions/audio/AudioPlayPauseAction.java

    r11452 r11746  
    77import java.awt.event.ActionEvent;
    88import java.awt.event.KeyEvent;
     9import java.io.IOException;
    910import java.net.URL;
    1011
     
    5354                }
    5455            }
    55         } catch (Exception ex) {
     56        } catch (IOException | InterruptedException ex) {
    5657            AudioPlayer.audioMalfunction(ex);
    5758        }
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmChangeTask.java

    r10463 r11746  
    1111import java.util.Map.Entry;
    1212import java.util.concurrent.Future;
     13import java.util.concurrent.RejectedExecutionException;
    1314import java.util.regex.Matcher;
    1415import java.util.regex.Pattern;
     
    115116                // Let's load all required history
    116117                Main.worker.submit(new HistoryLoaderAndListener(toLoad));
    117             } catch (Exception e) {
     118            } catch (RejectedExecutionException e) {
    118119                rememberException(e);
    119120                setFailed(true);
Note: See TracChangeset for help on using the changeset viewer.