[FFmpeg-devel] pixfmt.h installed header using HAVE_BIGENDIAN?

Stefano Sabatini stefano.sabatini-lala
Tue Jan 19 00:04:08 CET 2010


On date Sunday 2010-01-17 19:09:19 +0100, Stefano Sabatini encoded:
> On date Sunday 2010-01-17 18:49:33 +0100, Michael Niedermayer encoded:
> > On Sun, Jan 17, 2010 at 04:52:12PM +0100, Stefano Sabatini wrote:
> [...]
> > > This is the simplest of the many variants we may adopt, others:
> > > * create a pixfmt_internal.h included automatically in
> > >   libavutil/internal.h
> > > * make each file to explicitely include the pixfmt_internal.h header
> > >   when required.
> > > 
> > > I saw mans' RFC after writing the patch, so another option could be to
> > > *sanely* expose the _NE stuff.
> > > 
> > > Regards.
> > > -- 
> > > FFmpeg = Foolish and Fiendish Multimedia Powered Evangelical Gadget
> > 
> > >  internal.h |   22 ++++++++++++++++++++++
> > >  pixfmt.h   |   25 -------------------------
> > >  2 files changed, 22 insertions(+), 25 deletions(-)
> > > f632d952c51d36f801e49e834c28550bc0fe6f00  fix-pixfmt-ne-stuff.patch
> > 
> > iam not in favor of this patch
> > its semantically wrong, as that stuff are pixel formats and it would force
> > us to include libavutil/internal.h from many places. And whatever you say
> > people will include libavutil/internal.h if half of the API they want to
> > use is there
> 
> So what about this variant:
> * create a pixfmt_internal.h and make each file to explicitely include
>   the pixfmt_internal.h header when required
> ?

Like in attached, it also fixes swscale-example compilation.

Regards.
-- 
FFmpeg = Fantastic Frenzy Most Pacific Explosive Gadget
-------------- next part --------------
A non-text attachment was scrubbed...
Name: create-pix-fmt-internal.patch
Type: text/x-diff
Size: 22048 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100119/9de39b87/attachment.patch>



More information about the ffmpeg-devel mailing list