Changeset 10626 in josm


Ignore:
Timestamp:
2016-07-24T13:54:44+02:00 (3 years ago)
Author:
Don-vip
Message:

simplify handling of ignored/traced exceptions

Location:
trunk
Files:
30 edited

Legend:

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

    r10571 r10626  
    235235                } catch (NumberFormatException nfe) {
    236236                    // we repaint offset numbers in any case
    237                     if (Main.isTraceEnabled()) {
    238                         Main.trace(nfe.getMessage());
    239                     }
     237                    Main.trace(nfe);
    240238                }
    241239            }
  • trunk/src/org/openstreetmap/josm/actions/SearchNotesDownloadAction.java

    r10242 r10626  
    8888            return;
    8989        } catch (NumberFormatException ignore) {
    90             if (Main.isTraceEnabled()) {
    91                 Main.trace(ignore.getMessage());
    92             }
     90            Main.trace(ignore);
    9391        }
    9492
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r10600 r10626  
    734734                }
    735735            } catch (NumberFormatException ignore) {
    736                 if (Main.isTraceEnabled()) {
    737                     Main.trace(ignore.getMessage());
    738                 }
     736                Main.trace(ignore);
    739737            }
    740738            this.referenceNumber = v;
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r10608 r10626  
    793793        } catch (NumberFormatException e) {
    794794            // fall out
    795             if (Main.isTraceEnabled()) {
    796                 Main.trace(e.getMessage());
    797             }
     795            Main.trace(e);
    798796        }
    799797        return def;
     
    811809        } catch (NumberFormatException e) {
    812810            // fall out
    813             if (Main.isTraceEnabled()) {
    814                 Main.trace(e.getMessage());
    815             }
     811            Main.trace(e);
    816812        }
    817813        return def;
     
    827823        } catch (NumberFormatException e) {
    828824            // fall out
    829             if (Main.isTraceEnabled()) {
    830                 Main.trace(e.getMessage());
    831             }
     825            Main.trace(e);
    832826        }
    833827        return def;
     
    843837        } catch (NumberFormatException e) {
    844838            // fall out
    845             if (Main.isTraceEnabled()) {
    846                 Main.trace(e.getMessage());
    847             }
     839            Main.trace(e);
    848840        }
    849841        return def;
  • trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

    r10558 r10626  
    441441            } catch (NumberFormatException e) {
    442442                // ignore malformed Cache-Control headers
    443                 if (Main.isTraceEnabled()) {
    444                     Main.trace(e.getMessage());
    445                 }
     443                Main.trace(e);
    446444            }
    447445        }
  • trunk/src/org/openstreetmap/josm/data/osm/TagCollection.java

    r8510 r10626  
    770770                result += Integer.parseInt(value);
    771771            } catch (NumberFormatException e) {
    772                 if (Main.isTraceEnabled()) {
    773                     Main.trace(e.getMessage());
    774                 }
     772                Main.trace(e);
    775773            }
    776774        }
  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/StyledMapRenderer.java

    r10378 r10626  
    304304            gv.setGlyphTransform(0, AffineTransform.getTranslateInstance(1000, 1000));
    305305            Shape shape = gv.getGlyphOutline(0);
    306             Main.trace("#10446: shape: "+shape.getBounds());
     306            if (Main.isTraceEnabled()) {
     307                Main.trace("#10446: shape: "+shape.getBounds());
     308            }
    307309            // x is about 1000 on normal stystems and about 2000 when the bug occurs
    308310            int x = shape.getBounds().x;
  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java

    r10378 r10626  
    10751075                } catch (ParseException e) {
    10761076                    // Try next format
    1077                     if (Main.isTraceEnabled()) {
    1078                         Main.trace(e.getMessage());
    1079                     }
     1077                    Main.trace(e);
    10801078                }
    10811079            }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadStrategySelectionPanel.java

    r10217 r10626  
    318318        } catch (NumberFormatException e) {
    319319            // don't save invalid value to preferences
    320             if (Main.isTraceEnabled()) {
    321                 Main.trace(e.getMessage());
    322             }
     320            Main.trace(e);
    323321        }
    324322    }
  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r10619 r10626  
    427427                } catch (IOException e) {
    428428                    // silence exceptions
    429                     if (Main.isTraceEnabled()) {
    430                         Main.trace(e.getMessage());
    431                     }
     429                    Main.trace(e);
    432430                }
    433431
  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r10619 r10626  
    752752                    }
    753753                } catch (NumberFormatException e) {
    754                     if (Main.isTraceEnabled()) {
    755                         Main.trace(e.getMessage());
    756                     }
     754                    Main.trace(e);
    757755                }
    758756            }
     
    774772                    }
    775773                } catch (NumberFormatException e) {
    776                     if (Main.isTraceEnabled()) {
    777                         Main.trace(e.getMessage());
    778                     }
     774                    Main.trace(e);
    779775                }
    780776            }
  • trunk/src/org/openstreetmap/josm/gui/preferences/SourceEditor.java

    r10619 r10626  
    14471447                                } catch (NumberFormatException e) {
    14481448                                    // ignore
    1449                                     if (Main.isTraceEnabled()) {
    1450                                         Main.trace(e.getMessage());
    1451                                     }
     1449                                    Main.trace(e);
    14521450                                }
    14531451                            } else if ("style-type".equals(key)) {
  • trunk/src/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreference.java

    r10378 r10626  
    7272                        } catch (SAXException e) {
    7373                            // We will handle this in step with validation
    74                             if (Main.isTraceEnabled()) {
    75                                 Main.trace(e.getMessage());
    76                             }
     74                            Main.trace(e);
    7775                        }
    7876
  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginUpdatePolicyPanel.java

    r10179 r10626  
    196196            } catch (NumberFormatException e) {
    197197                // ignore - load from preference pluginmanager.time-based-update.interval
    198                 if (Main.isTraceEnabled()) {
    199                     Main.trace(e.getMessage());
    200                 }
     198                Main.trace(e);
    201199            }
    202200            if (days <= 0) {
  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingTextField.java

    r8840 r10626  
    9191                } catch (NumberFormatException e) {
    9292                    // either the new text or the current text isn't a number. We continue with autocompletion
    93                     if (Main.isTraceEnabled()) {
    94                         Main.trace(e.getMessage());
    95                     }
     93                    Main.trace(e);
    9694                }
    9795            }
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetItem.java

    r10413 r10626  
    119119            return Integer.valueOf(str);
    120120        } catch (NumberFormatException e) {
    121             if (Main.isTraceEnabled()) {
    122                 Main.trace(e.getMessage());
    123             }
     121            Main.trace(e);
    124122        }
    125123        return null;
  • trunk/src/org/openstreetmap/josm/gui/tagging/presets/items/Text.java

    r10590 r10626  
    8484                } catch (NumberFormatException ex) {
    8585                    // Ignore - cannot auto-increment if last was non-numeric
    86                     if (Main.isTraceEnabled()) {
    87                         Main.trace(ex.getMessage());
    88                     }
     86                    Main.trace(ex);
    8987                }
    9088            } else if (!usage.hadKeys() || PROP_FILL_DEFAULT.get() || "force".equals(use_last_as_default)) {
  • trunk/src/org/openstreetmap/josm/gui/widgets/ChangesetIdTextField.java

    r8840 r10626  
    8484                } catch (NumberFormatException e) {
    8585                    // Ignored
    86                     if (Main.isTraceEnabled()) {
    87                         Main.trace(e.getMessage());
    88                     }
     86                    Main.trace(e);
    8987                }
    9088            }
  • trunk/src/org/openstreetmap/josm/gui/widgets/TextContextualPopupMenu.java

    r10611 r10626  
    196196                undo.undo();
    197197            } catch (CannotUndoException ex) {
    198                 if (Main.isTraceEnabled()) {
    199                     Main.trace(ex.getMessage());
    200                 }
     198                Main.trace(ex);
    201199            } finally {
    202200                updateUndoState();
     
    231229                undo.redo();
    232230            } catch (CannotRedoException ex) {
    233                 if (Main.isTraceEnabled()) {
    234                     Main.trace(ex.getMessage());
    235                 }
     231                Main.trace(ex);
    236232            } finally {
    237233                updateRedoState();
  • trunk/src/org/openstreetmap/josm/io/OsmApiException.java

    r10212 r10626  
    146146        } catch (IllegalArgumentException e) {
    147147            // Ignored
    148             if (Main.isTraceEnabled()) {
    149                 Main.trace(e.getMessage());
    150             }
     148            Main.trace(e);
    151149        }
    152150        try {
     
    159157        } catch (IllegalArgumentException e) {
    160158            // Ignored
    161             if (Main.isTraceEnabled()) {
    162                 Main.trace(e.getMessage());
    163             }
     159            Main.trace(e);
    164160        }
    165161        return sb.toString();
  • trunk/src/org/openstreetmap/josm/io/session/GeoImageSessionImporter.java

    r10216 r10626  
    117117            // TODO: handle thumbnail loading
    118118        } catch (NumberFormatException e) {
    119             if (Main.isTraceEnabled()) {
    120                 Main.trace(e.getMessage());
    121             }
     119            Main.trace(e);
    122120        }
    123121    }
  • trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java

    r10619 r10626  
    467467                    OnlineResource.JOSM_WEBSITE.checkOfflineAccess(updateSite, Main.getJOSMWebsite());
    468468                } catch (OfflineAccessException e) {
    469                     if (Main.isTraceEnabled()) {
    470                         Main.trace(e.getMessage());
    471                     }
     469                    Main.trace(e);
    472470                    return false;
    473471                }
  • trunk/src/org/openstreetmap/josm/tools/ExceptionUtil.java

    r10616 r10626  
    476476        } catch (MalformedURLException ex) {
    477477            // shouldn't happen
    478             if (Main.isTraceEnabled()) {
    479                 Main.trace(e.getMessage());
    480             }
     478            Main.trace(ex);
    481479        }
    482480
     
    625623        } catch (MalformedURLException ex) {
    626624            // shouldn't happen
    627             if (Main.isTraceEnabled()) {
    628                 Main.trace(e.getMessage());
    629             }
     625            Main.trace(e);
    630626        }
    631627
  • trunk/src/org/openstreetmap/josm/tools/I18n.java

    r10625 r10626  
    486486        } catch (IOException e) {
    487487            // Ignore
    488             if (Main.isTraceEnabled()) {
    489                 Main.trace(e.getMessage());
    490             }
     488            Main.trace(e);
    491489        }
    492490    }
     
    518516        } catch (IOException e) {
    519517            // Ignore exception
    520             if (Main.isTraceEnabled()) {
    521                 Main.trace(e.getMessage());
    522             }
     518            Main.trace(e);
    523519        }
    524520        return false;
  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r10616 r10626  
    403403                    } catch (IOException e) {
    404404                        // Ignore
    405                         if (Main.isTraceEnabled()) {
    406                             Main.trace(e.getMessage());
    407                         }
     405                        Main.trace(e);
    408406                    }
    409407                }
  • trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java

    r10580 r10626  
    242242                    // If exception this is not a certificate related to JOSM, just trace it
    243243                    Main.trace(alias + " --> " + e.getClass().getName());
     244                    Main.trace(e);
    244245                }
    245246            }
  • trunk/test/unit/org/openstreetmap/josm/data/osm/history/HistoryWayTest.java

    r10378 r10626  
    8282        } catch (IndexOutOfBoundsException e) {
    8383            // OK
    84             if (Main.isTraceEnabled()) {
    85                 Main.trace(e.getMessage());
    86             }
     84            Main.trace(e);
    8785        }
    8886
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListMergeModelTest.java

    r9547 r10626  
    360360        } catch (IllegalArgumentException e) {
    361361            // OK
    362             if (Main.isTraceEnabled()) {
    363                 Main.trace(e.getMessage());
    364             }
     362            Main.trace(e);
    365363        }
    366364
     
    370368        } catch (IllegalArgumentException e) {
    371369            // OK
    372             if (Main.isTraceEnabled()) {
    373                 Main.trace(e.getMessage());
    374             }
     370            Main.trace(e);
    375371        }
    376372    }
  • trunk/test/unit/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeItemTest.java

    r9547 r10626  
    9393        } catch (IllegalArgumentException e) {
    9494            // OK
    95             if (Main.isTraceEnabled()) {
    96                 Main.trace(e.getMessage());
    97             }
     95            Main.trace(e);
    9896        }
    9997    }
     
    142140        } catch (IllegalStateException e) {
    143141            // OK
    144             if (Main.isTraceEnabled()) {
    145                 Main.trace(e.getMessage());
    146             }
     142            Main.trace(e);
    147143        }
    148144    }
     
    157153        } catch (IllegalArgumentException e) {
    158154            // OK
    159             if (Main.isTraceEnabled()) {
    160                 Main.trace(e.getMessage());
    161             }
     155            Main.trace(e);
    162156        }
    163157    }
  • trunk/test/unit/org/openstreetmap/josm/tools/OsmUrlToBoundsTest.java

    r8836 r10626  
    7272            } catch (IllegalArgumentException e) {
    7373                // Ignore. check if bounds is null after
    74                 if (Main.isTraceEnabled()) {
    75                     Main.trace(e.getMessage());
    76                 }
     74                Main.trace(e);
    7775            }
    7876            Assert.assertEquals(item.url, item.bounds, bounds);
Note: See TracChangeset for help on using the changeset viewer.