[MPlayer-cvslog] r30100 - trunk/libmpdemux/demux_lavf.c

[Reimar Döffinger]

On Mon, Dec 28, 2009 at 10:11:10AM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
> > > +        case DEMUXER_CTRL_CORRECT_PTS:
> > > +           if (!strcmp("mov,mp4,m4a,3gp,3g2,mj2", priv->avif->name))
> > > +               return DEMUXER_CTRL_OK;
> > > +           else
> > > +               return DEMUXER_CTRL_NOTIMPL;
> > 
> > I'm not sure this is a good idea, there were similar issues with flv
> > IIRC.
> 
> Please point me to a sample, the only problematic one I know of needs
> nocorrect-pts (issue 1040).

I don't know which it was, but they were the reason to make correct-pts
the default in the first place.
If you can find it, the "Alexander" WMV HD trailer might be a good
candidate as well, since it is real variable fps.

> > If the only think you could find an issue with is mkv files, IMO only
> > disable correct-pts only for mkv (though in that case I have to wonder
> > if there isn't something wrong with the mkv demuxer).
> > But I have some doubts this really depends on the container format...
> 
> So you can point me to a (non-mov) PAFF sample that works with correct-pts in
> current svn?

No, but do you have any non-mkv samples that do not work with it?
Could you try remuxing the working and non-working samples you have into
different formats with ffmpeg (without reencoding), in particular mkv,
mov, nut and maybe mpegts and see what happens? Does MPlayer's behaviour
differ from ffplay?