Michael W Thelen wrote:
>SteveKing wrote:
>
>
>>This is really a *very* little thing:
>>
>>The CL client uses an array to store all the auth providers. The initial
>>size of that array is set to 10, but on Windows, there's one auth
>>provider more so the array is always reallocated (or I'm counting wrong,
>>could happen too).
>>This very little patch takes care of this.
>>
>>
>
>Pinging this patch... Brane objected, saying that it's not worth the
>bother to change the initial size whenever a new auth provider is added.
> Steve responded that new auth providers are not added often, and the
>initial size already seems to be intended to accommodate all auth
>providers, which it doesn't quite do on Windows.
>
>My (relatively ignorant) opinion is that if we're going to use a magic
>number in the code (10), it might as well be large enough to accommodate
>all auth providers on Windows (11), so the array doesn't need to be
>reallocated. There are 11 calls to APR_ARRAY_PUSH in the code... the
>array may as well be preallocated to 11, right? I don't see any
>downsides to applying this patch. Are there any?
>
>
There are no downsides.
-- Brane
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 18 19:26:38 2005