Author: henning Date: Sat Oct 1 04:09:37 2005 New Revision: 292953 URL: http://svn.apache.org/viewcvs?rev=292953&view=rev Log: We can actually do even a bit better by turning the PrintWriter/StringWriter combo around. Funny how hard it is to get that small piece of code really right...
Modified: jakarta/velocity/core/trunk/src/java/org/apache/velocity/app/event/implement/PrintExceptions.java Modified: jakarta/velocity/core/trunk/src/java/org/apache/velocity/app/event/implement/PrintExceptions.java URL: http://svn.apache.org/viewcvs/jakarta/velocity/core/trunk/src/java/org/apache/velocity/app/event/implement/PrintExceptions.java?rev=292953&r1=292952&r2=292953&view=diff ============================================================================== --- jakarta/velocity/core/trunk/src/java/org/apache/velocity/app/event/implement/PrintExceptions.java (original) +++ jakarta/velocity/core/trunk/src/java/org/apache/velocity/app/event/implement/PrintExceptions.java Sat Oct 1 04:09:37 2005 @@ -87,26 +87,20 @@ private static String getStackTrace(Throwable throwable) { - StringWriter stackTraceWriter = null; + PrintWriter printWriter = null; try { - stackTraceWriter = new StringWriter(); - throwable.printStackTrace(new PrintWriter(stackTraceWriter)); - stackTraceWriter.flush(); + StringWriter stackTraceWriter = new StringWriter(); + printWriter = new PrintWriter(stackTraceWriter); + throwable.printStackTrace(printWriter); + printWriter.flush(); return stackTraceWriter.toString(); } finally { - if (stackTraceWriter != null) + if (printWriter != null) { - try - { - stackTraceWriter.close(); - } - catch (IOException ioe) - { - // do nothing - } + printWriter.close(); } } } --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]