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

On Sat, Feb 24, 2007 at 02:24:54PM -0600, Jonathan Gilbert wrote:
> Doesn't sprintf() return the number of bytes it wrote into the buffer anyway?
>
> bytes_used = sprintf(buf, "%d", namelen);
> svn_stringbuf_appendbytes(*strbuf, buf, bytes_used);
>

Yes, it does. I like that. I'm pretty sure the return of sprintf() is
sensible, unlike, say, snprintf() - where some implementations return -1
on truncation.

> ..or if one is feeling daring (might not be wise, since in the case of
> failure, a negative number is returned -- but then, what could go wrong
> with this particular invocation? :-):
>
> svn_stringbuf_appendbytes(*strbuf, buf,
> sprintf(buf, "%d", namelen));
>

That's a little less readable that what I was hoping for :-)

Regards,
Malcolm