Jim Jagielski wrote:
> William A. Rowe, Jr. wrote:
>>> The other fatal bug is:
>>>
>>>>      * mpm_winnt: Fix return values from wait_for_many_objects.
>>>>          http://svn.apache.org/viewvc?view=rev&revision=428029
>>>>        2.2.x version of patch:
>>>>          Trunk version works
>>>> +        http://people.apache.org/~wrowe/mpm_winnt_waits.patch
>>>> +        is easier to read (-U8)
>>>>        +1: mturk, wrowe
>>> Which i've created another, easier-to-read example of the backport, if you
>>> didn't follow the change last time please read the new patch and chime in
>>> a third +1?  You'll note it's simply a matter of unsetting errno before we
>>> invoke the wait, so we know if the 64th, 128th, etc are real errors or in
>>> fact indexes into the wait list array.
>>>
>>> That's it.  I'd like to roll tonight so the vote can be closed by the 22nd.
>> I'd like to have was dependent on someone tossing in a vote here /shrug.
> 
> I could vote +1 by inspection, but I can't test...

Of course... had I expected anyone would test, I wouldn't have wasted the time
and effort to generate a legigble diff -U8 - which by the way is somewhat
tricky...

svn diff --diff-cmd=diff --extentions=-U8

are both required since the built-in-diff'er doesn't know -u8 or -U8, and the
diff-cmd is an app path/name, not a command line.

Guess it was worth wasting the time to learn this even if this exercise
doesn't have any net results.

Reply via email to