[FFmpeg-devel] [PATCH] img2: added support for %t output pattern

[Yuval Adam]

Thanks to everyone for the constructive review, and for bearing with me as
this is my first patch for FFmpeg.
I absolutely agree with all of the comments, and indeed failed to notice
I'm changing a public API.
I'll revise the patch to address the issues you raised, and post it shortly.

Thanks again!

2011/12/19 Víctor Paesa <victorpaesa at googlemail.com>

> Hi,
>
> On Mon, Dec 19, 2011 at 10:21, Yuval Adam wrote:
> > Also reviewable on github:
> >
> https://github.com/yuvadm/FFmpeg/commit/3e94df886e63e0c185e5a244fceb0e0a37bf4800
> >
> > From 3e94df886e63e0c185e5a244fceb0e0a37bf4800 Mon Sep 17 00:00:00 2001
> > From: Yuval Adam <yuv.adm at gmail.com>
> > Date: Mon, 19 Dec 2011 10:46:02 +0200
> > Subject: [PATCH] img2: added support for %t output pattern
> >
> > The image2 muxer now supports timestamps in output filenames.
> > When used in an output patterm '%t' will be replaced with the frames
> > timestamp in hours, minutes and seconds (hh:mm:ss).
>
> Some platforms (MS-DOS/Windows) do not admit colons in file names.
> Maybe "hh.mm.ss" instead of "hh:mm:ss"
> A nice bonus with be to show the frame number within the second,
> "hh.mm.ss.nnn"
>
> Regards,
> Víctor
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>



-- 
Yuval Adam
http://y3xz.com