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

[Måns Rullgård]

Michael Niedermayer <michaelni at gmx.at> writes:

> On Sun, Jan 17, 2010 at 02:35:50PM +0000, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> 
>> >> config.h should be installed, or at the very very least a file that
>> >> contains the big_endian define, none of the arguments against
>> >> config.h installation apply to this define
>> 
>> I'll see about installing a sanitised subset of config.h.  We'll have
>> to rename the HAVE_ and CONFIG_ tokens in it though, or we'll be
>> clashing with others.
>
> :))))))))

One (bikeshed) issue though: where do we install this header?  We
currently install headers in $incdir/libav*/, but config.h applies to
all of them.

>> > besides intreadwrite.h needs config.h as well and intreadwrite.h should be
>> > an installed header!
>> 
>> That cannot happen.  It uses inline asm and other compiler-specifics
>> all over the place.
>
> what about intreadwrite_gcc_asm.h ?

Some parts depend on the gcc version too, and there are or will be
specifics for non-gcc compilers as well.

-- 
M?ns Rullg?rd
mans at mansr.com