[FFmpeg-devel] [RFC] SDP Generation

[Luca Abeni]

Hi,

Michael Niedermayer wrote:
[...]
>> I think this strlcatf() function can be useful for others too, and not 
>> only for the SDP generator. Should I submit a patch adding av_strlcatf() 
>> to libavutil/string.c?
> 
> yes

Ok, here it is... I hope I got it right.
The attached patch implements the semantic I proposed (the returned 
value is equal to the total length of the resulting string).
Maybe for consistency with av_strlcat() it could be more interesting to 
return the length of the string that would have been generated if enough 
space had been available. In such case, the implementation would have been

size_t av_strlcatf(char *dst, size_t size, const char *fmt, ...)
{
     int len = strlen(dst);
     va_list vl;
     int res;

     va_start(vl, fmt);
     res = vsnprintf(dst + len, size > len ? size - len : 0, fmt, vl);
     va_end(vl);

     return len + res;
}

Let me know which variant is preferred, and if it is ok for commit.


				Thanks,
					Luca
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: strlcatf.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070718/31152667/attachment.txt>