[FFmpeg-devel] [PATCH 07/12] mov: use new public timecode API.

Clément Bœsch ubitux at gmail.com
Mon Jan 23 17:04:26 CET 2012


From: Clément Bœsch <clement.boesch at smartjog.com>

---
 libavformat/mov.c |   21 ++++++++++-----------
 1 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/libavformat/mov.c b/libavformat/mov.c
index 3bb42fe..38fd0d5 100644
--- a/libavformat/mov.c
+++ b/libavformat/mov.c
@@ -30,13 +30,13 @@
 #include "libavutil/mathematics.h"
 #include "libavutil/avstring.h"
 #include "libavutil/dict.h"
+#include "libavutil/timecode.h"
 #include "avformat.h"
 #include "internal.h"
 #include "avio_internal.h"
 #include "riff.h"
 #include "isom.h"
 #include "libavcodec/get_bits.h"
-#include "libavcodec/timecode.h"
 #include "id3v1.h"
 #include "mov_chan.h"
 
@@ -2633,17 +2633,16 @@ finish:
 static int parse_timecode_in_framenum_format(AVFormatContext *s, AVStream *st,
                                              uint32_t value)
 {
-    char buf[16];
-    struct ff_timecode tc = {
-        .drop  = st->codec->flags2 & CODEC_FLAG2_DROP_FRAME_TIMECODE,
-        .rate  = (AVRational){st->codec->time_base.den,
-                              st->codec->time_base.num},
-    };
-
-    if (avpriv_check_timecode_rate(s, tc.rate, tc.drop) < 0)
-        return AVERROR(EINVAL);
+    AVTimecode tc;
+    char buf[AV_TIMECODE_STR_SIZE];
+    AVRational rate = {st->codec->time_base.den,
+                       st->codec->time_base.num};
+    int flags = st->codec->flags2 & CODEC_FLAG2_DROP_FRAME_TIMECODE ? AV_TIMECODE_FLAG_DROPFRAME : 0;
+    int ret = av_timecode_init(s, &tc, rate, flags, 0);
+    if (ret < 0)
+        return ret;
     av_dict_set(&st->metadata, "timecode",
-                avpriv_timecode_to_string(buf, &tc, value), 0);
+                av_timecode_get_string(buf, &tc, value), 0);
     return 0;
 }
 
-- 
1.7.8.3



More information about the ffmpeg-devel mailing list