From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 4 Mar 2011 15:52:01 +0000 (+0000)
Subject: Code clean-up
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=757ea8bad319525b3b2aaae1aa44b4021c05a27f;p=tomcat7.0

Code clean-up
 - fix FindBugs warnings
 - remove unused code

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1078022 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/core/ApplicationContext.java b/java/org/apache/catalina/core/ApplicationContext.java
index 33a8a67e3..22925c27d 100644
--- a/java/org/apache/catalina/core/ApplicationContext.java
+++ b/java/org/apache/catalina/core/ApplicationContext.java
@@ -520,12 +520,10 @@ public class ApplicationContext
     public URL getResource(String path)
         throws MalformedURLException {
 
-        if (path == null)
-            throw new MalformedURLException(sm.getString("applicationContext.requestDispatcher.iae", path));
-
-        if (!path.startsWith("/") && GET_RESOURCE_REQUIRE_SLASH)
-            throw new MalformedURLException(sm.getString("applicationContext.requestDispatcher.iae", path));
-
+        if (path == null ||
+                !path.startsWith("/") && GET_RESOURCE_REQUIRE_SLASH)
+            throw new MalformedURLException(sm.getString(
+                    "applicationContext.requestDispatcher.iae", path));
         
         String normPath = RequestUtil.normalize(path);
         if (normPath == null)
diff --git a/java/org/apache/catalina/core/ApplicationFilterChain.java b/java/org/apache/catalina/core/ApplicationFilterChain.java
index 5fcd27818..ec87d3165 100644
--- a/java/org/apache/catalina/core/ApplicationFilterChain.java
+++ b/java/org/apache/catalina/core/ApplicationFilterChain.java
@@ -239,7 +239,6 @@ final class ApplicationFilterChain implements FilterChain, CometFilterChain {
                     SecurityUtil.doAsPrivilege
                         ("doFilter", filter, classType, args, principal);
                     
-                    args = null;
                 } else {  
                     filter.doFilter(request, response, this);
                 }
@@ -301,7 +300,6 @@ final class ApplicationFilterChain implements FilterChain, CometFilterChain {
                                                classTypeUsedInService, 
                                                args,
                                                principal);   
-                    args = null;
                 } else {  
                     servlet.service(request, response);
                 }
@@ -426,7 +424,6 @@ final class ApplicationFilterChain implements FilterChain, CometFilterChain {
                     SecurityUtil.doAsPrivilege("doFilterEvent", filter,
                             cometClassType, args, principal);
 
-                    args = null;
                 } else {  
                     filter.doFilterEvent(event, this);
                 }
@@ -481,7 +478,6 @@ final class ApplicationFilterChain implements FilterChain, CometFilterChain {
                         classTypeUsedInEvent, 
                         args,
                         principal);
-                args = null;
             } else {  
                 ((CometProcessor) servlet).event(event);
             }
diff --git a/java/org/apache/catalina/core/ApplicationFilterConfig.java b/java/org/apache/catalina/core/ApplicationFilterConfig.java
index f0e8ba94f..2797a576f 100644
--- a/java/org/apache/catalina/core/ApplicationFilterConfig.java
+++ b/java/org/apache/catalina/core/ApplicationFilterConfig.java
@@ -110,7 +110,7 @@ public final class ApplicationFilterConfig implements FilterConfig, Serializable
     /**
      * The Context with which we are associated.
      */
-    private Context context = null;
+    private transient Context context = null;
 
 
     /**
diff --git a/java/org/apache/catalina/core/ApplicationHttpRequest.java b/java/org/apache/catalina/core/ApplicationHttpRequest.java
index a158dde30..df2c97373 100644
--- a/java/org/apache/catalina/core/ApplicationHttpRequest.java
+++ b/java/org/apache/catalina/core/ApplicationHttpRequest.java
@@ -39,7 +39,6 @@ import org.apache.catalina.Manager;
 import org.apache.catalina.Session;
 import org.apache.catalina.util.Enumerator;
 import org.apache.catalina.util.RequestUtil;
-import org.apache.tomcat.util.res.StringManager;
 
 
 /**
@@ -81,13 +80,6 @@ class ApplicationHttpRequest extends HttpServletRequestWrapper {
       RequestDispatcher.FORWARD_QUERY_STRING };
 
 
-    /**
-     * The string manager for this package.
-     */
-    protected static final StringManager sm =
-        StringManager.getManager(Constants.Package);
-
-
     // ----------------------------------------------------------- Constructors