httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From wr...@apache.org
Subject svn commit: r493292 - /httpd/httpd/branches/2.2.x/STATUS
Date Sat, 06 Jan 2007 03:53:33 GMT
Author: wrowe
Date: Fri Jan  5 19:53:33 2007
New Revision: 493292

URL: http://svn.apache.org/viewvc?view=rev&rev=493292
Log:
Each review of this code, something else looks wrong

Modified:
    httpd/httpd/branches/2.2.x/STATUS

Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?view=diff&rev=493292&r1=493291&r2=493292
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Fri Jan  5 19:53:33 2007
@@ -91,11 +91,10 @@
         http://people.apache.org/~wrowe/mpm_winnt_waits.patch
         is easier to read (-U8)
       +1: mturk
-      wrowe notes: WAIT_ABANDONED isn't the actual signal, it should
-        simply loop immediate.  This aspect needs to be revisited,
-        as Roy notes, a patch should have the necessary effect with
-        the minimum lines of code - there's alot of redecorating that's
-        going on in this patch to no net effect.
+      wrowe notes: a patch should have the necessary effect with the
+        minimum lines of code - there's alot of redecorating that's
+        going on in this patch to no net effect.  The WAIT_TIMEOUT
+        result value seems to be ignored in the revised code?
 
     * Bundled PCRE: backport r381783 from trunk
       Fix brokenness on certain platforms when building with -DDEBUG.



Mime
View raw message