Merge remote-tracking branch 'qatar/master'

[ffmpeg] / libavformat / microdvddec.c

diff --git a/libavformat/microdvddec.c b/libavformat/microdvddec.c
index b1b784c8838957d5869f67eb0e96f852e2e18b83..b47457454d9368dbac0f18e2ff66876964da4770 100644 (file)
--- a/libavformat/microdvddec.c
+++ b/libavformat/microdvddec.c
@@ -1,6 +1,7 @@
 /*
  * MicroDVD subtitle demuxer
  * Copyright (c) 2010  Aurelien Jacobs <aurel@gnuage.org>
+ * Copyright (c) 2012  Clément Bœsch <ubitux@gmail.com>
  *
  * This file is part of FFmpeg.
  *
@@ -21,14 +22,14 @@
 
 #include "avformat.h"
 #include "internal.h"
+#include "subtitles.h"
 #include "libavutil/intreadwrite.h"
 
 #define MAX_LINESIZE 2048
 
 
 typedef struct {
-    uint8_t lines[3][MAX_LINESIZE];
-    int64_t pos[3];
+    FFDemuxSubtitlesQueue q;
 } MicroDVDContext;
 
 
@@ -50,72 +51,86 @@ static int microdvd_probe(AVProbeData *p)
     return AVPROBE_SCORE_MAX;
 }
 
+static int64_t get_pts(const char *buf)
+{
+    int frame;
+    char c;
+
+    if (sscanf(buf, "{%d}{%c", &frame, &c) == 2)
+        return frame;
+    return AV_NOPTS_VALUE;
+}
+
+static int get_duration(const char *buf)
+{
+    int frame_start, frame_end;
+
+    if (sscanf(buf, "{%d}{%d}", &frame_start, &frame_end) == 2)
+        return frame_end - frame_start;
+    return -1;
+}
+
 static int microdvd_read_header(AVFormatContext *s)
 {
     AVRational pts_info = (AVRational){ 2997, 125 };  /* default: 23.976 fps */
     MicroDVDContext *microdvd = s->priv_data;
     AVStream *st = avformat_new_stream(s, NULL);
-    int i, frame;
-    double fps;
-    char c;
+    int i = 0;
+    char line[MAX_LINESIZE];
 
     if (!st)
-        return -1;
-    for (i=0; i<FF_ARRAY_ELEMS(microdvd->lines); i++) {
-        microdvd->pos[i] = avio_tell(s->pb);
-        ff_get_line(s->pb, microdvd->lines[i], sizeof(microdvd->lines[i]));
-        if ((sscanf(microdvd->lines[i], "{%d}{}%6lf",    &frame, &fps) == 2 ||
-             sscanf(microdvd->lines[i], "{%d}{%*d}%6lf", &frame, &fps) == 2)
-            && frame <= 1 && fps > 3 && fps < 100)
-            pts_info = av_d2q(fps, 100000);
-        if (sscanf(microdvd->lines[i], "{DEFAULT}{}%c", &c) == 1) {
-            st->codec->extradata = av_strdup(microdvd->lines[i] + 11);
-            st->codec->extradata_size = strlen(st->codec->extradata);
-            i--;
+        return AVERROR(ENOMEM);
+
+    while (!url_feof(s->pb)) {
+        AVPacket *sub;
+        int64_t pos = avio_tell(s->pb);
+        int len = ff_get_line(s->pb, line, sizeof(line));
+
+        if (!len)
+            break;
+        if (i < 3) {
+            int frame;
+            double fps;
+            char c;
+
+            i++;
+            if ((sscanf(line, "{%d}{}%6lf",    &frame, &fps) == 2 ||
+                 sscanf(line, "{%d}{%*d}%6lf", &frame, &fps) == 2)
+                && frame <= 1 && fps > 3 && fps < 100)
+                pts_info = av_d2q(fps, 100000);
+            if (!st->codec->extradata && sscanf(line, "{DEFAULT}{}%c", &c) == 1) {
+                st->codec->extradata = av_strdup(line + 11);
+                if (!st->codec->extradata)
+                    return AVERROR(ENOMEM);
+                st->codec->extradata_size = strlen(st->codec->extradata) + 1;
+                continue;
+            }
         }
+        sub = ff_subtitles_queue_insert(&microdvd->q, line, len, 0);
+        if (!sub)
+            return AVERROR(ENOMEM);
+        sub->pos = pos;
+        sub->pts = get_pts(sub->data);
+        sub->duration = get_duration(sub->data);
     }
+    ff_subtitles_queue_finalize(&microdvd->q);
     avpriv_set_pts_info(st, 64, pts_info.den, pts_info.num);
     st->codec->codec_type = AVMEDIA_TYPE_SUBTITLE;
     st->codec->codec_id   = CODEC_ID_MICRODVD;
     return 0;
 }
 
-static int64_t get_pts(const char *buf)
+static int microdvd_read_packet(AVFormatContext *s, AVPacket *pkt)
 {
-    int frame;
-    char c;
-
-    if (sscanf(buf, "{%d}{%c", &frame, &c) == 2)
-        return frame;
-    return AV_NOPTS_VALUE;
+    MicroDVDContext *microdvd = s->priv_data;
+    return ff_subtitles_queue_read_packet(&microdvd->q, pkt);
 }
 
-static int microdvd_read_packet(AVFormatContext *s, AVPacket *pkt)
+static int microdvd_read_close(AVFormatContext *s)
 {
     MicroDVDContext *microdvd = s->priv_data;
-    char buffer[MAX_LINESIZE];
-    int64_t pos = avio_tell(s->pb);
-    int i, len = 0, res = AVERROR_EOF;
-
-    for (i=0; i<FF_ARRAY_ELEMS(microdvd->lines); i++) {
-        if (microdvd->lines[i][0]) {
-            strcpy(buffer, microdvd->lines[i]);
-            pos = microdvd->pos[i];
-            len = strlen(buffer);
-            microdvd->lines[i][0] = 0;
-            break;
-        }
-    }
-    if (!len)
-        len = ff_get_line(s->pb, buffer, sizeof(buffer));
-
-    if (buffer[0] && !(res = av_new_packet(pkt, len))) {
-        memcpy(pkt->data, buffer, len);
-        pkt->flags |= AV_PKT_FLAG_KEY;
-        pkt->pos = pos;
-        pkt->pts = pkt->dts = get_pts(buffer);
-    }
-    return res;
+    ff_subtitles_queue_clean(&microdvd->q);
+    return 0;
 }
 
 AVInputFormat ff_microdvd_demuxer = {
@@ -125,5 +140,6 @@ AVInputFormat ff_microdvd_demuxer = {
     .read_probe     = microdvd_probe,
     .read_header    = microdvd_read_header,
     .read_packet    = microdvd_read_packet,
+    .read_close     = microdvd_read_close,
     .flags          = AVFMT_GENERIC_INDEX,
 };