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

[Reimar Doeffinger]

On Tue, Sep 09, 2008 at 09:15:51PM +0200, Diego Biurrun wrote:
> On Tue, Sep 09, 2008 at 04:34:07PM +0200, Aurelien Jacobs wrote:
> > On Mon,  8 Sep 2008 23:26:24 +0200 (CEST)
> > uau <subversion at mplayerhq.hu> wrote:
> > 
> > > 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.
> > > 
> > > 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.
> 
> I think that this is indeed a positive sideeffect of making the lavf
> Matroska demuxer the default.  However, I don't think it should be the
> default in the next release as long as there are regressions.

Do you intend to do a release in the next few days? If not, this seems
to miss the point IMO.
It may indeed be the better choice to revert the default again, but only
Aurelien can guess his timeline of fixing the existing problems, thus such
a change should take his plans into account.
Or am I the only one who thinks that switching default demuxers twice a week
(each time a bug is fixed or a new one uncovered) would be the worst outcome
for the users (after all they can use -demuxer mkv, causing confusion seems
much worse to me)?

Greetings,
Reimar Doeffinger