[FFmpeg-cvslog] img2 encoder: allow %t in filename, based on patch from Yuval Adam

rogerdpack git at videolan.org
Tue Nov 1 23:31:58 EET 2016


ffmpeg | branch: master | rogerdpack <rogerpack2005 at gmail.com> | Tue Oct 25 18:33:12 2016 -0600| [1a956c64c8eff5edecb004fc7aafd21207e6485c] | committer: Michael Niedermayer

img2 encoder: allow %t in filename, based on patch from Yuval Adam

Signed-off-by: rogerdpack <rogerpack2005 at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1a956c64c8eff5edecb004fc7aafd21207e6485c
---

 doc/muxers.texi        | 13 +++++++++++++
 libavformat/avformat.h |  3 ++-
 libavformat/hlsenc.c   |  6 +++---
 libavformat/img2enc.c  |  6 ++++--
 libavformat/utils.c    | 42 ++++++++++++++++++++++++++++++++++++++----
 5 files changed, 60 insertions(+), 10 deletions(-)

diff --git a/doc/muxers.texi b/doc/muxers.texi
index 0d856db..0c3a198 100644
--- a/doc/muxers.texi
+++ b/doc/muxers.texi
@@ -619,6 +619,12 @@ If the pattern contains "%d" or "%0 at var{N}d", the first filename of
 the file list specified will contain the number 1, all the following
 numbers will be sequential.
 
+If the pattern contains "%t", the frame's timestamps will be inserted
+in the filename like "00.00.00.000" for hours, minutes, seconds,
+and milliseconds.
+
+The "%t" and "%d" patterns may be used simultaneously.
+
 The pattern may contain a suffix which is used to automatically
 determine the format of the image files to write.
 
@@ -664,6 +670,13 @@ can be used:
 ffmpeg -f v4l2 -r 1 -i /dev/video0 -f image2 -strftime 1 "%Y-%m-%d_%H-%M-%S.jpg"
 @end example
 
+The following example uses the timestamp parameter to generate one
+image file per video frame from the input, and name it including its original
+timestamp.
+ at example
+ffmpeg -i in.avi -vsync vfr -copyts img-%t.jpg
+ at end example
+
 @subsection Options
 
 @table @option
diff --git a/libavformat/avformat.h b/libavformat/avformat.h
index f9f4d72..7f39698 100644
--- a/libavformat/avformat.h
+++ b/libavformat/avformat.h
@@ -2780,10 +2780,11 @@ void av_dump_format(AVFormatContext *ic,
  * @param path numbered sequence string
  * @param number frame number
  * @param flags AV_FRAME_FILENAME_FLAGS_*
+ * @param ts frame timestamp in AV_TIME_BASE fractional seconds.
  * @return 0 if OK, -1 on format error
  */
 int av_get_frame_filename2(char *buf, int buf_size,
-                          const char *path, int number, int flags);
+                          const char *path, int number, int flags, int64_t ts);
 
 int av_get_frame_filename(char *buf, int buf_size,
                           const char *path, int number);
diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index 9ca2df7..02d8d44 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -654,7 +654,7 @@ static int hls_start(AVFormatContext *s)
     } else if (c->max_seg_size > 0) {
         if (av_get_frame_filename2(oc->filename, sizeof(oc->filename),
             c->basename, c->wrap ? c->sequence % c->wrap : c->sequence,
-            AV_FRAME_FILENAME_FLAGS_MULTIPLE) < 0) {
+            AV_FRAME_FILENAME_FLAGS_MULTIPLE, 0) < 0) {
                 av_log(oc, AV_LOG_ERROR, "Invalid segment filename template '%s', you can try to use -use_localtime 1 with it\n", c->basename);
                 return AVERROR(EINVAL);
         }
@@ -685,14 +685,14 @@ static int hls_start(AVFormatContext *s)
             }
         } else if (av_get_frame_filename2(oc->filename, sizeof(oc->filename),
                                   c->basename, c->wrap ? c->sequence % c->wrap : c->sequence,
-                                  AV_FRAME_FILENAME_FLAGS_MULTIPLE) < 0) {
+                                  AV_FRAME_FILENAME_FLAGS_MULTIPLE, 0) < 0) {
             av_log(oc, AV_LOG_ERROR, "Invalid segment filename template '%s' you can try to use -use_localtime 1 with it\n", c->basename);
             return AVERROR(EINVAL);
         }
         if( c->vtt_basename) {
             if (av_get_frame_filename2(vtt_oc->filename, sizeof(vtt_oc->filename),
                               c->vtt_basename, c->wrap ? c->sequence % c->wrap : c->sequence,
-                              AV_FRAME_FILENAME_FLAGS_MULTIPLE) < 0) {
+                              AV_FRAME_FILENAME_FLAGS_MULTIPLE, 0) < 0) {
                 av_log(vtt_oc, AV_LOG_ERROR, "Invalid segment filename template '%s'\n", c->vtt_basename);
                 return AVERROR(EINVAL);
             }
diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c
index 1297b1a..69dbebe 100644
--- a/libavformat/img2enc.c
+++ b/libavformat/img2enc.c
@@ -80,10 +80,12 @@ static int write_packet(AVFormatContext *s, AVPacket *pkt)
     VideoMuxData *img = s->priv_data;
     AVIOContext *pb[4];
     char filename[1024];
-    AVCodecParameters *par = s->streams[pkt->stream_index]->codecpar;
+    AVStream *stream = s->streams[ pkt->stream_index ];
+    AVCodecParameters *par = stream->codecpar;
     const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(par->format);
     int i;
     int nb_renames = 0;
+    int64_t ts = av_rescale_q(pkt->pts, stream->time_base, AV_TIME_BASE_Q);
 
     if (!img->is_pipe) {
         if (img->update) {
@@ -99,7 +101,7 @@ static int write_packet(AVFormatContext *s, AVPacket *pkt)
             }
         } else if (av_get_frame_filename2(filename, sizeof(filename), img->path,
                                           img->img_number,
-                                          AV_FRAME_FILENAME_FLAGS_MULTIPLE) < 0 &&
+                                          AV_FRAME_FILENAME_FLAGS_MULTIPLE, ts) < 0 &&
                    img->img_number > 1) {
             av_log(s, AV_LOG_ERROR,
                    "Could not get frame filename number %d from pattern '%s' (either set updatefirst or use a pattern like %%03d within the filename pattern)\n",
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 31572f3..b6a7461 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -4379,15 +4379,18 @@ uint64_t ff_ntp_time(void)
     return (av_gettime() / 1000) * 1000 + NTP_OFFSET_US;
 }
 
-int av_get_frame_filename2(char *buf, int buf_size, const char *path, int number, int flags)
+int av_get_frame_filename2(char *buf, int buf_size, const char *path, int number, int flags, int64_t ts)
 {
     const char *p;
     char *q, buf1[20], c;
-    int nd, len, percentd_found;
+    int nd, len, percentd_found, percentt_found;
+    int hours, mins, secs, ms;
+    int64_t abs_ts;
 
     q = buf;
     p = path;
     percentd_found = 0;
+    percentt_found = 0;
     for (;;) {
         c = *p++;
         if (c == '\0')
@@ -4416,6 +4419,37 @@ int av_get_frame_filename2(char *buf, int buf_size, const char *path, int number
                 memcpy(q, buf1, len);
                 q += len;
                 break;
+            case 't':
+                if (!(flags & AV_FRAME_FILENAME_FLAGS_MULTIPLE) && percentt_found) {
+                    av_log(NULL, AV_LOG_ERROR, "double %%t not allowed");
+                    goto fail;
+                }
+                if (ts == 0) {
+                    av_log(NULL, AV_LOG_DEBUG, "%%t but no ts, using 0"); // necessary for first frame on some streams
+                }
+                percentt_found = 1;
+                abs_ts = llabs(ts);
+                ms = abs_ts % AV_TIME_BASE;
+                abs_ts /= AV_TIME_BASE;
+                secs = abs_ts % 60;
+                abs_ts /= 60;
+                mins = abs_ts % 60;
+                abs_ts /= 60;
+                hours = abs_ts;
+                if (ts < 0)
+                      snprintf(buf1, sizeof(buf1),
+                             "-%02d.%02d.%02d.%03d", hours, mins, secs, ms);
+                else
+                    snprintf(buf1, sizeof(buf1),
+                             "%02d.%02d.%02d.%03d", hours, mins, secs, ms);
+                len = strlen(buf1);
+                if ((q - buf + len) > buf_size - 1) {
+                   av_log(NULL, AV_LOG_ERROR, "%%t size overflow");
+                   goto fail;
+                }
+                memcpy(q, buf1, len);
+                q += len;
+                break;
             default:
                 goto fail;
             }
@@ -4425,7 +4459,7 @@ addchar:
                 *q++ = c;
         }
     }
-    if (!percentd_found)
+    if (!percentd_found && !percentt_found)
         goto fail;
     *q = '\0';
     return 0;
@@ -4436,7 +4470,7 @@ fail:
 
 int av_get_frame_filename(char *buf, int buf_size, const char *path, int number)
 {
-    return av_get_frame_filename2(buf, buf_size, path, number, 0);
+    return av_get_frame_filename2(buf, buf_size, path, number, 0, 0);
 }
 
 void av_url_split(char *proto, int proto_size,



More information about the ffmpeg-cvslog mailing list