svn commit: r599327 - /httpd/httpd/branches/2.2.x/STATUS

[rp...@apache.org]

Author: rpluem
Date: Thu Nov 29 00:19:15 2007
New Revision: 599327

URL: http://svn.apache.org/viewvc?rev=599327&view=rev
Log:
* Add comments, remove comments, vote

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?rev=599327&r1=599326&r2=599327&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Thu Nov 29 00:19:15 2007
@@ -195,11 +195,9 @@
    * http_filters: Fix handling of unrecognised Transfer Encodings
      PR 43882
      http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/http_filters.c?r1=592951&r2=599137
-     +1: niq
-     rpluem says: 1. I assume you are +1 to your own proposal, aren't you? :-)
-                  2. Please add a CHANGES entry since this fixes a PR.
-     niq says: modified in 599059 (following suggestion by trawick) since
-     rpluem's +1; resetting your vote. (2) There's a CHANGES entry in r595672.
+     http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?r1=595672&r2=595671&pathrev=595672 (CHANGES)
+     +1: niq, rpluem
+     niq says: modified in 599059 (following suggestion by trawick)
 
     * mod_proxy: Support variable interpolation in reverse proxy configuration
       (revised proposal dealing with wrowe's concerns).
@@ -241,6 +239,9 @@
      2.2.x:
         http://people.apache.org/~jim/patches/bsel-patch.txt
      +1: jim
+     rpluem says: I don't get the reason for the include of <unistd.h>. There
+     is no call to getpid anywhere in mod_proxy_balancer (it was removed on
+     trunk in r394065). If you remove the include from your proposal I am +1.
 
 PATCHES/ISSUES THAT ARE STALLED