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

Stefano Sabatini stefasab at gmail.com
Sat Dec 1 18:20:50 CET 2012


On date Saturday 2012-12-01 18:09:33 +0100, Reimar Döffinger encoded:
> Minor nitpick...
> 
> On Sat, Dec 01, 2012 at 05:00:32PM +0100, Nicolas George wrote:
> > + * - %h: the hour as a decimal number, using a 24-hour clock, in the
> > + * range '00' through INT_MAX
> 
> It's no longer hour as in "time of day", so I think the reference to
> "24-hour clock" no longer makes sense, and there's little point in
> writing '00' with two letters.

On the other hand I'm not even sure it does make sense to use strptime
in this way (or you end with all kind of extensions, like supporting
100:99:99:99).

If we want to express durations in a convenient way we should rather
adopt something like 1d10h (1 day and 10 hours). I had a patch for
that, but I need to dig it out.
-- 
FFmpeg = Fostering Foolish Minimal Pitiless Erudite Gymnast


More information about the ffmpeg-devel mailing list