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

[Diego Biurrun]

On Sat, Jul 14, 2007 at 02:46:00PM +0200, Roberto Togni wrote:
> On Fri, 13 Jul 2007 22:15:35 +0200
> Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> > 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.

If at some point in time the libavformat demuxers are better and faster
than the native ones then the native ones should at least become a
compile-time option.

Diego