On Sep 10, 2007, at 10:42 AM, Plüm, Rüdiger, VF-Group wrote:



-----Ursprüngliche Nachricht-----
Von: Jim Jagielski
Gesendet: Montag, 10. September 2007 16:06
An: [email protected]
Betreff: Re: AW: mod_proxy_balancer



Regarding the 2nd one... the:

+            else {
+                worker_is_initialized = 0;
+            }

seems redundant... But other than that nit, looks good!

Point taken. Thanks for catching it.
My approach would be to commit the 2nd patch first, get it backported
and improve things on trunk with the first patch afterwards.
Another approach would be to commit the first patch to trunk and propose
the second for backport. I am not sure about the best approach here.


The 1st sounds better to me... And it's not *that* ugly :) :)

Reply via email to