Re: [PATCH] svndumpfilter's use of %n

On Sat, Feb 24, 2007 at 09:22:59AM -0700, D.J. Heap wrote:
> Are there any objections to changing svndumpfilter's use of sprintf
> with %n over to apr_snprintf with %n?
>

I had to look up %n to find out what it did. It might be better to do
something like the following:

> - sprintf(buf, "%d%n", namelen, &bytes_used);
> - svn_stringbuf_appendbytes(*strbuf, buf, bytes_used);
> + sprintf(buf, "%d", namelen);
> + svn_stringbuf_appendbytes(*strbuf, buf, strlen(buf));
> svn_stringbuf_appendbytes(*strbuf, "\n", 1);
>

Slightly less efficient, but a whole lot clearer.

Regards,
Malcolm