You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by mt...@apache.org on 2008/05/29 20:08:21 UTC

svn commit: r661428 - /tomcat/connectors/trunk/jni/native/src/poll.c

Author: mturk
Date: Thu May 29 11:08:20 2008
New Revision: 661428

URL: http://svn.apache.org/viewvc?rev=661428&view=rev
Log:
Move the ttl as well when the socket is removed. Patch provided by Alex Barclay

Modified:
    tomcat/connectors/trunk/jni/native/src/poll.c

Modified: tomcat/connectors/trunk/jni/native/src/poll.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jni/native/src/poll.c?rev=661428&r1=661427&r2=661428&view=diff
==============================================================================
--- tomcat/connectors/trunk/jni/native/src/poll.c (original)
+++ tomcat/connectors/trunk/jni/native/src/poll.c Thu May 29 11:08:20 2008
@@ -215,6 +215,7 @@
                 }
                 else {
                     p->socket_set[dst] = p->socket_set[i];
+                    p->socket_ttl[dst] = p->socket_ttl[i];
                     dst++;
                 }
             }



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


Re: svn commit: r661428 - /tomcat/connectors/trunk/jni/native/src/poll.c

Posted by Mladen Turk <mt...@apache.org>.
jean-frederic clere wrote:
> mturk@apache.org wrote:
>> Author: mturk
>> Date: Thu May 29 11:08:20 2008
>> New Revision: 661428
>>
>> URL: http://svn.apache.org/viewvc?rev=661428&view=rev
>> Log:
>> Move the ttl as well when the socket is removed. Patch provided by 
>> Alex Barclay
> 
> Cool but could you put the information in 
> jk/xdocs/miscellaneous/changelog.xml?

Why would I put that in mod_jk changelog?
There is already a note in jni/CHANGELOG.txt for this BZ

> BTW: Is there a bug report corresponding to the fix?
> 

Yes. 45071

Regards
-- 
^(TM)

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


Re: svn commit: r661428 - /tomcat/connectors/trunk/jni/native/src/poll.c

Posted by jean-frederic clere <jf...@gmail.com>.
mturk@apache.org wrote:
> Author: mturk
> Date: Thu May 29 11:08:20 2008
> New Revision: 661428
> 
> URL: http://svn.apache.org/viewvc?rev=661428&view=rev
> Log:
> Move the ttl as well when the socket is removed. Patch provided by Alex Barclay

Cool but could you put the information in 
jk/xdocs/miscellaneous/changelog.xml?
BTW: Is there a bug report corresponding to the fix?

Cheers

Jean-Frederic

> 
> Modified:
>     tomcat/connectors/trunk/jni/native/src/poll.c
> 
> Modified: tomcat/connectors/trunk/jni/native/src/poll.c
> URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jni/native/src/poll.c?rev=661428&r1=661427&r2=661428&view=diff
> ==============================================================================
> --- tomcat/connectors/trunk/jni/native/src/poll.c (original)
> +++ tomcat/connectors/trunk/jni/native/src/poll.c Thu May 29 11:08:20 2008
> @@ -215,6 +215,7 @@
>                  }
>                  else {
>                      p->socket_set[dst] = p->socket_set[i];
> +                    p->socket_ttl[dst] = p->socket_ttl[i];
>                      dst++;
>                  }
>              }
> 
> 
> 
> ---------------------------------------------------------------------
> 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