Index: /trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java =================================================================== --- /trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java (revision 10626) @@ -235,7 +235,5 @@ } catch (NumberFormatException nfe) { // we repaint offset numbers in any case - if (Main.isTraceEnabled()) { - Main.trace(nfe.getMessage()); - } + Main.trace(nfe); } } Index: /trunk/src/org/openstreetmap/josm/actions/SearchNotesDownloadAction.java =================================================================== --- /trunk/src/org/openstreetmap/josm/actions/SearchNotesDownloadAction.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/actions/SearchNotesDownloadAction.java (revision 10626) @@ -88,7 +88,5 @@ return; } catch (NumberFormatException ignore) { - if (Main.isTraceEnabled()) { - Main.trace(ignore.getMessage()); - } + Main.trace(ignore); } Index: /trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java =================================================================== --- /trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java (revision 10626) @@ -734,7 +734,5 @@ } } catch (NumberFormatException ignore) { - if (Main.isTraceEnabled()) { - Main.trace(ignore.getMessage()); - } + Main.trace(ignore); } this.referenceNumber = v; Index: /trunk/src/org/openstreetmap/josm/data/Preferences.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/Preferences.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/data/Preferences.java (revision 10626) @@ -793,7 +793,5 @@ } catch (NumberFormatException e) { // fall out - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return def; @@ -811,7 +809,5 @@ } catch (NumberFormatException e) { // fall out - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return def; @@ -827,7 +823,5 @@ } catch (NumberFormatException e) { // fall out - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return def; @@ -843,7 +837,5 @@ } catch (NumberFormatException e) { // fall out - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return def; Index: /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java (revision 10626) @@ -441,7 +441,5 @@ } catch (NumberFormatException e) { // ignore malformed Cache-Control headers - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/data/osm/TagCollection.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/osm/TagCollection.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/data/osm/TagCollection.java (revision 10626) @@ -770,7 +770,5 @@ result += Integer.parseInt(value); } catch (NumberFormatException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java =================================================================== --- /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java (revision 10626) @@ -304,5 +304,7 @@ gv.setGlyphTransform(0, AffineTransform.getTranslateInstance(1000, 1000)); Shape shape = gv.getGlyphOutline(0); - Main.trace("#10446: shape: "+shape.getBounds()); + if (Main.isTraceEnabled()) { + Main.trace("#10446: shape: "+shape.getBounds()); + } // x is about 1000 on normal stystems and about 2000 when the bug occurs int x = shape.getBounds().x; Index: /trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java (revision 10626) @@ -1075,7 +1075,5 @@ } catch (ParseException e) { // Try next format - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java (revision 10626) @@ -318,7 +318,5 @@ } catch (NumberFormatException e) { // don't save invalid value to preferences - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java (revision 10626) @@ -427,7 +427,5 @@ } catch (IOException e) { // silence exceptions - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } Index: /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java (revision 10626) @@ -752,7 +752,5 @@ } } catch (NumberFormatException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } @@ -774,7 +772,5 @@ } } catch (NumberFormatException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java (revision 10626) @@ -1447,7 +1447,5 @@ } catch (NumberFormatException e) { // ignore - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } else if ("style-type".equals(key)) { Index: /trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java (revision 10626) @@ -72,7 +72,5 @@ } catch (SAXException e) { // We will handle this in step with validation - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } Index: /trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginUpdatePolicyPanel.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginUpdatePolicyPanel.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginUpdatePolicyPanel.java (revision 10626) @@ -196,7 +196,5 @@ } catch (NumberFormatException e) { // ignore - load from preference pluginmanager.time-based-update.interval - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } if (days <= 0) { Index: /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java (revision 10626) @@ -91,7 +91,5 @@ } catch (NumberFormatException e) { // either the new text or the current text isn't a number. We continue with autocompletion - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItem.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItem.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItem.java (revision 10626) @@ -119,7 +119,5 @@ return Integer.valueOf(str); } catch (NumberFormatException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return null; Index: /trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java (revision 10626) @@ -84,7 +84,5 @@ } catch (NumberFormatException ex) { // Ignore - cannot auto-increment if last was non-numeric - if (Main.isTraceEnabled()) { - Main.trace(ex.getMessage()); - } + Main.trace(ex); } } else if (!usage.hadKeys() || PROP_FILL_DEFAULT.get() || "force".equals(use_last_as_default)) { Index: /trunk/src/org/openstreetmap/josm/gui/widgets/ChangesetIdTextField.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/widgets/ChangesetIdTextField.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/widgets/ChangesetIdTextField.java (revision 10626) @@ -84,7 +84,5 @@ } catch (NumberFormatException e) { // Ignored - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/gui/widgets/TextContextualPopupMenu.java =================================================================== --- /trunk/src/org/openstreetmap/josm/gui/widgets/TextContextualPopupMenu.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/gui/widgets/TextContextualPopupMenu.java (revision 10626) @@ -196,7 +196,5 @@ undo.undo(); } catch (CannotUndoException ex) { - if (Main.isTraceEnabled()) { - Main.trace(ex.getMessage()); - } + Main.trace(ex); } finally { updateUndoState(); @@ -231,7 +229,5 @@ undo.redo(); } catch (CannotRedoException ex) { - if (Main.isTraceEnabled()) { - Main.trace(ex.getMessage()); - } + Main.trace(ex); } finally { updateRedoState(); Index: /trunk/src/org/openstreetmap/josm/io/OsmApiException.java =================================================================== --- /trunk/src/org/openstreetmap/josm/io/OsmApiException.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/io/OsmApiException.java (revision 10626) @@ -146,7 +146,5 @@ } catch (IllegalArgumentException e) { // Ignored - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } try { @@ -159,7 +157,5 @@ } catch (IllegalArgumentException e) { // Ignored - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return sb.toString(); Index: /trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java =================================================================== --- /trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java (revision 10626) @@ -117,7 +117,5 @@ // TODO: handle thumbnail loading } catch (NumberFormatException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java =================================================================== --- /trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java (revision 10626) @@ -467,7 +467,5 @@ OnlineResource.JOSM_WEBSITE.checkOfflineAccess(updateSite, Main.getJOSMWebsite()); } catch (OfflineAccessException e) { - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); return false; } Index: /trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java =================================================================== --- /trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java (revision 10626) @@ -476,7 +476,5 @@ } catch (MalformedURLException ex) { // shouldn't happen - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(ex); } @@ -625,7 +623,5 @@ } catch (MalformedURLException ex) { // shouldn't happen - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } Index: /trunk/src/org/openstreetmap/josm/tools/I18n.java =================================================================== --- /trunk/src/org/openstreetmap/josm/tools/I18n.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/tools/I18n.java (revision 10626) @@ -486,7 +486,5 @@ } catch (IOException e) { // Ignore - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } @@ -518,7 +516,5 @@ } catch (IOException e) { // Ignore exception - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } return false; Index: /trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java =================================================================== --- /trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java (revision 10626) @@ -403,7 +403,5 @@ } catch (IOException e) { // Ignore - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java =================================================================== --- /trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java (revision 10625) +++ /trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java (revision 10626) @@ -242,4 +242,5 @@ // If exception this is not a certificate related to JOSM, just trace it Main.trace(alias + " --> " + e.getClass().getName()); + Main.trace(e); } } Index: /trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java =================================================================== --- /trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java (revision 10625) +++ /trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java (revision 10626) @@ -82,7 +82,5 @@ } catch (IndexOutOfBoundsException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } Index: /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergeModelTest.java =================================================================== --- /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergeModelTest.java (revision 10625) +++ /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergeModelTest.java (revision 10626) @@ -360,7 +360,5 @@ } catch (IllegalArgumentException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } @@ -370,7 +368,5 @@ } catch (IllegalArgumentException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeItemTest.java =================================================================== --- /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeItemTest.java (revision 10625) +++ /trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeItemTest.java (revision 10626) @@ -93,7 +93,5 @@ } catch (IllegalArgumentException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } @@ -142,7 +140,5 @@ } catch (IllegalStateException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } @@ -157,7 +153,5 @@ } catch (IllegalArgumentException e) { // OK - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } } Index: /trunk/test/unit/org/openstreetmap/josm/tools/OsmUrlToBoundsTest.java =================================================================== --- /trunk/test/unit/org/openstreetmap/josm/tools/OsmUrlToBoundsTest.java (revision 10625) +++ /trunk/test/unit/org/openstreetmap/josm/tools/OsmUrlToBoundsTest.java (revision 10626) @@ -72,7 +72,5 @@ } catch (IllegalArgumentException e) { // Ignore. check if bounds is null after - if (Main.isTraceEnabled()) { - Main.trace(e.getMessage()); - } + Main.trace(e); } Assert.assertEquals(item.url, item.bounds, bounds);