Changeset 5874 in josm for trunk/src/org/openstreetmap/josm/tools/BugReportExceptionHandler.java

Timestamp:

2013-04-16T19:57:43+02:00 (13 years ago)

Author:

Don-vip

Message:

see #8570, #7406 - I/O refactorization:

• Move different file copy functions to Utils.copyFile (to be replaced later by Files.copy when switching to Java 7)
• Replace all Utils.close(XXX) by Utils.close(Closeable) -> impacted plugins: commandline, mirrored_download, opendata, piclayer
• Add new Utils.close(ZipFile)
• Use of Utils.close almost everywhere
• Javadoc fixes

File:

• trunk/src/org/openstreetmap/josm/tools/BugReportExceptionHandler.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/tools/BugReportExceptionHandler.java

@@ -44 +44 @@
 
     //http://stuffthathappens.com/blog/2007/10/15/one-more-note-on-uncaught-exception-handlers/
+    /**
+     * Handles the given throwable object
+     * @param t The throwable object
+     */
     public void handle(Throwable t) {
         handleException(t);
     }
 
+    /**
+     * Handles the given exception
+     * @param e the exception
+     */
     public static void handleException(final Throwable e) {
         if (handlingInProgress)
@@ -165 +173 @@
         }
     }
+    
+    /**
+     * Determines if an exception is currently being handled
+     * @return {@code true} if an exception is currently being handled, {@code false} otherwise
+     */
     public static boolean exceptionHandlingInProgress() {
         return handlingInProgress;
@@ -180 +193 @@
             GZIPOutputStream gzip = new GZIPOutputStream(out);
             gzip.write(debugText.getBytes("UTF-8"));
-            gzip.close();
+            Utils.close(gzip);
     
             return new URL("http://josm.openstreetmap.de/josmticket?" +