On Sat, Jul 12, 2014 at 03:44:55PM +0200, Marton Balint wrote: > > > On Sat, 12 Jul 2014, Michael Niedermayer wrote: > > >Signed-off-by: Michael Niedermayer <michae...@gmx.at> > >--- > >ffplay.c | 1 + > >1 file changed, 1 insertion(+) > > > >diff --git a/ffplay.c b/ffplay.c > >index 7e954fd..6572c42 100644 > >--- a/ffplay.c > >+++ b/ffplay.c > >@@ -1446,6 +1446,7 @@ retry: > > sp2 = NULL; > > > > if (sp->serial != is->subtitleq.serial > >+ || !sp->sub.num_rects > > || (is->vidclk.pts > (sp->pts + ((float) > > sp->sub.end_display_time / 1000))) > > || (sp2 && is->vidclk.pts > (sp2->pts + > > ((float) sp2->sub.start_display_time / 1000)))) > > { > > I don't think this patch is needed. If the subpicture queue is > empty, it does not matter. If it is not empty then checking sp2 pts > in the last condition is enough to remove the empty subtitle (sp) > from the queue, if sp2 has the same pts as sp.
ok, patch droped then thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in which you can choose your dictator
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel