svn commit: r1044091 - /tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java

[markt]

Author: markt
Date: Thu Dec  9 19:02:01 2010
New Revision: 1044091

URL: http://svn.apache.org/viewvc?rev=1044091&view=rev
Log:
Re-factoring in support of 
https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Log the right message at the right point

Modified:
    tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java

Modified: tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java?rev=1044091&r1=1044090&r2=1044091&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java (original)
+++ tomcat/trunk/java/org/apache/coyote/AbstractProtocolHandler.java Thu Dec  9 
19:02:01 2010
@@ -392,16 +392,16 @@ public abstract class AbstractProtocolHa
 
     @Override
     public void stop() throws Exception {
+        if(getLog().isInfoEnabled())
+            getLog().info(sm.getString("abstractProtocolHandler.stop",
+                    getName()));
         try {
             endpoint.stop();
         } catch (Exception ex) {
-            getLog().error(sm.getString("abstractProtocolHandler.stop",
+            getLog().error(sm.getString("abstractProtocolHandler.stopError",
                     getName()), ex);
             throw ex;
         }
-        if(getLog().isInfoEnabled())
-            getLog().info(sm.getString("abstractProtocolHandler.stopError",
-                    getName()));
     }
 
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org