Marc Dietrich wrote: > this is needed at least for gcc. dunno about the other compilers. > --- > src/gallium/include/pipe/p_compiler.h | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/gallium/include/pipe/p_compiler.h > b/src/gallium/include/pipe/p_compiler.h > index 272d030..cdda20c 100644 > --- a/src/gallium/include/pipe/p_compiler.h > +++ b/src/gallium/include/pipe/p_compiler.h > @@ -144,7 +144,7 @@ typedef unsigned char boolean; > #if defined(__GNUC__) || (defined(__SUNPRO_C) && (__SUNPRO_C >= 0x590)) > > /* See http://gcc.gnu.org/onlinedocs/gcc-4.4.2/gcc/Type-Attributes.html */ > -#define PIPE_ALIGN_TYPE(_alignment, _type) _type > __attribute__((aligned(_alignment))) > +#define PIPE_ALIGN_TYPE(_alignment, _type...) _type > __attribute__((aligned(_alignment))) > > /* See http://gcc.gnu.org/onlinedocs/gcc-4.4.2/gcc/Variable-Attributes.html > */ > #define PIPE_ALIGN_VAR(_alignment) __attribute__((aligned(_alignment))) > @@ -158,14 +158,14 @@ typedef unsigned char boolean; > #elif defined(_MSC_VER) > > /* See http://msdn.microsoft.com/en-us/library/83ythb65.aspx */ > -#define PIPE_ALIGN_TYPE(_alignment, _type) __declspec(align(_alignment)) > _type > +#define PIPE_ALIGN_TYPE(_alignment, _type...) __declspec(align(_alignment)) > _type > #define PIPE_ALIGN_VAR(_alignment) __declspec(align(_alignment)) > > #define PIPE_ALIGN_STACK > > #elif defined(SWIG) > > -#define PIPE_ALIGN_TYPE(_alignment, _type) _type > +#define PIPE_ALIGN_TYPE(_alignment, _type...) _type > #define PIPE_ALIGN_VAR(_alignment) > > #define PIPE_ALIGN_STACK
With your patch I'm seeing new warnings that I didn't get before: ../../src/gallium/include/pipe/p_compiler.h:147:42: warning: ISO C does not permit named variadic macros I'm using gcc 4.3.2 Also, if you didn't test other compilers, the patch probably should not change PIPE_ALIGN_TYPE for MSVC or SWIG, right? -Brian ------------------------------------------------------------------------------ The Planet: dedicated and managed hosting, cloud storage, colocation Stay online with enterprise data centers and the best network in the business Choose flexible plans and management services without long-term contracts Personal 24x7 support from experience hosting pros just a phone call away. http://p.sf.net/sfu/theplanet-com _______________________________________________ Mesa3d-dev mailing list Mesa3d-dev@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mesa3d-dev