2012/5/24 Konstantin Kolinko <knst.koli...@gmail.com>:
> 2012/5/24  <ma...@apache.org>:
>> Author: markt
>> Date: Wed May 23 20:07:46 2012
>> New Revision: 1342010
>>
>> URL: http://svn.apache.org/viewvc?rev=1342010&view=rev
>> Log:
>> Renames to make code clearer
>> socket_ttl -> socket_last_active
>> max_ttl -> default_timeout
>> Back port of r1342008
>>
>> Modified:
>>    tomcat/native/branches/1.1.x/   (props changed)
>>    tomcat/native/branches/1.1.x/native/src/poll.c
>>
>> Propchange: tomcat/native/branches/1.1.x/
>> ------------------------------------------------------------------------------
>>  Merged /tomcat/native/trunk:r1342008
>>
>> Modified: tomcat/native/branches/1.1.x/native/src/poll.c
>> URL: 
>> http://svn.apache.org/viewvc/tomcat/native/branches/1.1.x/native/src/poll.c?rev=1342010&r1=1342009&r2=1342010&view=diff
>> ==============================================================================
>> --- tomcat/native/branches/1.1.x/native/src/poll.c (original)
>> +++ tomcat/native/branches/1.1.x/native/src/poll.c Wed May 23 20:07:46 2012
>> @@ -40,8 +40,8 @@ typedef struct tcn_pollset {
>>     apr_pollset_t *pollset;
>>     jlong         *set;
>>     apr_pollfd_t  *socket_set;
>> -    apr_time_t    *socket_ttl;
>> -    apr_interval_time_t max_ttl;
>> +    apr_time_t    *socket_last_active;
>> +    apr_interval_time_t default_timeout;
>>  #ifdef TCN_DO_STATISTICS
>>     int sp_added;
>>     int sp_max_count;
>> @@ -128,12 +128,12 @@ TCN_IMPLEMENT_CALL(jlong, Poll, create)(
>>     TCN_CHECK_ALLOCATED(tps->set);
>>     tps->socket_set = apr_palloc(p, size * sizeof(apr_pollfd_t));
>>     TCN_CHECK_ALLOCATED(tps->socket_set);
>> -    tps->socket_ttl = apr_palloc(p, size * sizeof(apr_interval_time_t));
>> -    TCN_CHECK_ALLOCATED(tps->socket_ttl);
>> +    tps->socket_last_active = apr_palloc(p, size * 
>> sizeof(apr_interval_time_t));
>
> Looking at the declaration of "socket_last_active" field, the above
> should have used sizeof(apr_time_t).
>

I see it is has already been fixed by
http://svn.apache.org/viewvc?view=revision&revision=1342014

Sorry for the noise.

>
>> +    TCN_CHECK_ALLOCATED(tps->socket_last_active);
>>     tps->nelts  = 0;
>>     tps->nalloc = size;
>>     tps->pool   = p;
>> -    tps->max_ttl = J2T(ttl);
>> +    tps->default_timeout = J2T(ttl);
>> (...)

Best regards,
Konstantin Kolinko

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

Reply via email to