[MPlayer-dev-eng] [PATCH] add support for subtitles thru lavf

[Roberto Togni]

On Fri, 13 Jul 2007 22:15:35 +0200
Michael Niedermayer <michaelni at gmx.at> wrote:

> Hi
> 
> On Fri, Jul 13, 2007 at 05:47:53PM +0200, Diego Biurrun wrote:
> > On Fri, Jul 13, 2007 at 05:25:19PM +0200, Attila Kinali wrote:
> > > On Fri, 13 Jul 2007 14:44:05 +0200
> > > Diego Biurrun <diego at biurrun.de> wrote:
> > > 
> > > > Didn't test the patch, but this is a very welcome feature!  One step
> > > > closer to replacing native demuxers...
> > > 
> > > Stupid question, but why do you try to get rid of all native
> > > demuxers?
> > 
> > In order to replace them with the libavformat ones, which are maintained
> > and tend to work better.
> 
> additionally, libmpdemux + libavformat = code duplication
> 

<noflame> The same is true for MPlayer, xine, vlc... And for lavc mpeg4
and xvid... And...</noflame>

Mplayer is not FFmpeg. Supporting also lavf demuxers is good, I
disagree with removing our demuxers just because something similar
exists also in libavformat.

Ciao,
 Roberto