Changeset 10463 in josm for trunk/src/org


Ignore:
Timestamp:
2016-06-23T02:35:10+02:00 (8 years ago)
Author:
Don-vip
Message:

sonar - squid:S1166 - Exception handlers should preserve the original exceptions

Location:
trunk/src/org/openstreetmap/josm
Files:
17 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/Main.java

    r10442 r10463  
    10131013        } catch (final NoClassDefFoundError | ClassNotFoundException e) {
    10141014            // Try to find look and feel in plugin classloaders
     1015            Main.trace(e);
    10151016            Class<?> klass = null;
    10161017            for (ClassLoader cl : PluginHandler.getResourceClassLoaders()) {
    … …  
    10191020                    break;
    10201021                } catch (ClassNotFoundException ex) {
    1021                     if (Main.isTraceEnabled()) {
    1022                         Main.trace(ex.getMessage());
    1023                     }
     1022                    Main.trace(ex);
    10241023                }
    10251024            }
    … …  
    10281027                    UIManager.setLookAndFeel((LookAndFeel) klass.getConstructor().newInstance());
    10291028                } catch (ReflectiveOperationException ex) {
    1030                     warn("Cannot set Look and Feel: " + laf + ": "+ex.getMessage());
     1029                    warn(ex, "Cannot set Look and Feel: " + laf + ": "+ex.getMessage());
    10311030                } catch (UnsupportedLookAndFeelException ex) {
    10321031                    info("Look and Feel not supported: " + laf);
    10331032                    Main.pref.put("laf", defaultlaf);
     1033                    trace(ex);
    10341034                }
    10351035            } else {
    … …  
    10401040            info("Look and Feel not supported: " + laf);
    10411041            Main.pref.put("laf", defaultlaf);
     1042            trace(e);
    10421043        } catch (InstantiationException | IllegalAccessException e) {
    10431044            error(e);
    … …  
    10651066            CoordinateFormat.setCoordinateFormat(CoordinateFormat.valueOf(Main.pref.get("coordinates")));
    10661067        } catch (IllegalArgumentException iae) {
     1068            Main.trace(iae);
    10671069            CoordinateFormat.setCoordinateFormat(CoordinateFormat.DECIMAL_DEGREES);
    10681070        }
    1069 
    10701071    }
    10711072
    … …  
    12181219                    f = new File(new URI(s));
    12191220                } catch (URISyntaxException e) {
     1221                    Main.warn(e);
    12201222                    JOptionPane.showMessageDialog(
    12211223                            Main.parent,

  • trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java

    r10373 r10463  
    2929import org.openstreetmap.josm.io.imagery.WMSImagery;
    3030import org.openstreetmap.josm.io.imagery.WMSImagery.LayerDetails;
     31import org.openstreetmap.josm.io.imagery.WMSImagery.WMSGetCapabilitiesException;
    3132import org.openstreetmap.josm.tools.GBC;
    3233import org.openstreetmap.josm.tools.ImageProvider;
    … …  
    151152            }
    152153            Main.error(ex, false);
    153         } catch (WMSImagery.WMSGetCapabilitiesException ex) {
     154        } catch (WMSGetCapabilitiesException ex) {
    154155            if (!GraphicsEnvironment.isHeadless()) {
    155156                JOptionPane.showMessageDialog(Main.parent, tr("Could not parse WMS layer list."),
    156157                        tr("WMS Error"), JOptionPane.ERROR_MESSAGE);
    157158            }
    158             Main.error("Could not parse WMS layer list. Incoming data:\n"+ex.getIncomingData());
     159            Main.error(ex, "Could not parse WMS layer list. Incoming data:\n"+ex.getIncomingData());
    159160        }
    160161        return null;

  • trunk/src/org/openstreetmap/josm/actions/AlignInLineAction.java

    r10383 r10463  
    208208
    209209        } catch (InvalidSelection except) {
     210            Main.debug(except);
    210211            new Notification(except.getMessage())
    211212                .setIcon(JOptionPane.INFORMATION_MESSAGE)

  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r10382 r10463  
    224224            combineResult = combineWaysWorker(selectedWays);
    225225        } catch (UserCancelException ex) {
     226            Main.trace(ex);
    226227            return;
    227228        }

  • trunk/src/org/openstreetmap/josm/actions/ExtensionFileFilter.java

    r10407 r10463  
    6767                importers.add(importer);
    6868            } catch (ReflectiveOperationException e) {
    69                 if (Main.isDebugEnabled()) {
    70                     Main.debug(e.getMessage());
    71                 }
     69                Main.debug(e);
    7270            } catch (ServiceConfigurationError e) {
    7371                // error seen while initializing WMSLayerImporter in plugin unit tests:
    … …  
    108106                Main.getLayerManager().addAndFireActiveLayerChangeListener(exporter);
    109107            } catch (ReflectiveOperationException e) {
    110                 if (Main.isDebugEnabled()) {
    111                     Main.debug(e.getMessage());
    112                 }
     108                Main.debug(e);
    113109            } catch (ServiceConfigurationError e) {
    114110                // see above in importers initialization

  • trunk/src/org/openstreetmap/josm/actions/JoinAreasAction.java

    r10457 r10463  
    508508            }
    509509        } catch (UserCancelException exception) {
     510            Main.trace(exception);
    510511            //revert changes
    511512            //FIXME: this is dirty hack
    … …  
    681682            return true;
    682683        } catch (UserCancelException ex) {
     684            Main.trace(ex);
    683685            return false;
    684686        }

  • trunk/src/org/openstreetmap/josm/actions/MapRectifierWMSmenuAction.java

    r10436 r10463  
    191191                        break outer;
    192192                    } catch (IllegalStateException ex) {
    193                         Main.error(ex.getMessage());
     193                        Main.error(ex, false);
    194194                    }
    195195                }

  • trunk/src/org/openstreetmap/josm/actions/MergeNodesAction.java

    r10448 r10463  
    346346                    trn("Merge {0} node", "Merge {0} nodes", nodes.size(), nodes.size()), cmds);
    347347        } catch (UserCancelException ex) {
     348            Main.trace(ex);
    348349            return null;
    349350        }

  • trunk/src/org/openstreetmap/josm/actions/OrthogonalizeAction.java

    r10383 r10463  
    118118                }
    119119            } catch (InvalidUserInputException ex) {
     120                Main.debug(ex);
    120121                new Notification(
    121122                        tr("Orthogonalize Shape / Undo<br>"+
    … …  
    154155            Main.map.repaint();
    155156        } catch (InvalidUserInputException ex) {
     157            Main.debug(ex);
    156158            String msg;
    157159            if ("usage".equals(ex.getMessage())) {

  • trunk/src/org/openstreetmap/josm/actions/ReverseWayAction.java

    r10382 r10463  
    9696                revResult = reverseWay(w);
    9797            } catch (UserCancelException ex) {
     98                Main.trace(ex);
    9899                return;
    99100            }

  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r10446 r10463  
    136136            }
    137137        } catch (SecurityException e) {
    138             if (Main.isTraceEnabled()) {
    139                 Main.trace(e.getMessage());
    140             }
     138            Main.trace(e);
    141139        }
    142140        List<String> commandLineArgs = Main.getCommandLineArgs();

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java

    r10436 r10463  
    2121import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    2222import org.openstreetmap.josm.io.BoundingBoxDownloader;
     23import org.openstreetmap.josm.io.BoundingBoxDownloader.MoreNotesException;
    2324import org.openstreetmap.josm.io.OsmApi;
    2425import org.openstreetmap.josm.io.OsmServerLocationReader;
    … …  
    147148            try {
    148149                notesData = reader.parseNotes(DOWNLOAD_LIMIT.get(), DAYS_CLOSED.get(), subMonitor);
    149             } catch (BoundingBoxDownloader.MoreNotesException e) {
     150            } catch (MoreNotesException e) {
     151                Main.debug(e);
    150152                notesData = e.notes;
    151153                JOptionPane.showMessageDialog(Main.parent, "<html>"

  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmChangeTask.java

    r10308 r10463  
    181181                            data.setVisible(hp.isVisible());
    182182                        } catch (IllegalStateException e) {
    183                             Main.error("Cannot change visibility for "+p+": "+e.getMessage());
     183                            Main.error(e, "Cannot change visibility for "+p+':');
    184184                        }
    185185                        data.setTimestamp(hp.getTimestamp());
    … …  
    193193                            it.remove();
    194194                        } catch (AssertionError e) {
    195                             Main.error("Cannot load "+p + ": " + e.getMessage());
     195                            Main.error(e, "Cannot load "+p+':');
    196196                        }
    197197                    }

  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r10413 r10463  
    11951195                    + (unitvector.getY() * linelength)));
    11961196        } catch (NoninvertibleTransformException e) {
     1197            Main.debug(e);
    11971198            return new Line2D.Double(start, new Point2D.Double(start.getX() + (unitvector.getX() * 10), start.getY()
    11981199                    + (unitvector.getY() * 10)));

  • trunk/src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java

    r10382 r10463  
    546546            return true;
    547547        } catch (IllegalArgumentException e) {
     548            Main.debug(e);
    548549            new Notification(tr("ParallelWayAction\n" +
    549550                    "The ways selected must form a simple branchless path"))

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r10459 r10463  
    351351                        super.buttonAction(buttonIndex, evt);
    352352                    } catch (ParseError e) {
     353                        Main.debug(e);
    353354                        JOptionPane.showMessageDialog(
    354355                                Main.parent,
    … …  
    704705                }
    705706                subMonitor.finishTask();
    706             } catch (SearchCompiler.ParseError e) {
     707            } catch (ParseError e) {
     708                Main.debug(e);
    707709                JOptionPane.showMessageDialog(
    708710                        Main.parent,

  • trunk/src/org/openstreetmap/josm/actions/upload/ApiPreconditionCheckerHook.java

    r10446 r10463  
    4242            }
    4343        } catch (OsmTransferCanceledException e) {
     44            Main.trace(e);
    4445            return false;
    4546        } catch (OsmApiInitializationException e) {
Note: See TracChangeset for help on using the changeset viewer.