Mailing-List: contact cvs-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r493292 - /httpd/httpd/branches/2.2.x/STATUS
Date: Sat, 06 Jan 2007 03:53:33 -0000
To: cvs@httpd.apache.org
From: wrowe@apache.org
Message-Id: <20070106035333.DEC901A981A@eris.apache.org>

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.