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

[Uoti Urpala]

On Tue, 2008-09-09 at 19:58 +0200, Reimar Döffinger wrote:
> On Tue, Sep 09, 2008 at 04:45:50PM +0200, uau wrote:
> > The internal demuxer must remain the default at least
> > until the subtitle issues are resolved.
> 
> Let me make this clear, and I intend to say it only once: unless you
> intend to maintain both the subtitle code and the internal matroska
> demuxer, this is not for you to decide, at least certainly not you alone

SSA/ASS subtitle functionality is an important feature. It's not OK to
just leave that broken in svn. And the breakage did already lead to user
complaints. I can keep the subtitle code working (and have just had to
fix the breakage Aurelien caused...). If you mean that Aurelien should
be allowed to leave MPlayer Matroska handling broken in any way he wants
if he's maintaining the lavf Matroska demuxer I do not agree.

> and not by overriding the decision of the matroska maintainer without
> any input from the person maintaining the file you change.

Aurelien has not maintained Matroska in MPlayer for years. And even if
he had that would not make it his decision to leave important
functionality broken for very little benefit. What file the demuxer
preferences are expressed in is completely irrelevant.


The main point is: the svn version should stay usable. The current state
of the lavf Matroska demuxer causes practical problems for a lot of
people. Using the internal demuxer causes problems for far fewer. There
is no development reason to make the lavf demuxer the default until the
known problems are fixed.