Author: markt Date: Wed Dec 14 11:05:03 2016 New Revision: 1774176 URL: http://svn.apache.org/viewvc?rev=1774176&view=rev Log: Don't duplicate storage of compressableMimeTypes in the Processor
Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java?rev=1774176&r1=1774175&r2=1774176&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java (original) +++ tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Protocol.java Wed Dec 14 11:05:03 2016 @@ -711,7 +711,6 @@ public abstract class AbstractHttp11Prot processor.setCompressionMinSize(getCompressionMinSize()); processor.setCompression(getCompression()); processor.setNoCompressionUserAgents(getNoCompressionUserAgents()); - processor.setCompressableMimeTypes(getCompressableMimeTypes()); return processor; } Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java?rev=1774176&r1=1774175&r2=1774176&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java (original) +++ tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Wed Dec 14 11:05:03 2016 @@ -147,12 +147,6 @@ public class Http11Processor extends Abs /** - * List of MIMES for which compression may be enabled. - */ - protected String[] compressableMimeTypes; - - - /** * Host name (used to avoid useless B2C conversion on the host name). */ protected char[] hostNameC = new char[0]; @@ -265,19 +259,6 @@ public class Http11Processor extends Abs /** - * Set compressible mime-type list (this method is best when used with - * a large number of connectors, where it would be better to have all of - * them referenced a single array). - * - * @param compressableMimeTypes MIME types for which compression should be - * enabled - */ - public void setCompressableMimeTypes(String[] compressableMimeTypes) { - this.compressableMimeTypes = compressableMimeTypes; - } - - - /** * Return compression level. * * @return The current compression level in string form (off/on/force) @@ -338,6 +319,7 @@ public class Http11Processor extends Abs if ((contentLength == -1) || (contentLength > compressionMinSize)) { // Check for compatible MIME-TYPE + String[] compressableMimeTypes = protocol.getCompressableMimeTypes(); if (compressableMimeTypes != null) { return (startsWithStringArray(compressableMimeTypes, response.getContentType())); --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org