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

[Compn]

On Sat, 14 Jul 2007 14:46:00 +0200, Roberto Togni scribed:

>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.


i thought the point was to get vlc/xine/mplayer to use libavformat
so that all of the devs could work on one single codebase instead
of reinventing the wheel four times.

maybe someone should write up what is planned for mplayer's future?
of course, no one would agree on it, so lets just drop this now :)

-compn