[FFmpeg-devel] [PATCH] Enable pixdesc.c compilation

[Michael Niedermayer]

On Sat, Nov 07, 2009 at 04:46:53PM +0100, Stefano Sabatini wrote:
> On date Saturday 2009-11-07 13:42:50 +0100, Michael Niedermayer encoded:
> > On Sat, Nov 07, 2009 at 12:40:29AM +0100, Stefano Sabatini wrote:
> > > Hi,
> > > 
> > > as in patch, I plan to use it for some incoming filter so I need it.
> > 
> > until the headers are installed and the API made public theres no way you
> > can use this code from outside libavutil
> 
> Strictly speaking, no this isn't required since we can consider it
> just a sort of an internal API, and use it from within FFmpeg as it's
> done for many other modules.

any use of code outside libx is external to libx.
If you dislike this talk with all the distro maintainers they will explain
you with a pitchfork why


> 
> > making these things public needs another carefull review of the API as
> > we wont be able to change it afterwards.
> > 
> > Invest your time in reviwing the API before exporting it and using it all
> > over the place please.
> 
> Well I suppose this means to make libavcodec/imgconvert.c use pixdescs
> and dump PixFmtInfo, I'll do that, but I'd like not to block lavfi
> development in the meaningwhile.

then dont make lavfi depend on it

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

> ... defining _GNU_SOURCE...
For the love of all that is holy, and some that is not, don't do that.
-- Luca & Mans
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091107/301bdc65/attachment.pgp>