Am Montag, den 20.08.2007, 09:55 +0000 schrieb [EMAIL PROTECTED]:
> 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


Reply via email to