[MPlayer-cvslog] r25951 - in trunk: cfg-mplayer.h libmpdemux/demux_lavf.c libmpdemux/demux_mkv.c libmpdemux/demux_real.c libmpdemux/demuxer.c libmpdemux/demuxer.h

[Reimar Döffinger]

Hello,
On Sun, Feb 17, 2008 at 11:38:50AM +0200, Ivan Kalvachev wrote:
> On Feb 3, 2008 2:14 PM, reimar <subversion at mplayerhq.hu> wrote:
> > Author: reimar
> > Date: Sun Feb  3 13:14:29 2008
> > New Revision: 25951
> >
> > Log:
> > Allow demuxers to choose a default value for correct_pts
> >
> >
> > Modified:
> >    trunk/cfg-mplayer.h
> >    trunk/libmpdemux/demux_lavf.c
> >    trunk/libmpdemux/demux_mkv.c
> 
> I have just found out (empirically) that -correct-pts doesn't support
> any kind of -framedrop. It makes playback of some HDTV samples real
> nightmare. Not to say that this shortcoming is not documented.
> 
> I'm afraid it is too early to be set automatically on.

Firstly, I can not see why working -framedrop should be _more_ important
than playing flv and other files in sync.
Secondly, the right solution is of course to implement framedrop for
this case instead of waiting another few years for things to fix
themselves magically.