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

[Roger Pack]

On 10/10/16, Roger Pack <rogerdpack2 at gmail.com> wrote:
> On 9/22/16, Roger Pack <rogerdpack2 at gmail.com> wrote:
>> On 1/4/12, Yuval Adam <yuv.adm at gmail.com> wrote:
>>> From: Yuval Adam <yuv.adm at gmail.com>
>>>
>>> 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).
>>
>> A somewhat updated (but not yet cleaned up) revision:
>>
>> https://gist.github.com/rdp/e518616f2a702367ae5a922b56e09e04
>>
>> see also https://trac.ffmpeg.org/ticket/1452
>
> OK attached is the "cleaned up" patch, ready for review/commit.
>
> how to test:
> (apply then) run this:
>
> ./ffmpeg -i input -copyts -vsync vfr temp/abc-%d-%t.jpeg
> and compare filenames with the timestamps from video packets of
> ffprobe -show_packets.
>
> Probably a better way would have been to mix it into
> av_bprint_strftime however I wasn't sure how to use that within
> libavformat/utils.c av_get_frame_filename2
>
> Adam's initial patch
> (https://github.com/yuvadm/FFmpeg/commit/0eb002821a2076cb3593c823399aeef9fdd29525)
> also deprecated av_get_frame_filename
>
> but I wasn't sure if we wanted that here or not so didn't include it.
> Thank you for your consideration.

bump (this last one is cleaned up enough to be considered for merge please...).
-roger-