matroskadec: ensure that overlapping subtitles are not added to the index
authorAurelien Jacobs <aurel@gnuage.org>
Tue, 9 Sep 2008 12:07:10 +0000 (12:07 +0000)
committerAurelien Jacobs <aurel@gnuage.org>
Tue, 9 Sep 2008 12:07:10 +0000 (12:07 +0000)
Originally committed as revision 15283 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavformat/matroskadec.c

index a6a682f..948238c 100644 (file)
@@ -139,6 +139,7 @@ typedef struct {
     EbmlList encodings;
 
     AVStream *stream;
     EbmlList encodings;
 
     AVStream *stream;
+    int64_t end_timecode;
 } MatroskaTrack;
 
 typedef struct {
 } MatroskaTrack;
 
 typedef struct {
@@ -1462,8 +1463,12 @@ static int matroska_parse_block(MatroskaDemuxContext *matroska, uint8_t *data,
     if (cluster_time != (uint64_t)-1
         && (block_time >= 0 || cluster_time >= -block_time)) {
         timecode = cluster_time + block_time;
     if (cluster_time != (uint64_t)-1
         && (block_time >= 0 || cluster_time >= -block_time)) {
         timecode = cluster_time + block_time;
+        if (track->type == MATROSKA_TRACK_TYPE_SUBTITLE
+            && timecode < track->end_timecode)
+            is_keyframe = 0;  /* overlapping subtitles are not key frame */
         if (is_keyframe)
             av_add_index_entry(st, cluster_pos, timecode, 0,0,AVINDEX_KEYFRAME);
         if (is_keyframe)
             av_add_index_entry(st, cluster_pos, timecode, 0,0,AVINDEX_KEYFRAME);
+        track->end_timecode = FFMAX(track->end_timecode, timecode+duration);
     }
 
     if (matroska->skip_to_keyframe) {
     }
 
     if (matroska->skip_to_keyframe) {