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

[Diego Biurrun]

On Mon, Jan 28, 2008 at 11:17:10PM +0100, Ivo wrote:
> 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).

Adding native codecs to libmpcodecs is deprecated.  Interfacing with
external libraries is not.

Diego