Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Received-SPF: pass (athena.apache.org: local policy)
Message-ID: <4925777E.80002@hanik.com>
Date: Thu, 20 Nov 2008 07:43:10 -0700
From: Filip Hanik - Dev Lists <devlists@hanik.com>
User-Agent: Thunderbird 2.0.0.16 (X11/20080723)
MIME-Version: 1.0
To: Tomcat Developers List <dev@tomcat.apache.org>
Subject: Re: svn commit: r719093 -
 /tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java
References: <20081119221457.F0AE02388975@eris.apache.org>
 <49254FCB.1030200@joedog.org>
In-Reply-To: <49254FCB.1030200@joedog.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

http://tomcat.apache.org/tomcat-6.0-doc/config/http.html

look for tcpNoDelay

there has been a default for years

Filip

Tim Funk wrote:
> This doesn't make sense to me. The Javadoc and the code check for null 
> to allow the use of the default value in the JVM. This patch makes all 
> of that invalid and sets it to true.
>
> So either
> 1) this would need reverted -or-
> 2) Or the if (tcpNoDelay != null) needs removed - or -
> 3) The comments get updated (not sure if there associated docs too)
>
> But by defaulting the value to Boolean.TRUE - it does eliminate the 
> chance of a NPE here:
>     public boolean getTcpNoDelay() {
>         return tcpNoDelay.booleanValue();
>     }
>
> I'm indifferent on the fix. (Just the code comment VS code inconsistency)
>
>
> -Tim
>
> fhanik@apache.org wrote:
>> Author: fhanik
>> Date: Wed Nov 19 14:14:57 2008
>> New Revision: 719093
>>
>> URL: http://svn.apache.org/viewvc?rev=719093&view=rev
>> Log:
>> keep the correct default value
>>
>> Modified:
>>     tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java
>>
>> Modified: 
>> tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java
>> URL: 
>> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java?rev=719093&r1=719092&r2=719093&view=diff 
>>
>> ============================================================================== 
>>
>> --- 
>> tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java 
>> (original)
>> +++ 
>> tomcat/trunk/java/org/apache/tomcat/util/net/SocketProperties.java 
>> Wed Nov 19 14:14:57 2008
>> @@ -104,7 +104,7 @@
>>      /**
>>       * TCP_NO_DELAY option. JVM default used if not set.
>>       */
>> -    protected Boolean tcpNoDelay = null;
>> +    protected Boolean tcpNoDelay = Boolean.TRUE;
>>  
>>      /**
>>       * SO_KEEPALIVE option. JVM default used if not set.
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: dev-help@tomcat.apache.org
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org