Code clean-up

- fix FindBugs warnings - remove unused code git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1078049 13f79535-47bb-0310-9956-ffa450edef68
apache · Mar 4, 2011 · 008ce97 · 008ce97
1 parent 2c42e87
commit 008ce97
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 38 deletions.
diff --git a/java/org/apache/catalina/core/ApplicationHttpResponse.java b/java/org/apache/catalina/core/ApplicationHttpResponse.java
@@ -14,20 +14,15 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-
 package org.apache.catalina.core;
 
-
 import java.io.IOException;
 import java.util.Locale;
 
 import javax.servlet.http.Cookie;
 import javax.servlet.http.HttpServletResponse;
 import javax.servlet.http.HttpServletResponseWrapper;
 
-import org.apache.tomcat.util.res.StringManager;
-
 
 /**
  * Wrapper around a <code>javax.servlet.http.HttpServletResponse</code>
@@ -96,13 +91,6 @@ public ApplicationHttpResponse(HttpServletResponse response,
         "org.apache.catalina.core.ApplicationHttpResponse/1.0";
 
 
-    /**
-     * The string manager for this package.
-     */
-    protected static final StringManager sm =
-        StringManager.getManager(Constants.Package);
-
-
     // ------------------------------------------------ ServletResponse Methods
 
 

diff --git a/java/org/apache/catalina/core/ApplicationRequest.java b/java/org/apache/catalina/core/ApplicationRequest.java
@@ -27,7 +27,6 @@
 import javax.servlet.ServletRequestWrapper;
 
 import org.apache.catalina.util.Enumerator;
-import org.apache.tomcat.util.res.StringManager;
 
 
 /**
@@ -95,13 +94,6 @@ public ApplicationRequest(ServletRequest request) {
         new HashMap<String, Object>();
 
 
-    /**
-     * The string manager for this package.
-     */
-    protected static final StringManager sm =
-        StringManager.getManager(Constants.Package);
-
-
     // ------------------------------------------------- ServletRequest Methods
 
 

diff --git a/java/org/apache/catalina/core/ApplicationResponse.java b/java/org/apache/catalina/core/ApplicationResponse.java
@@ -14,18 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
-
 package org.apache.catalina.core;
 
-
 import java.util.Locale;
 
 import javax.servlet.ServletResponse;
 import javax.servlet.ServletResponseWrapper;
 
-import org.apache.tomcat.util.res.StringManager;
-
 
 /**
  * Wrapper around a <code>javax.servlet.ServletResponse</code>
@@ -86,13 +81,6 @@ public ApplicationResponse(ServletResponse response, boolean included) {
     protected boolean included = false;
 
 
-    /**
-     * The string manager for this package.
-     */
-    protected static final StringManager sm =
-        StringManager.getManager(Constants.Package);
-
-
     // ------------------------------------------------ ServletResponse Methods
 
 

diff --git a/java/org/apache/catalina/core/AsyncContextImpl.java b/java/org/apache/catalina/core/AsyncContextImpl.java
@@ -83,6 +83,7 @@ public void complete() {
         request.getCoyoteRequest().action(ActionCode.ASYNC_COMPLETE, null);
     }
 
+    @Override
     public void fireOnComplete() {
         List<AsyncListenerWrapper> listenersCopy =
             new ArrayList<AsyncListenerWrapper>();
@@ -383,7 +384,7 @@ private void logDebug(String method) {
                 "Req: %1$8s  CReq: %2$8s  RP: %3$8s  Stage: %4$s  " +
                 "Thread: %5$20s  State: %6$20s  Method: %7$11s  URI: %8$s",
                 rHashCode, crHashCode, rpHashCode, stage,
-                Thread.currentThread().getName(), "N/A", method, uri);
+                threadName, "N/A", method, uri);
         if (log.isTraceEnabled()) {
             log.trace(msg, new DebugException());
         } else {

diff --git a/java/org/apache/catalina/core/Constants.java b/java/org/apache/catalina/core/Constants.java
@@ -27,9 +27,4 @@ public class Constants {
 
     public static final String JSP_SERVLET_CLASS =
         "org.apache.jasper.servlet.JspServlet";
-    public static final String JSP_SERVLET_NAME = "jsp";
-    public static final String PRECOMPILE = 
-        System.getProperty("org.apache.jasper.Constants.PRECOMPILE",
-                "jsp_precompile");
-
 }