[MPlayer-dev-eng] libavutil/bswap.h vs bswap.h
nicolas.plourde at gmail.com
Thu Dec 7 12:54:20 CET 2006
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
> _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
In file included from libmpdemux/aviheader.h:6,
./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>
More information about the MPlayer-dev-eng