[MPlayer-dev-eng] [PATCH]Set default mc to 1

[Uoti Urpala]

On Thu, 2009-04-23 at 23:49 +0200, Carl Eugen Hoyos wrote:
> Attached patch makes all PAFF samples playable (with a small, for some 
> samples even acceptable delay), while I do not know of any sample that 
> does not desync immediately with latest svn.

That has unwanted side effects. I've implemented better handling of
unstable timestamps in git (though for correct-pts mode only).

I tested reordering the timestamps in the decoder after this was last
mentioned on advusers. However lavf fails to return pts values for some
packets, even with AVFMT_FLAG_GENPTS, so some frames are left without a
timestamp and will be timed somewhat incorrectly. I haven't followed
FFmpeg discussion that closely so I'm not sure whether this is currently
considered a bug, a known limitation in parsing or what. Also lavc
decoder reordering doesn't seem to work when using VDPAU, so that would
need separate code. The internal mpeg demuxers do not try to return
timestamps suitable for reordering and do not work properly with
correct-pts.

I think the players that produce least visible errors on the
MPEG-container PAFF samples are doing timing based on dts. I could add
that mode to MPlayer too, but to produce correct results it'd require
the demuxer and decoder to agree on exactly how "dts" is defined - which
likely will not be the case at least when using ffmpeg-mt. Some more
hacky special-case modes would be possible too, but so far I haven't
bothered trying to implement those as I've had no need to watch
interlaced content myself.