[MPlayer-dev-eng] libavutil/bswap.h vs bswap.h

[Nicolas Plourde]

On 6-Dec-06, at 11:20 PM, Uoti Urpala wrote:

> On Thu, 2006-12-07 at 05:08 +0100, Diego Biurrun wrote:
>> On Thu, Dec 07, 2006 at 06:00:52AM +0200, Uoti Urpala wrote:
>>> I think it's Måns's changes which broke this in MPlayer. After the
>>> changes always_inline is only defined in libavutil/internal.h, which
>>> isn't included in MPlayer. The libavutil bswap.h uses that define.
>>
>> It was broken before.
>
> Are you sure you're talking about the same issue? There were problems
> caused by _having_ the always_inline define (since it breaks non-macro
> __attribute__((always_inline)) anywhere else). Were there problems  
> with
> _lack_ of that define before?
>

cc -Wdeclaration-after-statement -O4 -march=pentium-m -mtune=pentium- 
m -pipe -ffast-math -fomit-frame-pointer -mdynamic-no-pic -falign- 
loops=16 -DSYS_DARWIN -DCONFIG_DARWIN -shared-libgcc -arch i386 - 
D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -I/ 
usr/local/include -I/usr/X11R6/include -I/usr/local/include/freetype2  
-I/usr/local/include -I. -I./libavutil -I./libavcodec   -c -o  
mplayer.o mplayer.c
In file included from libmpdemux/aviheader.h:6,
                  from libmpdemux/stheader.h:4,
                  from mplayer.c:178:
./libavutil/bswap.h:40: error: syntax error before 'uint16_t'
./libavutil/bswap.h:48: error: syntax error before 'uint32_t'

libmpdemux/aviheader.h include common.h that include internal.h but  
to define always_inline we need to have HAVE_AV_CONFIG defined right?

====
Nicolas Plourde <nicolas.plourde at gmail.com>