[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Choice of constant in libsvn_fs_base

From: Branko ─îibej <brane_at_xbc.nu>
Date: Sun, 27 Nov 2011 20:10:20 +0100

On 25.11.2011 16:24, Julian Foad wrote:
> Daniel Shahaf wrote:
>> While reviewing r1205726 I came across this pearl:
>> reps-strings.c- /* Make a list of all the rep's we need to undeltify this range.
>> reps-strings.c- We'll have to read them within this trail anyway, so we might
>> reps-strings.c- as well do it once and up front. */
>> reps-strings.c- apr_array_header_t *reps = /* ### what constant here? */
>> reps-strings.c: apr_array_make(pool, 666, sizeof(rep));
>> Isn't that constant a little too large? Will any existing repository
>> ever need a chain of deltas that contains >600 elements? Can we set the
>> constant to 30?
> +1.

Oh come now. Leave a bit of fun in the code. You're going to save all of
10k memory by making that change.

-- Brane
Received on 2011-11-27 20:11:01 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.