Mercurial > hg > graal-jvmci-8
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(); }