cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Glen Mazza <glen.ma...@verizon.net>
Subject Re: svn commit: r567631 - in /incubator/cxf/trunk/tools: common/src/main/java/org/apache/cxf/tools/common/toolspec/ javato/core/src/main/java/org/apache/cxf/tools/java2wsdl/ javato/ws/src/main/java/org/apache/cxf/tools/java2ws/ wsdlto/core/src/main/java/or...
Date Mon, 20 Aug 2007 12:02:44 GMT
Am Montag, den 20.08.2007, 09:55 +0000 schrieb mmao@apache.org:
> Author: mmao
> Date: Mon Aug 20 02:55:57 2007
> New Revision: 567631
> 
> URL: http://svn.apache.org/viewvc?rev=567631&view=rev
> Log:
> CXF-912
> Restore the system standard out/err, before tools exit
> 
> Modified:
> incubator/cxf/trunk/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
> URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java?rev=567631&r1=567630&r2=567631&view=diff
> ==============================================================================
> --- incubator/cxf/trunk/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
(original)
> +++ incubator/cxf/trunk/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
Mon Aug 20 02:55:57 2007
> @@ -240,6 +240,8 @@
>              throw ex;
>          } catch (Exception ex) {
>              throw new ToolException(ex);
> +        } finally {
> +            tearDown();
>          }
>      }
>  

There seems to be something wrong with this one.  If you're going to
throw new ToolException(ex) the tearDown() in the finally block will
*not* be called, correct?  Perhaps the tearDown() should be moved into
the code before the catch() block.

Glen



Mime
View raw message