[Mesa-dev] [PATCH 1/4] mesa: Add an ifndef guard around the definition of the INLINE macro

Ian Romanick idr at freedesktop.org
Tue Jul 19 13:15:42 PDT 2011


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 07/18/2011 02:28 PM, Paul Berry wrote:
> On 18 July 2011 11:58, Brian Paul <brianp at vmware.com> wrote:
>> On 07/18/2011 12:37 PM, Paul Berry wrote:
>>>
>>> Several headers redundantly define the INLINE macro.  Adding this
>>> guard prevents the compiler from complaining about macro redefinition.
>>
>> Other Mesa headers?  Or system headers?  Or?
> 
> Other Mesa headers.  I found definitions of INLINE in four header files:
> 
> src/egl/main/eglcompiler.h
> src/gallium/include/pipe/p_compiler.h
> src/mapi/mapi/u_compiler.h
> src/mesa/main/compiler.h

None of those headers should ever cross paths.  The only one of those
that anything under src/mesa/main or src/glsl should *ever* hit is
src/mesa/main/compiler.h.  Where did the colliding definition come from?

> Ideally we  should put all of our definitions of INLINE in one place,
> but I was doing enough in other parts of the patch series that I
> didn't want to shave that particular yak :)
> 
> I'll update the commit comment to clarify.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk4l5e0ACgkQX1gOwKyEAw+QVwCgk1cdXMlGsADztGASrpRnNnXR
XVcAoJPPwaYzwNEkDtB+H2hZbqNTqXzh
=XXUE
-----END PGP SIGNATURE-----


More information about the mesa-dev mailing list