Remove unused code
authormarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Wed, 5 Jan 2011 07:50:11 +0000 (07:50 +0000)
committermarkt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Wed, 5 Jan 2011 07:50:11 +0000 (07:50 +0000)
git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1055341 13f79535-47bb-0310-9956-ffa450edef68

java/org/apache/coyote/http11/AbstractHttp11Processor.java
java/org/apache/coyote/http11/AbstractInputBuffer.java
java/org/apache/coyote/http11/AbstractOutputBuffer.java
java/org/apache/coyote/http11/Constants.java
java/org/apache/coyote/http11/InternalAprOutputBuffer.java
java/org/apache/coyote/http11/InternalInputBuffer.java
java/org/apache/coyote/http11/InternalNioOutputBuffer.java
java/org/apache/coyote/http11/InternalOutputBuffer.java
java/org/apache/coyote/http11/filters/VoidOutputFilter.java

index 190de24..5b7d3bf 100644 (file)
@@ -61,9 +61,6 @@ public abstract class AbstractHttp11Processor implements ActionHook, Processor {
     protected static final StringManager sm =
         StringManager.getManager(Constants.Package);
 
-    protected static boolean isSecurityEnabled = 
-        org.apache.coyote.Constants.IS_SECURITY_ENABLED;
-
     /*
      * Tracks how many internal filters are in the filter library so they
      * are skipped when looking for pluggable filters. 
@@ -360,14 +357,6 @@ public abstract class AbstractHttp11Processor implements ActionHook, Processor {
 
 
     /**
-     * Return the list of compressable mime-types.
-     */
-    public String[] findCompressableMimeTypes() {
-        return (compressableMimeTypes);
-    }
-
-
-    /**
      * Return compression level.
      */
     public String getCompression() {
@@ -490,19 +479,6 @@ public abstract class AbstractHttp11Processor implements ActionHook, Processor {
 
 
     /**
-     * Return the list of restricted user agents.
-     */
-    public String[] findRestrictedUserAgents() {
-        String[] sarr = new String [restrictedUserAgents.length];
-
-        for (int i = 0; i < restrictedUserAgents.length; i++)
-            sarr[i] = restrictedUserAgents[i].toString();
-
-        return (sarr);
-    }
-
-
-    /**
      * Set the maximum number of Keep-Alive requests to honor.
      * This is to safeguard from DoS attacks.  Setting to a negative
      * value disables the check.
@@ -796,30 +772,6 @@ public abstract class AbstractHttp11Processor implements ActionHook, Processor {
 
     
     /**
-     * Add input or output filter.
-     *
-     * @param className class name of the filter
-     */
-    protected void addFilter(String className) {
-        try {
-            Class<?> clazz = Class.forName(className);
-            Object obj = clazz.newInstance();
-            if (obj instanceof InputFilter) {
-                getInputBuffer().addFilter((InputFilter) obj);
-            } else if (obj instanceof OutputFilter) {
-                getOutputBuffer().addFilter((OutputFilter) obj);
-            } else {
-                getLog().warn(sm.getString("http11processor.filter.unknown",
-                        className));
-            }
-        } catch (Exception e) {
-            getLog().error(sm.getString(
-                    "http11processor.filter.error", className), e);
-        }
-    }
-
-
-    /**
      * Add an input filter to the current request.
      *
      * @return false if the encoding was not found (which would mean it is
index c2ac664..aab5a4d 100644 (file)
@@ -188,17 +188,6 @@ public abstract class AbstractInputBuffer implements InputBuffer{
 
 
     /**
-     * Clear filters.
-     */
-    public void clearFilters() {
-
-        filterLibrary = new InputFilter[0];
-        lastActiveFilter = -1;
-
-    }
-
-
-    /**
      * Add an input filter to the filter library.
      */
     public void addActiveFilter(InputFilter filter) {
index d761128..e5bef9d 100644 (file)
@@ -28,7 +28,6 @@ import org.apache.tomcat.util.buf.ByteChunk;
 import org.apache.tomcat.util.buf.CharChunk;
 import org.apache.tomcat.util.buf.MessageBytes;
 import org.apache.tomcat.util.http.HttpMessages;
-import org.apache.tomcat.util.http.MimeHeaders;
 import org.apache.tomcat.util.res.StringManager;
 
 public abstract class AbstractOutputBuffer implements OutputBuffer{
@@ -43,12 +42,6 @@ public abstract class AbstractOutputBuffer implements OutputBuffer{
 
 
     /**
-     * Headers of the associated request.
-     */
-    protected MimeHeaders headers;
-
-
-    /**
      * Committed flag.
      */
     protected boolean committed;
@@ -142,17 +135,6 @@ public abstract class AbstractOutputBuffer implements OutputBuffer{
 
 
     /**
-     * Clear filters.
-     */
-    public void clearFilters() {
-
-        filterLibrary = new OutputFilter[0];
-        lastActiveFilter = -1;
-
-    }
-
-
-    /**
      * Add an output filter to the filter library.
      */
     public void addActiveFilter(OutputFilter filter) {
@@ -403,42 +385,6 @@ public abstract class AbstractOutputBuffer implements OutputBuffer{
 
 
     /**
-     * Send a header.
-     * 
-     * @param name Header name
-     * @param value Header value
-     */
-    public void sendHeader(ByteChunk name, ByteChunk value) {
-
-        write(name);
-        buf[pos++] = Constants.COLON;
-        buf[pos++] = Constants.SP;
-        write(value);
-        buf[pos++] = Constants.CR;
-        buf[pos++] = Constants.LF;
-
-    }
-
-
-    /**
-     * Send a header.
-     * 
-     * @param name Header name
-     * @param value Header value
-     */
-    public void sendHeader(String name, String value) {
-
-        write(name);
-        buf[pos++] = Constants.COLON;
-        buf[pos++] = Constants.SP;
-        write(value);
-        buf[pos++] = Constants.CR;
-        buf[pos++] = Constants.LF;
-
-    }
-
-
-    /**
      * End the header block.
      */
     public void endHeaders() {
index 46514b4..0791c4d 100644 (file)
@@ -38,8 +38,6 @@ public final class Constants {
 
     public static final int DEFAULT_CONNECTION_LINGER = -1;
     public static final int DEFAULT_CONNECTION_TIMEOUT = 60000;
-    public static final int DEFAULT_CONNECTION_UPLOAD_TIMEOUT = 300000;
-    public static final int DEFAULT_SERVER_SOCKET_TIMEOUT = 0;
     public static final boolean DEFAULT_TCP_NO_DELAY = true;
     
     
@@ -129,8 +127,6 @@ public final class Constants {
 
 
     /* Various constant "strings" */
-    public static final byte[] CRLF_BYTES = ByteChunk.convertToBytes(CRLF);
-    public static final byte[] COLON_BYTES = ByteChunk.convertToBytes(": ");
     public static final String CONNECTION = "Connection";
     public static final String CLOSE = "close";
     public static final byte[] CLOSE_BYTES = 
index 84ebbe3..a2918b7 100644 (file)
@@ -35,27 +35,14 @@ import org.apache.tomcat.util.http.HttpMessages;
 public class InternalAprOutputBuffer extends AbstractOutputBuffer {
 
 
-    // -------------------------------------------------------------- Constants
-
-
     // ----------------------------------------------------------- Constructors
 
-
     /**
      * Default constructor.
      */
-    public InternalAprOutputBuffer(Response response) {
-        this(response, Constants.DEFAULT_HTTP_HEADER_BUFFER_SIZE);
-    }
-
-
-    /**
-     * Alternate constructor.
-     */
     public InternalAprOutputBuffer(Response response, int headerBufferSize) {
 
         this.response = response;
-        headers = response.getMimeHeaders();
 
         buf = new byte[headerBufferSize];
         if (headerBufferSize < (8 * 1024)) {
index 4ded3b1..101a96c 100644 (file)
@@ -37,14 +37,6 @@ public class InternalInputBuffer extends AbstractInputBuffer {
     /**
      * Default constructor.
      */
-    public InternalInputBuffer(Request request) {
-        this(request, Constants.DEFAULT_HTTP_HEADER_BUFFER_SIZE);
-    }
-
-
-    /**
-     * Alternate constructor.
-     */
     public InternalInputBuffer(Request request, int headerBufferSize) {
 
         this.request = request;
index 4468b0d..8f2bea2 100644 (file)
@@ -39,26 +39,14 @@ import org.apache.tomcat.util.net.NioSelectorPool;
  */
 public class InternalNioOutputBuffer extends AbstractOutputBuffer {
 
-
-    // -------------------------------------------------------------- Constants
-
-
     // ----------------------------------------------------------- Constructors
-    /**
-     * Default constructor.
-     */
-    public InternalNioOutputBuffer(Response response) {
-        this(response, Constants.DEFAULT_HTTP_HEADER_BUFFER_SIZE);
-    }
-
 
     /**
-     * Alternate constructor.
+     * Default constructor.
      */
     public InternalNioOutputBuffer(Response response, int headerBufferSize) {
 
         this.response = response;
-        headers = response.getMimeHeaders();
 
         buf = new byte[headerBufferSize];
         
index 480fab9..6780583 100644 (file)
@@ -32,28 +32,14 @@ import org.apache.tomcat.util.buf.ByteChunk;
 public class InternalOutputBuffer extends AbstractOutputBuffer
     implements ByteChunk.ByteOutputChannel {
 
-    // -------------------------------------------------------------- Constants
-
-
     // ----------------------------------------------------------- Constructors
 
-
     /**
      * Default constructor.
      */
-    public InternalOutputBuffer(Response response) {
-        this(response, Constants.DEFAULT_HTTP_HEADER_BUFFER_SIZE);
-    }
-
-
-    /**
-     * Alternate constructor.
-     */
     public InternalOutputBuffer(Response response, int headerBufferSize) {
 
         this.response = response;
-        
-        headers = response.getMimeHeaders();
 
         buf = new byte[headerBufferSize];
 
index f37cdf2..44e14f6 100644 (file)
@@ -48,15 +48,6 @@ public class VoidOutputFilter implements OutputFilter {
     }
 
 
-    // ----------------------------------------------------- Instance Variables
-
-
-    /**
-     * Next buffer in the pipeline.
-     */
-    protected OutputBuffer buffer;
-
-
     // --------------------------------------------------- OutputBuffer Methods
 
 
@@ -93,7 +84,7 @@ public class VoidOutputFilter implements OutputFilter {
      */
     @Override
     public void setBuffer(OutputBuffer buffer) {
-        this.buffer = buffer;
+        // NO-OP
     }