[FFmpeg-devel] [PATCH 1/4] lavu/bprint: implement av_bprint_strftime().
nicolas.george at normalesup.org
Sat Nov 17 11:52:10 CET 2012
Le quintidi 25 brumaire, an CCXXI, Stefano Sabatini a écrit :
> > + if (!room)
> > + room = strlen(fmt) / 2 + 1;
> Is this required? Shouldn't the previous assignment guarantee that
> room is > 0?
I just forgot to remove it. Fixed.
> Nit: @note
> missing documentation for fmt (which should mention the function
> strftime()) and tm parameters
I figured it was completely obvious, given the name of the function itself,
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel