Changeset 10627 in josm


Ignore:
Timestamp:
2016-07-24T14:48:47+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:
67 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

    r10469 r10627  
    128128            } catch (IOException e) {
    129129                LOG.log(Level.WARNING, "JCS TMS - error loading from cache for tile {0}: {1}", new Object[] {tile.getKey(), e.getMessage()});
     130                Main.warn(e);
    130131            }
    131132        }
    … …  
    296297            } catch (IOException e) {
    297298                LOG.log(Level.WARNING, "JCS TMS - error loading object for tile {0}: {1}", new Object[] {tile.getKey(), e.getMessage()});
     299                Main.warn(e);
    298300                return null;
    299301            }

  • trunk/src/org/openstreetmap/josm/data/osm/DatasetConsistencyTest.java

    r10378 r10627  
    215215            writer.println("Exception during dataset integrity test:");
    216216            e.printStackTrace(writer);
     217            Main.warn(e);
    217218        }
    218219    }

  • trunk/src/org/openstreetmap/josm/gui/GettingStarted.java

    r10620 r10627  
    145145                    ProxyPreference.removeProxyPreferenceListener(GettingStarted.this);
    146146                } catch (IOException ex) {
    147                     Main.warn(tr("Failed to read MOTD. Exception was: {0}", ex.toString()));
     147                    Main.warn(ex, tr("Failed to read MOTD. Exception was: {0}", ex.toString()));
    148148                    content = "<html>" + STYLE + "<h1>" + "JOSM - " + tr("Java OpenStreetMap Editor")
    149149                            + "</h1>\n<h2 align=\"center\">(" + tr("Message of the day not available") + ")</h2></html>";

  • trunk/src/org/openstreetmap/josm/gui/MainApplication.java

    r10611 r10627  
    536536                    Main.setOffline(OnlineResource.valueOf(s.toUpperCase(Locale.ENGLISH)));
    537537                } catch (IllegalArgumentException e) {
    538                     Main.error(tr("''{0}'' is not a valid value for argument ''{1}''. Possible values are {2}, possibly delimited by commas.",
     538                    Main.error(e, tr("''{0}'' is not a valid value for argument ''{1}''. Possible values are {2}, possibly delimited by commas.",
    539539                            s.toUpperCase(Locale.ENGLISH), Option.OFFLINE.getName(), Arrays.toString(OnlineResource.values())));
    540540                    System.exit(1);

  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r10611 r10627  
    361361        } catch (IllegalArgumentException e) {
    362362            // Ignored in old implementation
    363             if (Main.isDebugEnabled()) {
    364                 Main.debug(e.getMessage());
    365             }
     363            Main.debug(e);
    366364        }
    367365        try {
    … …  
    369367        } catch (IllegalArgumentException e) {
    370368            // Ignored in old implementation
    371             if (Main.isDebugEnabled()) {
    372                 Main.debug(e.getMessage());
    373             }
     369            Main.debug(e);
    374370        }
    375371    }
    … …  
    389385        } catch (IllegalArgumentException e) {
    390386            // Ignored in old implementation
    391             if (Main.isDebugEnabled()) {
    392                 Main.debug(e.getMessage());
    393             }
     387            Main.debug(e);
    394388        }
    395389    }

  • trunk/src/org/openstreetmap/josm/gui/bbox/SlippyMapBBoxChooser.java

    r10611 r10627  
    7575                    }
    7676                } catch (IllegalArgumentException ex) {
     77                    Main.warn(ex);
    7778                    if (ex.getMessage() != null && !ex.getMessage().isEmpty()) {
    7879                        JOptionPane.showMessageDialog(Main.parent,

  • trunk/src/org/openstreetmap/josm/gui/dialogs/FilterTableModel.java

    r10446 r10627  
    6464                filterMatcher.add(filter);
    6565            } catch (ParseError e) {
     66                Main.error(e);
    6667                JOptionPane.showMessageDialog(
    6768                        Main.parent,

  • trunk/src/org/openstreetmap/josm/gui/dialogs/LatLonDialog.java

    r10611 r10627  
    236236            }
    237237        } catch (IllegalArgumentException e) {
     238            Main.trace(e);
    238239            latLon = null;
    239240        }
    … …  
    254255            en = parseEastNorth(tfEastNorth.getText());
    255256        } catch (IllegalArgumentException e) {
     257            Main.trace(e);
    256258            en = null;
    257259        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/MapPaintDialog.java

    r10611 r10627  
    656656                }
    657657            } catch (IOException ex) {
     658                Main.error(ex);
    658659                txtSource.append("<ERROR: failed to read file!>");
    659660            }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ToggleDialog.java

    r10611 r10627  
    110110            } catch (IllegalArgumentException e) {
    111111                // Legacy settings
     112                Main.trace(e);
    112113                return Boolean.parseBoolean(s) ? ButtonHidingType.DYNAMIC : ButtonHidingType.ALWAYS_SHOWN;
    113114            }
    … …  
    699700                new WindowGeometry(preferencePrefix+".geometry").applySafe(this);
    700701            } catch (WindowGeometryException e) {
     702                Main.debug(e);
    701703                ToggleDialog.this.setPreferredSize(ToggleDialog.this.getDefaultDetachedSize());
    702704                pack();

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetCacheManager.java

    r10611 r10627  
    393393                    }
    394394                } catch (IllegalStateException e) {
     395                    Main.error(e);
    395396                    JOptionPane.showMessageDialog(parent, e.getMessage(), tr("Error"), JOptionPane.ERROR_MESSAGE);
    396397                }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/BasicChangesetQueryPanel.java

    r10378 r10627  
    195195                q = BasicQuery.valueOf(BasicQuery.class, value);
    196196            } catch (IllegalArgumentException e) {
    197                 Main.warn(tr("Unexpected value for preference ''{0}'', got ''{1}''. Resetting to default query.",
     197                Main.warn(e, tr("Unexpected value for preference ''{0}'', got ''{1}''. Resetting to default query.",
    198198                        "changeset-query.basic.query", value));
    199199                q = BasicQuery.MOST_RECENT_CHANGESETS;

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/ChangesetQueryDialog.java

    r10479 r10627  
    2323import javax.swing.KeyStroke;
    2424
     25import org.openstreetmap.josm.Main;
    2526import org.openstreetmap.josm.gui.HelpAwareOptionPane;
    2627import org.openstreetmap.josm.gui.help.ContextSensitiveHelpAction;
    … …  
    192193                setVisible(false);
    193194            } catch (IllegalStateException e) {
     195                Main.error(e);
    194196                JOptionPane.showMessageDialog(ChangesetQueryDialog.this, e.getMessage(), tr("Error"), JOptionPane.ERROR_MESSAGE);
    195197            }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/UrlBasedQueryPanel.java

    r10611 r10627  
    134134            return ChangesetQuery.buildFromUrlQuery(url.getQuery());
    135135        } catch (ChangesetQueryUrlException e) {
    136             Main.warn(e.getMessage());
     136            Main.warn(e);
    137137            return null;
    138138        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r10611 r10627  
    935935            return modified ? new ChangeCommand(orig, relation) : null;
    936936        } catch (AddAbortException ign) {
     937            Main.trace(ign);
    937938            return null;
    938939        }

  • trunk/src/org/openstreetmap/josm/gui/download/BookmarkList.java

    r10212 r10627  
    148148                    bookmarks.add(new Bookmark(entry));
    149149                } catch (IllegalArgumentException e) {
    150                     Main.error(tr("Error reading bookmark entry: %s", e.getMessage()));
     150                    Main.error(e, tr("Error reading bookmark entry: %s", e.getMessage()));
    151151                }
    152152            }

  • trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

    r10611 r10627  
    138138            tpDownloadAreaSelectors.setSelectedIndex(Main.pref.getInteger("download.tab", 0));
    139139        } catch (IndexOutOfBoundsException ex) {
     140            Main.trace(ex);
    140141            Main.pref.putInteger("download.tab", 0);
    141142        }

  • trunk/src/org/openstreetmap/josm/gui/help/HelpBrowser.java

    r10611 r10627  
    301301            content = reader.fetchHelpTopicContent(url, true);
    302302        } catch (MissingHelpContentException e) {
     303            Main.trace(e);
    303304            url = getHelpTopicUrl(buildAbsoluteHelpTopic(relativeHelpTopic, LocaleType.BASELANGUAGE));
    304305            try {
    305306                content = reader.fetchHelpTopicContent(url, true);
    306307            } catch (MissingHelpContentException e1) {
     308                Main.trace(e1);
    307309                url = getHelpTopicUrl(buildAbsoluteHelpTopic(relativeHelpTopic, LocaleType.ENGLISH));
    308310                try {
    309311                    content = reader.fetchHelpTopicContent(url, true);
    310312                } catch (MissingHelpContentException e2) {
     313                    Main.debug(e2);
    311314                    this.url = url;
    312315                    handleMissingHelpContent(relativeHelpTopic);
    … …  
    344347            content = reader.fetchHelpTopicContent(url, true);
    345348        } catch (MissingHelpContentException e) {
     349            Main.debug(e);
    346350            this.url = url;
    347351            handleMissingHelpContent(absoluteHelpTopic);

  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r10413 r10627  
    144144            HistoryOsmPrimitive.forOsmPrimitive(primitive);
    145145        } catch (IllegalArgumentException ign) {
     146            Main.trace(ign);
    146147            return false;
    147148        }

  • trunk/src/org/openstreetmap/josm/gui/io/CredentialDialog.java

    r10179 r10627  
    110110            setAlwaysOnTop(true);
    111111        } catch (SecurityException e) {
    112             Main.warn(tr("Failed to put Credential Dialog always on top. Caught security exception."));
     112            Main.warn(e, tr("Failed to put Credential Dialog always on top. Caught security exception."));
    113113        }
    114114        build();

  • trunk/src/org/openstreetmap/josm/gui/io/DownloadOpenChangesetsTask.java

    r10611 r10627  
    110110                im.setPartiallyIdentified(im.getUserName());
    111111            }
    112             Main.warn(tr("Failed to retrieve user infos for the current JOSM user. Exception was: {0}", e.toString()));
     112            Main.warn(e, tr("Failed to retrieve user infos for the current JOSM user. Exception was: {0}", e.toString()));
    113113        }
    114114    }

  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersDialog.java

    r10611 r10627  
    493493                    currentFuture.get();
    494494                } catch (CancellationException e) {
     495                    Main.trace(e);
    495496                    model.setUploadState(layer, UploadOrSaveState.CANCELED);
    496497                } catch (InterruptedException | ExecutionException e) {
    … …  
    534535                    currentFuture.get();
    535536                } catch (CancellationException e) {
     537                    Main.trace(e);
    536538                    model.setSaveState(layerInfo.getLayer(), UploadOrSaveState.CANCELED);
    537539                } catch (InterruptedException | ExecutionException e) {

  • trunk/src/org/openstreetmap/josm/gui/io/UploadPrimitivesTask.java

    r10611 r10627  
    228228            SwingUtilities.invokeAndWait(r);
    229229        } catch (InterruptedException e) {
     230            Main.trace(e);
    230231            lastException = e;
    231232        } catch (InvocationTargetException e) {
     233            Main.trace(e);
    232234            lastException = new OsmTransferException(e.getCause());
    233235        }

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r10619 r10627  
    942942                sldSeconds.setValue((int) (deciSeconds % 60));
    943943            } catch (RuntimeException e) {
     944                Main.warn(e);
    944945                JOptionPane.showMessageDialog(Main.parent,
    945946                        tr("An error occurred while trying to match the photos to the GPX track."

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r10611 r10627  
    183183                addRecursiveFiles(files, selection);
    184184            } catch (IllegalStateException e) {
     185                Main.debug(e);
    185186                rememberError(e.getMessage());
    186187            }

  • trunk/src/org/openstreetmap/josm/gui/layer/markerlayer/Marker.java

    r10611 r10627  
    152152                return new TemplateParser(s).parse();
    153153            } catch (ParseError e) {
     154                Main.debug(e);
    154155                Main.warn("Unable to parse template engine pattern ''{0}'' for property {1}. Using default (''{2}'') instead",
    155156                        s, getKey(), super.getDefaultValueAsString());

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/ExpressionFactory.java

    r10611 r10627  
    261261                return new Color(r, g, b);
    262262            } catch (IllegalArgumentException e) {
     263                Main.trace(e);
    263264                return null;
    264265            }
    … …  
    279280                return new Color(r, g, b, alpha);
    280281            } catch (IllegalArgumentException e) {
     282                Main.trace(e);
    281283                return null;
    282284            }
    … …  
    294296                return Color.getHSBColor(h, s, b);
    295297            } catch (IllegalArgumentException e) {
     298                Main.trace(e);
    296299                return null;
    297300            }
    … …  
    628631                return RotationAngle.parseCardinalRotation(cardinal);
    629632            } catch (IllegalArgumentException ignore) {
     633                Main.trace(ignore);
    630634                return null;
    631635            }
    … …  
    668672                m = SearchCompiler.compile(searchStr);
    669673            } catch (ParseError ex) {
     674                Main.trace(ex);
    670675                return null;
    671676            }

  • trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/Selector.java

    r10300 r10627  
    331331                    };
    332332                } catch (NoSuchElementException ignore) {
     333                    Main.trace(ignore);
    333334                    containsFinder = new ContainsFinder(e);
    334335                }
    … …  
    468469                    if (!c.applies(env)) return false;
    469470                } catch (PatternSyntaxException e) {
    470                     Main.error("PatternSyntaxException while applying condition" + c +": "+e.getMessage());
     471                    Main.error(e, "PatternSyntaxException while applying condition" + c + ':');
    471472                    return false;
    472473                }

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

    r10378 r10627  
    343343                        userName));
    344344        } catch (OsmOAuthAuthorizationException e) {
     345            Main.debug(e);
    345346            throw new OsmLoginFailedException(e.getCause());
    346347        } catch (IOException e) {

  • trunk/src/org/openstreetmap/josm/gui/oauth/RetrieveAccessTokenTask.java

    r10611 r10627  
    8787            accessToken = client.getAccessToken(getProgressMonitor().createSubTaskMonitor(0, false));
    8888        } catch (OsmTransferCanceledException e) {
     89            Main.trace(e);
    8990            return;
    9091        } catch (final OsmOAuthAuthorizationException e) {

  • trunk/src/org/openstreetmap/josm/gui/oauth/RetrieveRequestTokenTask.java

    r10611 r10627  
    8181            requestToken = client.getRequestToken(getProgressMonitor().createSubTaskMonitor(0, false));
    8282        } catch (OsmTransferCanceledException e) {
     83            Main.trace(e);
    8384            return;
    8485        } catch (final OsmOAuthAuthorizationException e) {

  • trunk/src/org/openstreetmap/josm/gui/preferences/advanced/AdvancedPreference.java

    r10619 r10627  
    338338                        Main.pref.save();
    339339                    } catch (IOException e) {
    340                         Main.warn("IOException while saving preferences: "+e.getMessage());
     340                        Main.warn(e, "IOException while saving preferences:");
    341341                    }
    342342                    readPreferences(Main.pref);

  • trunk/src/org/openstreetmap/josm/gui/preferences/display/GPXSettingsPanel.java

    r10611 r10627  
    453453            parser.parse();
    454454        } catch (ParseError e) {
     455            Main.warn(e);
    455456            JOptionPane.showMessageDialog(Main.parent,
    456457                    tr("Incorrect waypoint label pattern: {0}", e.getMessage()), tr("Incorrect pattern"), JOptionPane.ERROR_MESSAGE);
    … …  
    462463            parser.parse();
    463464        } catch (ParseError e) {
     465            Main.warn(e);
    464466            JOptionPane.showMessageDialog(Main.parent,
    465467                    tr("Incorrect audio waypoint label pattern: {0}", e.getMessage()), tr("Incorrect pattern"), JOptionPane.ERROR_MESSAGE);

  • trunk/src/org/openstreetmap/josm/gui/preferences/display/LafPreference.java

    r10212 r10627  
    8686            } catch (ReflectiveOperationException ex) {
    8787                // just debug, Quaqua may not even be installed...
    88                 Main.debug(ex.getMessage());
     88                Main.debug(ex);
    8989            }
    9090        }

  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/ImageryPreference.java

    r10611 r10627  
    764764                    htmlPane = new JosmEditorPane(url);
    765765                } catch (IOException e1) {
     766                    Main.trace(e1);
    766767                    // give a second chance with a default Locale 'en'
    767768                    try {
    … …  
    769770                        htmlPane = new JosmEditorPane(url);
    770771                    } catch (IOException e2) {
     772                        Main.debug(e2);
    771773                        JOptionPane.showMessageDialog(gui, tr("EULA license URL not available: {0}", eulaUrl));
    772774                        return false;

  • trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java

    r10626 r10627  
    5757                            canLoad = true;
    5858                        } catch (IOException e) {
    59                             Main.warn(tr("Could not read tagging preset source: {0}", source));
     59                            Main.warn(e, tr("Could not read tagging preset source: {0}", source));
    6060                            ExtendedDialog ed = new ExtendedDialog(Main.parent, tr("Error"),
    6161                                    new String[] {tr("Yes"), tr("No"), tr("Cancel")});
    … …  
    8282                            // Should not happen, but at least show message
    8383                            String msg = tr("Could not read tagging preset source {0}", source);
    84                             Main.error(msg);
     84                            Main.error(e, msg);
    8585                            JOptionPane.showMessageDialog(Main.parent, msg);
    8686                            return false;
    … …  
    9595                                        source, e.getLineNumber(), e.getColumnNumber(), e.getMessage());
    9696                            }
     97                            Main.warn(e, errorMessage);
    9798                        } catch (SAXException e) {
    9899                            if (canLoad) {
    … …  
    105106                                        source, e.getMessage());
    106107                            }
     108                            Main.warn(e, errorMessage);
    107109                        }
    108110

  • trunk/src/org/openstreetmap/josm/gui/preferences/projection/CustomProjectionChoice.java

    r10611 r10627  
    9797                        test.update(input.getText());
    9898                    } catch (ProjectionConfigurationException ex) {
     99                        Main.warn(ex);
    99100                        error = ex.getMessage();
    100101                        valStatus.setIcon(ImageProvider.get("data", "error"));

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/ApiUrlTestTask.java

    r10173 r10627  
    191191                Capabilities.CapabilitiesParser.parse(new InputSource(connection.getResponse().getContent()));
    192192            } catch (SAXException | ParserConfigurationException e) {
    193                 Main.warn(e.getMessage());
     193                Main.warn(e);
    194194                alertInvalidCapabilities();
    195195                return;

  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionManager.java

    r10611 r10627  
    222222                    PRESET_TAG_CACHE.putAll(ki.key, ki.getValues());
    223223                } catch (NullPointerException e) {
    224                     Main.error(p + ": Unable to cache " + ki);
     224                    Main.error(e, p + ": Unable to cache " + ki);
    225225                }
    226226            }

  • trunk/src/org/openstreetmap/josm/gui/widgets/JosmPasswordField.java

    r8419 r10627  
    126126                        pasteAction.actionPerformed(e);
    127127                    } catch (NullPointerException npe) {
    128                         Main.error("NullPointerException occured because of JDK bug 6322854. "
     128                        Main.error(npe, "NullPointerException occured because of JDK bug 6322854. "
    129129                                +"Copy/Paste operation has not been performed. Please complain to Oracle: "+
    130130                                "https://bugs.openjdk.java.net/browse/JDK-6322854");

  • trunk/src/org/openstreetmap/josm/gui/widgets/OsmIdTextField.java

    r8496 r10627  
    1010import javax.swing.text.JTextComponent;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    1314import org.openstreetmap.josm.data.osm.PrimitiveId;
    … …  
    110111                    } catch (IllegalArgumentException ex) {
    111112                        try {
     113                            Main.trace(ex);
    112114                            long id = Long.parseLong(s);
    113115                            if (id <= 0) {
    … …  
    123125                            }
    124126                        } catch (IllegalArgumentException ex2) {
     127                            Main.trace(ex2);
    125128                            return false;
    126129                        }

  • trunk/src/org/openstreetmap/josm/io/CacheCustomContent.java

    r8926 r10627  
    9797            return false;
    9898        } catch (OfflineAccessException e) {
     99            Main.trace(e);
    99100            return true;
    100101        }
    … …  
    185186            }
    186187        } catch (IOException e) {
     188            Main.trace(e);
    187189            if (!isOffline()) {
    188190                this.data = updateForce();

  • trunk/src/org/openstreetmap/josm/io/CachedFile.java

    r10212 r10627  
    354354        } catch (IOException e) {
    355355            if (file.getName().endsWith(".zip")) {
    356                 Main.warn(tr("Failed to open file with extension ''{2}'' and namepart ''{3}'' in zip file ''{0}''. Exception was: {1}",
     356                Main.warn(e, tr("Failed to open file with extension ''{2}'' and namepart ''{3}'' in zip file ''{0}''. Exception was: {1}",
    357357                        file.getName(), e.toString(), extension, namepart));
    358358            }
    … …  
    425425            checkOfflineAccess(urlStr);
    426426        } catch (OfflineAccessException e) {
     427            Main.trace(e);
    427428            offline = true;
    428429        }

  • trunk/src/org/openstreetmap/josm/io/MessageNotifier.java

    r10615 r10627  
    147147                }
    148148            } catch (CredentialsAgentException e) {
    149                 Main.warn("Unable to get credentials: "+e.getMessage());
     149                Main.warn(e, "Unable to get credentials:");
    150150            }
    151151        }

  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r10475 r10627  
    440440        } catch (RuntimeException x) {
    441441            // out of bounds and such
     442            Main.debug(x);
    442443            ps.malformed++;
    443444            ps.pWp = null;

  • trunk/src/org/openstreetmap/josm/io/OsmApi.java

    r10436 r10627  
    212212                initializeCapabilities(cache.updateIfRequiredString());
    213213            } catch (SAXParseException parseException) {
     214                Main.trace(parseException);
    214215                // XML parsing may fail if JOSM previously stored a corrupted capabilities document (see #8278)
    215216                // In that case, force update and try again

  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r10615 r10627  
    528528            } catch (IllegalStateException e) {
    529529                // thrown for positive changeset id on new primitives
     530                Main.debug(e);
    530531                Main.info(e.getMessage());
    531532                current.setChangesetId(0);

  • trunk/src/org/openstreetmap/josm/io/auth/CredentialsManager.java

    r10600 r10627  
    77import java.util.Objects;
    88
     9import org.openstreetmap.josm.Main;
    910import org.openstreetmap.josm.data.oauth.OAuthToken;
    1011import org.openstreetmap.josm.gui.JosmUserIdentityManager;
    … …  
    103104            }
    104105        } catch (CredentialsAgentException ex) {
     106            Main.debug(ex);
    105107            return null;
    106108        }

  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r10217 r10627  
    203203                                        atts.getValue("max-lon"), ",");
    204204                    } catch (IllegalArgumentException e) {
     205                        Main.trace(e);
    205206                        break;
    206207                    }
    … …  
    235236                        shape.addPoint(atts.getValue("lat"), atts.getValue("lon"));
    236237                    } catch (IllegalArgumentException e) {
     238                        Main.trace(e);
    237239                        break;
    238240                    }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpServer.java

    r10212 r10627  
    3737            instance4.start();
    3838        } catch (IOException ex) {
     39            Main.debug(ex);
    3940            Main.warn(marktr("Cannot start IPv4 remotecontrol server on port {0}: {1}"),
    4041                    Integer.toString(port), ex.getLocalizedMessage());
    … …  
    4647            /* only show error when we also have no IPv4 */
    4748            if (instance4 == null) {
     49                Main.debug(ex);
    4850                Main.warn(marktr("Cannot start IPv6 remotecontrol server on port {0}: {1}"),
    4951                    Integer.toString(port), ex.getLocalizedMessage());

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpsServer.java

    r10580 r10627  
    318318                instance4.start();
    319319            } catch (IOException | GeneralSecurityException ex) {
     320                Main.debug(ex);
    320321                Main.warn(marktr("Cannot start IPv4 remotecontrol https server on port {0}: {1}"),
    321322                        Integer.toString(port), ex.getLocalizedMessage());
    … …  
    327328                /* only show error when we also have no IPv4 */
    328329                if (instance4 == null) {
     330                    Main.debug(ex);
    329331                    Main.warn(marktr("Cannot start IPv6 remotecontrol https server on port {0}: {1}"),
    330332                        Integer.toString(port), ex.getLocalizedMessage());

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r10587 r10627  
    241241                    out.flush();
    242242                } catch (RequestHandlerErrorException ex) {
     243                    Main.debug(ex);
    243244                    sendError(out);
    244245                } catch (RequestHandlerBadRequestException ex) {
     246                    Main.debug(ex);
    245247                    sendBadRequest(out, ex.getMessage());
    246248                } catch (RequestHandlerForbiddenException ex) {
     249                    Main.debug(ex);
    247250                    sendForbidden(out, ex.getMessage());
    248251                }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java

    r10615 r10627  
    101101                ps.add(SimplePrimitiveId.fromString(i));
    102102            } catch (IllegalArgumentException e) {
    103                 Main.warn("RemoteControl: invalid selection '"+i+"' ignored");
     103                Main.warn(e, "RemoteControl: invalid selection '"+i+"' ignored.");
    104104            }
    105105        }

  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r10626 r10627  
    257257            sources.add(org.openstreetmap.josm.gui.MainApplication.class.getClassLoader());
    258258        } catch (SecurityException ex) {
     259            Main.debug(ex);
    259260            sources.add(ImageProvider.class.getClassLoader());
    260261        }

  • trunk/src/org/openstreetmap/josm/plugins/PluginInformation.java

    r10358 r10627  
    216216                    s = tr(s);
    217217                } catch (IllegalArgumentException e) {
     218                    Main.debug(e);
    218219                    Main.info(tr("Invalid plugin description ''{0}'' in plugin {1}", s, name));
    219220                }

  • trunk/src/org/openstreetmap/josm/plugins/PluginProxy.java

    r10212 r10627  
    4545            plugin.getClass().getMethod("mapFrameInitialized", MapFrame.class, MapFrame.class).invoke(plugin, oldFrame, newFrame);
    4646        } catch (NoSuchMethodException e) {
     47            Main.trace(e);
    4748            Main.debug("Plugin "+plugin+" does not define mapFrameInitialized");
    4849        } catch (InvocationTargetException | IllegalAccessException | IllegalArgumentException e) {
    … …  
    5657            return (PreferenceSetting) plugin.getClass().getMethod("getPreferenceSetting").invoke(plugin);
    5758        } catch (NoSuchMethodException e) {
     59            Main.trace(e);
    5860            Main.debug("Plugin "+plugin+" does not define getPreferenceSetting");
    5961            return null;
    … …  
    6971            plugin.getClass().getMethod("addDownloadSelection", List.class).invoke(plugin, list);
    7072        } catch (NoSuchMethodException e) {
     73            Main.trace(e);
    7174            Main.debug("Plugin "+plugin+" does not define addDownloadSelection");
    7275        } catch (InvocationTargetException | IllegalAccessException | IllegalArgumentException e) {

  • trunk/src/org/openstreetmap/josm/tools/AudioUtil.java

    r9144 r10627  
    4141            return naturalLength / calibration;
    4242        } catch (UnsupportedAudioFileException | IOException e) {
    43             if (Main.isDebugEnabled()) {
    44                 Main.debug(e.getMessage());
    45             }
     43            Main.debug(e);
    4644            return 0.0;
    4745        }

  • trunk/src/org/openstreetmap/josm/tools/Geometry.java

    r10378 r10627  
    911911            innerRings = MultipolygonBuilder.joinWays(mpm.inners);
    912912        } catch (MultipolygonBuilder.JoinedPolygonCreationException ex) {
     913            Main.trace(ex);
    913914            Main.debug("Invalid multipolygon " + multiPolygon);
    914915            return false;

  • trunk/src/org/openstreetmap/josm/tools/I18n.java

    r10626 r10627  
    642642            }
    643643        } catch (IOException e) {
     644            Main.trace(e);
    644645            return false;
    645646        }

  • trunk/src/org/openstreetmap/josm/tools/ImageProvider.java

    r10616 r10627  
    959959                    img = read(Utils.fileToURL(cf.getFile()), false, false);
    960960                } catch (IOException e) {
    961                     Main.warn("IOException while reading HTTP image: "+e.getMessage());
     961                    Main.warn(e, "IOException while reading HTTP image:");
    962962                }
    963963                return img == null ? null : new ImageResource(img);
    … …  
    966966            }
    967967        } catch (IOException e) {
     968            Main.debug(e);
    968969            return null;
    969970        } finally {
    … …  
    990991                    bytes = Utils.decodeUrl(data).getBytes(StandardCharsets.UTF_8);
    991992                } catch (IllegalArgumentException ex) {
    992                     Main.warn("Unable to decode URL data part: "+ex.getMessage() + " (" + data + ')');
     993                    Main.warn(ex, "Unable to decode URL data part: "+ex.getMessage() + " (" + data + ')');
    993994                    return null;
    994995                }
    … …  
    10171018                    return img == null ? null : new ImageResource(img);
    10181019                } catch (IOException e) {
    1019                     Main.warn("IOException while reading image: "+e.getMessage());
     1020                    Main.warn(e, "IOException while reading image:");
    10201021                }
    10211022            }
    … …  
    11111112            }
    11121113        } catch (IOException e) {
    1113             Main.warn(tr("Failed to handle zip file ''{0}''. Exception was: {1}", archive.getName(), e.toString()));
     1114            Main.warn(e, tr("Failed to handle zip file ''{0}''. Exception was: {1}", archive.getName(), e.toString()));
    11141115        }
    11151116        return null;
    … …  
    11821183                        return u;
    11831184                } catch (SecurityException e) {
    1184                     Main.warn(tr(
     1185                    Main.warn(e, tr(
    11851186                            "Failed to access directory ''{0}'' for security reasons. Exception was: {1}",
    11861187                            name, e.toString()));
    … …  
    11971198                    return u;
    11981199            } catch (SecurityException e) {
    1199                 Main.warn(tr(
     1200                Main.warn(e, tr(
    12001201                        "Failed to access directory ''{0}'' for security reasons. Exception was: {1}", dir, e
    12011202                        .toString()));
    … …  
    12711272            cf.close();
    12721273        } catch (SAXReturnException r) {
     1274            Main.trace(r);
    12731275            return r.getResult();
    12741276        } catch (IOException | SAXException e) {
    … …  
    15451547            }
    15461548        } catch (SVGException ex) {
    1547             Main.error("Unable to load svg: {0}", ex.getMessage());
     1549            Main.error(ex, "Unable to load svg:");
    15481550            return null;
    15491551        }

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

    r10212 r10627  
    5959                        // Workaround for KDE (Desktop API is severely flawed)
    6060                        // see https://bugs.openjdk.java.net/browse/JDK-6486393
    61                         Main.warn("Desktop class failed. Platform dependent fall back for open url in browser.");
     61                        Main.warn(e, "Desktop class failed. Platform dependent fall back for open url in browser.");
    6262                        displayUrlFallback(uri);
    6363                    }
    … …  
    7272                displayUrlFallback(uri);
    7373            } catch (IOException e) {
     74                Main.debug(e);
    7475                return e.getMessage();
    7576            }
    … …  
    9091            return displayUrl(new URI(url));
    9192        } catch (URISyntaxException e) {
     93            Main.debug(e);
    9294            return e.getMessage();
    9395        }

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookOsx.java

    r10580 r10627  
    6767                enableOSXFullscreen((Window) Main.parent);
    6868            } catch (IllegalAccessException e) {
     69                Main.debug(e);
    6970                // with Java 9, module java.desktop does not export com.apple.eawt, use new Desktop API instead
    7071                setHandlers(Desktop.class, quitHandler, aboutHandler, openFilesHandler, preferencesHandler, proxy, Desktop.getDesktop());
    … …  
    8990            return Class.forName("com.apple.eawt."+className);
    9091        } catch (ClassNotFoundException e) {
     92            Main.trace(e);
    9193            return Class.forName("java.awt.desktop."+className);
    9294        }
    … …  
    148150                    args[1].getClass().getDeclaredMethod(closed ? "performQuit" : "cancelQuit").invoke(args[1]);
    149151                } catch (IllegalAccessException e) {
     152                    Main.debug(e);
    150153                    // with Java 9, module java.desktop does not export com.apple.eawt, use new Desktop API instead
    151154                    Class.forName("java.awt.desktop.QuitResponse").getMethod(closed ? "performQuit" : "cancelQuit").invoke(args[1]);

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r10626 r10627  
    192192            return "Debian".equalsIgnoreCase(dist) || "Ubuntu".equalsIgnoreCase(dist) || "Mint".equalsIgnoreCase(dist);
    193193        } catch (IOException e) {
    194             if (Main.isDebugEnabled()) {
    195                 // lsb_release is not available on all Linux systems, so don't log at warning level
    196                 Main.debug(e.getMessage());
    197             }
     194            // lsb_release is not available on all Linux systems, so don't log at warning level
     195            Main.debug(e);
    198196            return false;
    199197        }
    … …  
    303301                }
    304302            } catch (IOException e) {
     303                Main.debug(e);
    305304                // Non LSB-compliant Linux system. List of common fallback release files: http://linuxmafia.com/faq/Admin/release-files.html
    306305                for (LinuxReleaseInfo info : new LinuxReleaseInfo[]{

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java

    r10626 r10627  
    268268                    ks.deleteEntry(alias);
    269269                } catch (KeyStoreException e) {
    270                     Main.error(tr("Unable to remove insecure certificate from keystore: {0}", e.getMessage()));
     270                    Main.error(e, tr("Unable to remove insecure certificate from keystore: {0}", e.getMessage()));
    271271                }
    272272            }

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

    r10616 r10627  
    13261326            return true;
    13271327        } catch (MalformedURLException | NullPointerException e) {
     1328            Main.trace(e);
    13281329            return false;
    13291330        }

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

    r10384 r10627  
    8989            initFromPreferences(preferenceKey);
    9090        } catch (WindowGeometryException e) {
    91             if (Main.isDebugEnabled()) {
    92                 Main.debug(e.getMessage());
    93             }
     91            Main.debug(e);
    9492            initFromWindowGeometry(defaultGeometry);
    9593        }

  • trunk/src/org/openstreetmap/josm/tools/XmlObjectParser.java

    r10378 r10627  
    204204                    return f;
    205205                } catch (NoSuchFieldException ex) {
     206                    Main.trace(ex);
    206207                    fields.put(s, null);
    207208                    return null;
    … …  
    251252            } catch (SAXException e) {
    252253                // Exception very unlikely to happen, so no need to translate this
    253                 Main.error("Cannot disable 'load-external-dtd' feature: "+e.getMessage());
     254                Main.error(e, "Cannot disable 'load-external-dtd' feature:");
    254255            }
    255256            reader.parse(new InputSource(in));
Note: See TracChangeset for help on using the changeset viewer.