On 09/09/2007 07:24 PM, Jim Jagielski wrote: > > On Sep 7, 2007, at 6:20 PM, Ruediger Pluem wrote: > >> >> >> On 09/07/2007 03:03 PM, Jim Jagielski wrote: >>> >>> On Sep 7, 2007, at 6:29 AM, jean-frederic clere wrote: >>> >>>> >>>> I think I have patched it. Could you try it? >>>> >>> >>> Ahh... this is much nicer that my idea of breaking out >>> the data struct defs from the rest of mod_proxy.h >> >> I do not share this view and think that we should do this >> differently (haven't worked out a patch so far). >> >> 1. IMHO we change the public API with jean-frederic's patch >> (we remove ap_proxy_lb_workers from mod_proxy.h so a major >> bump would be due IMHO). >> > > FWIW, I don't agree with that. ap_proxy_lb_workers() was > not and is not part of the public API for mod_proxy. > It was never declared as such. From what I can see, it
Just out of curiosity: Why? Because it was int ap_proxy_lb_workers(void); and not PROXY_DECLARE(int)ap_proxy_lb_workers(void); ? Regards Rüdiger
