[MPlayer-cvslog] r22162 - trunk/mplayer.c

[Reimar Doeffinger]

Hello,
On Tue, Feb 06, 2007 at 04:08:51PM +0200, Uoti Urpala wrote:
> On Tue, 2007-02-06 at 14:41 +0100, Reimar Doeffinger wrote:
> > Hmm... even if it works as it should with current MP_NOPTS_VALUE IMO it
> > would be a good idea to add an explicit check for it...
> 
> Yes, I thought so too after committing but forgot to change it. I think
> this case (pts == MP_NO_PTS_VALUE) should be avoided though; if there's
> a subtitle without explicit timestamp the demuxer should set it to the
> previous demuxed video pts or something.

I agree, so maybe the best solution would be to handle it but also print
a warning...

Greetings,
Reimar Doeffinger