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

[Aurelien Jacobs]

On Mon,  8 Sep 2008 23:26:24 +0200 (CEST)
uau <subversion at mplayerhq.hu> wrote:

> Author: uau
> Date: Mon Sep  8 23:26:24 2008
> New Revision: 27551
> 
> Log:
> Use internal demuxer for Matroska files for now
> 
> Change the default demuxer back to the internal one at least until the
> current lavf breakage with SSA/ASS subtitles is sorted out.

This breaks much more things that it fixes.
Those files have at least one video or audio track which don't work anymore,
all for different reason:
http://samples.mplayerhq.hu/Matroska/switzler084d_dl.mkv
http://samples.mplayerhq.hu/Matroska/H264%2bEAC3.mkv
http://samples.mplayerhq.hu/Matroska/sample_44_2_16_TTA.mka
http://samples.mplayerhq.hu/Matroska/atlantis405-test.mkv
I guess I could find more if needed.

It also breaks other features, such as metadata demuxing or various kind
of compressed tracks decoding...

This is much worse than the few small issues the lavf demuxer have.
Moreover, I'm working on fixing the lavf issues, while no one is trying
to fix the numerous issues of native demuxer (AFAIK).

Only possible conclusion => reverted.

> There have
> also been quite a few other regressions so maybe the lavf demuxer
> should be tested a bit more before trying to make it the default again.

This was one of the purpose of making lavf demuxer the default one:
giving it more testing. So far, it has worked well, since it has
uncovered a few issues that I fixed, or am in the process of fixing.
But it seems you just don't want the lavf demuxer to improve or become
the default one.

Aurel