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

Re: svn commit: r1634875 - /subversion/trunk/subversion/libsvn_fs_fs/revprops.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Wed, 29 Oct 2014 13:35:32 +0100

On 29.10.2014 12:21, Stefan Fuhrmann wrote:
>
>
> On Wed, Oct 29, 2014 at 9:44 AM, Branko Čibej <brane_at_wandisco.com
> <mailto:brane_at_wandisco.com>> wrote:
>
> On 28.10.2014 14:19, stefan2_at_apache.org
> <mailto:stefan2_at_apache.org> wrote:
> > Author: stefan2
> > Date: Tue Oct 28 13:19:30 2014
> > New Revision: 1634875
> >
> > URL: http://svn.apache.org/r1634875
> > Log:
> > Speed up packed revprop access by tuning the manifest file parser.
>
> [...]
>
> > +/* Return the minimum length of any packed revprop file name in
> REVPROPS. */
> > +static apr_size_t
> > +get_min_filename_len(packed_revprops_t *revprops)
> > +{
> > + char number_buffer[SVN_INT64_BUFFER_SIZE];
> > +
> > + /* The revprop filenames have the format <REV>.<COUNT> - with
> <REV> being
> > + * at least the first rev in the shard and <COUNT> having at
> least one
> > + * digit. Thus, the minimum is 2 + #decimal places in the
> start rev.
> > + */
> > + return svn__i64toa(number_buffer, revprops->manifest_start) + 2;
> > +}
>
> Are you absolutely sure this is correct? According to the comment, you
> should be returning
>
> strlen(svn_i64toa(...)) + 2
>
>
> svn_i64toa returns the number of non-NUL chars written into the
> buffer provided by the caller.

Yeah ... for some reason I was thinking it returned the address of the
buffer. Sorry for the noise.

-- Brane
Received on 2014-10-29 13:37:35 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.