changeset 7032:5a1f6e96e163

close() method for dump handlers should not throw IOException
author Christian Wimmer <christian.wimmer@oracle.com>
date Mon, 26 Nov 2012 18:24:08 -0800
parents 2e577202843c
children d9930fec7553
files graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugDumpHandler.java graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinterObserver.java graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CompilationPrinter.java graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinter.java graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinterDumpHandler.java
diffstat 6 files changed, 16 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugDumpHandler.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/DebugDumpHandler.java	Mon Nov 26 18:24:08 2012 -0800
@@ -26,4 +26,7 @@
 
 public interface DebugDumpHandler extends Closeable {
     void dump(Object object, String message);
+
+    @Override
+    void close();
 }
--- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/BinaryGraphPrinter.java	Mon Nov 26 18:24:08 2012 -0800
@@ -438,8 +438,12 @@
     }
 
     @Override
-    public void close() throws IOException {
-        flush();
-        channel.close();
+    public void close() {
+        try {
+            flush();
+            channel.close();
+        } catch (IOException ex) {
+            throw new Error(ex);
+        }
     }
 }
--- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinterObserver.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CFGPrinterObserver.java	Mon Nov 26 18:24:08 2012 -0800
@@ -195,7 +195,7 @@
     }
 
     @Override
-    public void close() throws IOException {
+    public void close() {
         if (cfgPrinter != null) {
             cfgPrinter.close();
         }
--- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CompilationPrinter.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/CompilationPrinter.java	Mon Nov 26 18:24:08 2012 -0800
@@ -82,7 +82,7 @@
     }
 
     @Override
-    public void close() throws IOException {
+    public void close() {
         out.out().close();
     }
 
--- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinter.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinter.java	Mon Nov 26 18:24:08 2012 -0800
@@ -43,4 +43,7 @@
      * Ends the current group.
      */
     void endGroup() throws IOException;
+
+    @Override
+    void close();
 }
--- a/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinterDumpHandler.java	Mon Nov 26 18:17:14 2012 -0800
+++ b/graal/com.oracle.graal.printer/src/com/oracle/graal/printer/GraphPrinterDumpHandler.java	Mon Nov 26 18:24:08 2012 -0800
@@ -200,7 +200,7 @@
     }
 
     @Override
-    public void close() throws IOException {
+    public void close() {
         for (int i = 0; i < previousInlineContext.size(); i++) {
             closeScope();
         }