On 22/06/2009, ma...@apache.org <ma...@apache.org> wrote:
> Author: markt
> Date: Mon Jun 22 11:58:07 2009
> New Revision: 787211
>
> URL: http://svn.apache.org/viewvc?rev=787211&view=rev
> Log:
> Fix a couple of Eclipse warnings
>
> Modified:
> tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java
>
> Modified: tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java
> URL:
> http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java?rev=787211&r1=787210&r2=787211&view=diff
>
> ==============================================================================
> --- tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java (original)
> +++ tomcat/trunk/test/org/apache/catalina/startup/TestTomcat.java Mon Jun 22
> 11:58:07 2009
> @@ -88,8 +88,7 @@
> // You can customize the context by calling
> // its API
>
> - tomcat.addServlet(ctx, "myServlet",
> - new HelloWorld());
> + Tomcat.addServlet(ctx, "myServlet", new HelloWorld());
Looks a bit odd - are there really both "tomcat" and "Tomcat" objects/classes?
> ctx.addServletMapping("/", "myServlet");
>
> tomcat.start();
If so, is this the correct cat?
> @@ -103,9 +102,7 @@
> File appDir =
> new File(base + "output/build/webapps/examples");
> // app dir is relative to server home
> - StandardContext ctx =
> - tomcat.addWebapp(null, "/examples",
> - appDir.getAbsolutePath());
> + tomcat.addWebapp(null, "/examples", appDir.getAbsolutePath());
>
> tomcat.start();
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
