You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2010/06/24 20:54:25 UTC

svn commit: r957681 - /tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java

Author: markt
Date: Thu Jun 24 18:54:24 2010
New Revision: 957681

URL: http://svn.apache.org/viewvc?rev=957681&view=rev
Log:
Clean-up

Modified:
    tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java

Modified: tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java?rev=957681&r1=957680&r2=957681&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java (original)
+++ tomcat/trunk/java/org/apache/catalina/manager/host/HostManagerServlet.java Thu Jun 24 18:54:24 2010
@@ -567,15 +567,17 @@ public class HostManagerServlet
             return;
         }
 
+        Container host = engine.findChild(name);
+        
         // Check if host exists
-        if (engine.findChild(name) == null) {
+        if (host == null) {
             writer.println
                 (sm.getString("hostManagerServlet.noHost", name));
             return;
         }
 
         // Prevent starting our own host
-        if (engine.findChild(name) == installedHost) {
+        if (host == installedHost) {
             writer.println
                 (sm.getString("hostManagerServlet.cannotStartOwnHost", name));
             return;
@@ -583,7 +585,7 @@ public class HostManagerServlet
 
         // Start host
         try {
-            engine.findChild(name).start();
+            host.start();
             writer.println
                 (sm.getString("hostManagerServlet.started", name));
         } catch (Throwable t) {
@@ -617,15 +619,17 @@ public class HostManagerServlet
             return;
         }
 
+        Container host = engine.findChild(name);
+
         // Check if host exists
-        if (engine.findChild(name) == null) {
+        if (host == null) {
             writer.println
                 (sm.getString("hostManagerServlet.noHost", name));
             return;
         }
 
         // Prevent starting our own host
-        if (engine.findChild(name) == installedHost) {
+        if (host == installedHost) {
             writer.println
                 (sm.getString("hostManagerServlet.cannotStopOwnHost", name));
             return;
@@ -633,7 +637,7 @@ public class HostManagerServlet
 
         // Start host
         try {
-            engine.findChild(name).stop();
+            host.stop();
             writer.println
                 (sm.getString("hostManagerServlet.stopped", name));
         } catch (Throwable t) {



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