[MPlayer-cvslog] r25900 - in trunk: DOCS/tech/codec-devel.txt DOCS/tech/libmpcodecs.txt libmpcodecs/vd.c libmpcodecs/ve.c libmpdemux/demuxer.c

[Ivo]

On Monday 28 January 2008 22:09, Roberto Togni wrote:
> On Mon, 28 Jan 2008 13:11:42 +0100 (CET)
> ivo <subversion at mplayerhq.hu> wrote:
> > +NOTE: If you want to implement a new codec, please add it to
> > libavcodec. +libmpcodecs is considered deprecated.
> > +
>
> This is totally wrong. libmpcodecs is not deprecated.
> Asking to add native codecs to lavc instead of libmpcodecs is ok, but
> everything else like wrapper for external libraries, decoders requiring
> binary support, ... must be added to libmpcodecs.

I clarified the comments/docs. In case it's still lacking, feel free to 
reword it. At least there's a message now which prevents people from 
wasting their time at implementing a codec/(de)muxer for libmp* instead of 
lav*.

BTW I think the wrapper for external libraries case is unclear. I guess it's 
OK to add those to lav* as well as long as there's no native equivalent. 
i.e. libmp3lame for encoding mp3 is accepted, libxvid is not (because 
there's fmp4).

--Ivo