You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by mc...@apache.org on 2008/03/21 13:56:49 UTC

svn commit: r639610 - in /incubator/tuscany/java/sca/modules/host-tomcat/src: main/java/org/apache/tuscany/sca/http/tomcat/ main/java/org/apache/tuscany/sca/http/tomcat/module/ test/java/org/apache/tuscany/sca/http/tomcat/

Author: mcombellack
Date: Fri Mar 21 05:56:41 2008
New Revision: 639610

URL: http://svn.apache.org/viewvc?rev=639610&view=rev
Log:
Minor JavaDoc updates and spelling corrections

Modified:
    incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/ServletWrapper.java
    incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/TomcatServer.java
    incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/module/TomcatRuntimeModuleActivator.java
    incubator/tuscany/java/sca/modules/host-tomcat/src/test/java/org/apache/tuscany/sca/http/tomcat/TomcatServerTestCase.java

Modified: incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/ServletWrapper.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/ServletWrapper.java?rev=639610&r1=639609&r2=639610&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/ServletWrapper.java (original)
+++ incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/ServletWrapper.java Fri Mar 21 05:56:41 2008
@@ -24,7 +24,7 @@
 import org.apache.catalina.core.StandardWrapper;
 
 /**
- * A servlet wrapper.
+ * A Servlet wrapper.
  * 
  * @version $Rev$ $Date$
  */

Modified: incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/TomcatServer.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/TomcatServer.java?rev=639610&r1=639609&r2=639610&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/TomcatServer.java (original)
+++ incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/TomcatServer.java Fri Mar 21 05:56:41 2008
@@ -261,7 +261,7 @@
             ports.put(portNumber, port);
         }
 
-        // Register the servlet mapping
+        // Register the Servlet mapping
         String path = uri.getPath();
 
         if (!path.startsWith("/")) {
@@ -276,8 +276,8 @@
         if (servlet instanceof DefaultResourceServlet) {
             String defaultServletPath = path;
 
-            // Optimize the handling of resource requests, use the Tomcat default servlet
-            // instead of our default resource servlet
+            // Optimize the handling of resource requests, use the Tomcat default Servlet
+            // instead of our default resource Servlet
             if (defaultServletPath.endsWith("*")) {
                 defaultServletPath = defaultServletPath.substring(0, defaultServletPath.length() - 1);
             }
@@ -299,7 +299,7 @@
         context.addServletMapping(path, path);
         port.getConnector().getMapper().addWrapper("localhost", "", path, wrapper);
 
-        // Initialize the servlet
+        // Initialize the Servlet
         try {
             wrapper.initServlet();
         } catch (ServletException e) {
@@ -452,15 +452,15 @@
                 context.removeServletMapping(mapping);
             } catch (NegativeArraySizeException e) {
                 // JIRA TUSCANY-1599
-                //FIXME Looks like a bug in Tomcat when removing the last
-                // servlet in the list, catch the exception for now as it doesn't
+                // FIXME Looks like a bug in Tomcat when removing the last
+                // Servlet in the list, catch the exception for now as it doesn't
                 // seem harmful, will find a better solution for the next release
             }
             context.removeChild(servletWrapper);
             try {
                 servletWrapper.destroyServlet();
             } catch (Exception ex) {
-                // Temporary hack to stop destruction of servlets without servlet
+                // Temporary hack to stop destruction of Servlets without Servlet
                 // context 
             }
             //logger.info("Remove Servlet mapping: " + suri);
@@ -489,7 +489,7 @@
         private class CustomHttpProtocolHandler extends Http11Protocol {
 
             /**
-             * An Executor wrappering our WorkScheduler
+             * An Executor wrapping our WorkScheduler
              */
             private class WorkSchedulerExecutor implements Executor {
                 public void execute(Runnable command) {

Modified: incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/module/TomcatRuntimeModuleActivator.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/module/TomcatRuntimeModuleActivator.java?rev=639610&r1=639609&r2=639610&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/module/TomcatRuntimeModuleActivator.java (original)
+++ incubator/tuscany/java/sca/modules/host-tomcat/src/main/java/org/apache/tuscany/sca/http/tomcat/module/TomcatRuntimeModuleActivator.java Fri Mar 21 05:56:41 2008
@@ -34,7 +34,7 @@
 
     public void start(ExtensionPointRegistry extensionPointRegistry) {
 
-        // Register a Tomcat servlet host
+        // Register a Tomcat Servlet host
         ServletHostExtensionPoint servletHosts =
             extensionPointRegistry.getExtensionPoint(ServletHostExtensionPoint.class);
         WorkScheduler workScheduler = extensionPointRegistry.getExtensionPoint(WorkScheduler.class);

Modified: incubator/tuscany/java/sca/modules/host-tomcat/src/test/java/org/apache/tuscany/sca/http/tomcat/TomcatServerTestCase.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/host-tomcat/src/test/java/org/apache/tuscany/sca/http/tomcat/TomcatServerTestCase.java?rev=639610&r1=639609&r2=639610&view=diff
==============================================================================
--- incubator/tuscany/java/sca/modules/host-tomcat/src/test/java/org/apache/tuscany/sca/http/tomcat/TomcatServerTestCase.java (original)
+++ incubator/tuscany/java/sca/modules/host-tomcat/src/test/java/org/apache/tuscany/sca/http/tomcat/TomcatServerTestCase.java Fri Mar 21 05:56:41 2008
@@ -77,7 +77,7 @@
     };      
 
     /**
-     * Verifies requests are properly routed according to the servlet mapping
+     * Verifies requests are properly routed according to the Servlet mapping
      */
     public void testRegisterServletMapping() throws Exception {
         TomcatServer service = new TomcatServer(workScheduler);
@@ -93,7 +93,7 @@
     }
 
     /**
-     * Verifies that servlets can be registered with multiple ports
+     * Verifies that Servlets can be registered with multiple ports
      */
     public void testRegisterMultiplePorts() throws Exception {
         TomcatServer service = new TomcatServer(workScheduler);



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org