Le 04/06/2014 09:45, Plüm, Rüdiger, Vodafone Group a écrit :
-----Original Message-----
From: Christophe JAILLET [mailto:[email protected]]
Sent: Mittwoch, 4. Juni 2014 05:40
To: [email protected]
Subject: Re: svn commit: r1572905 - /httpd/httpd/trunk/server/connection.c
Hi,
while looking for backport candidates to synch 2.4.x and trunk, I came
accross this one.
I don't see where it "avoids a double apr_time_now() call on the first
succeeding read".
Because of the 'continue' after the 'timeup' computation, I think that
the previous code already had only one apr_time_now call on the first
succeeding read.
But the continue causes the while loop condition to be checked which had an
apr_time_now in the previous code.
So we have two calls to apr_time_now in this case.
Regards
Rüdiger
Arghh, I misinterpreted this continue. You are obviously right.
I'll queue it to my next proposal of easy patches to synch 2.4.x with trunk.
Thanks.
CJ
