You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hc.apache.org by og...@apache.org on 2006/11/26 20:33:43 UTC

svn commit: r479414 - /jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java

Author: oglueck
Date: Sun Nov 26 11:33:42 2006
New Revision: 479414

URL: http://svn.apache.org/viewvc?view=rev&rev=479414
Log:
checking for 0 length is faster than comparing with empty string

Modified:
    jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java

Modified: jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java?view=diff&rev=479414&r1=479413&r2=479414
==============================================================================
--- jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java (original)
+++ jakarta/httpcomponents/tcconnector/trunk/src/java/org/apache/http/tcconnector/impl/CoyoteHttpConnectionImpl.java Sun Nov 26 11:33:42 2006
@@ -188,7 +188,7 @@
                 HeaderElement[] encodings = transferEncodingHeader.getElements();
                 for (i = 0; i < encodings.length; i++) {
                     String encoding = encodings[i].getName();
-                    if (encoding != null && !encoding.equals("") 
+                    if (encoding != null && encoding.length() > 0 
                         && !encoding.equalsIgnoreCase(HTTP.CHUNK_CODING)
                         && !encoding.equalsIgnoreCase(HTTP.IDENTITY_CODING)) {
                         throw new ProtocolException("Unsupported transfer encoding: " + encoding);