[FFmpeg-cvslog] r11889 - trunk/libavutil/mem.h

Rich Felker dalias
Mon Feb 11 03:19:24 CET 2008


On Sun, Feb 10, 2008 at 10:16:10PM +0100, Diego Biurrun wrote:
> I suggest the following, which is farm more readable IMO even if -
> strictly speaking - it duplicates one line:
> 
> #ifdef __GNUC__
>   #define DECLARE_ALIGNED(n,t,v)       t v __attribute__ ((aligned (n)))
>   #define DECLARE_ASM_CONST(n,t,v)     static const t v attribute_used __attribute__ ((aligned (n)))
> #elif __ICC
>   #define DECLARE_ALIGNED(n,t,v)       t v __attribute__ ((aligned (n)))
>   #define DECLARE_ASM_CONST(n,t,v)     const t __attribute__ ((aligned (n))) v
> #else
>   #define DECLARE_ALIGNED(n,t,v)       __declspec(align(n)) t v
>   #define DECLARE_ASM_CONST(n,t,v)     __declspec(align(n)) static const t v
> #endif

Having MSVC crap in the default case is simply broken! This must be
fixed!

Rich




More information about the ffmpeg-cvslog mailing list