[FFmpeg-devel] [PATCH] lavu/parseutils: accept %h for hours >= 24.

Nicolas George nicolas.george at normalesup.org
Sat Dec 1 17:31:49 CET 2012

Le primidi 11 frimaire, an CCXXI, Michael Niedermayer a écrit :
> I prefer this patch here, our API is by a factor of about 2-3 too
> complex we should reduce the number of non static functions IMHO

I agree on the principle, but I believe you missed the fact that the first
patch does not add a new non-static function. Would it change your mind?

> also the feature is usefull for other users of av_small_strptime


> %h is already used, see:
> man strptime says:
>     %b or %B or %h
>               The month name according to the current locale, in abbreviated form or the full name.

I know, but that conversion is not supported by this implementation; and if
anyone wants to add it, %b is still available. I am afraid that any letter
will have been used by one libc or another.


  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121201/893fe061/attachment.asc>

More information about the ffmpeg-devel mailing list