Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
Received-SPF: pass (nike.apache.org: local policy)
Mime-Version: 1.0 (Apple Message framework v752.2)
In-Reply-To: <46E1CE97.8020400@apache.org>
References: <20070906134647.AB2C81A9832@eris.apache.org>
 <46E10C15.9080608@gmail.com> <20070907093714.GC14044@deejai2.mch.fsc.net>
 <46E127F1.2050203@gmail.com>
 <00363420-127B-4046-99BA-D9B4CC07C964@jaguNET.com>
 <46E1CE97.8020400@apache.org>
Content-Type: text/plain; charset=US-ASCII; format=flowed
Message-Id: <7AC40721-1751-43F1-8720-1C50DB114ECA@jaguNET.com>
Content-Transfer-Encoding: 7bit
From: Jim Jagielski <jim@jaguNET.com>
Subject: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h
Date: Sun, 9 Sep 2007 13:24:07 -0400
To: dev@httpd.apache.org


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
was added in mod_proxy.h simply because the original
patch wanted it defined in proxy_util.c but used in
mod_proxy.c and so added it to mod_proxy.h.

A case could be made for extern int PROXY_DECLARE_DATA proxy_lb_workers
which, upon 2nd and 3rd glance, should be re-added
and can be by simply renaming the confusingly-name
proxy_lb_workers pointer in scoreboard.c. I will do this
later on today.