Commit 4f26c999 authored by Clément Bœsch's avatar Clément Bœsch

avformat/subtitles: drop duplicated events

Fix Ticket #4843
parent 4f7189f0
......@@ -166,6 +166,34 @@ static int cmp_pkt_sub_pos_ts(const void *a, const void *b)
return s1->pos > s2->pos ? 1 : -1;
}
static void drop_dups(FFDemuxSubtitlesQueue *q)
{
int i, drop = 0;
for (i = 1; i < q->nb_subs; i++) {
const int last_id = i - 1 - drop;
const AVPacket *last = &q->subs[last_id];
if (q->subs[i].pts == last->pts &&
q->subs[i].duration == last->duration &&
!strcmp(q->subs[i].data, last->data)) {
av_free_packet(&q->subs[i]);
drop++;
} else if (drop) {
q->subs[last_id + 1] = q->subs[i];
memset(&q->subs[i], 0, sizeof(q->subs[i])); // for safety
}
}
if (drop) {
q->nb_subs -= drop;
// TODO: forward log context down here
av_log(NULL, AV_LOG_WARNING, "Dropping %d duplicated subtitle events\n", drop);
}
}
void ff_subtitles_queue_finalize(FFDemuxSubtitlesQueue *q)
{
int i;
......@@ -176,6 +204,7 @@ void ff_subtitles_queue_finalize(FFDemuxSubtitlesQueue *q)
for (i = 0; i < q->nb_subs; i++)
if (q->subs[i].duration == -1 && i < q->nb_subs - 1)
q->subs[i].duration = q->subs[i + 1].pts - q->subs[i].pts;
drop_dups(q);
}
int ff_subtitles_queue_read_packet(FFDemuxSubtitlesQueue *q, AVPacket *pkt)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment