[PATCH] glx: Revert part of 9838b703
Ian Romanick
idr at freedesktop.org
Tue Jul 3 12:03:46 PDT 2012
On 07/02/2012 06:29 PM, Eric Anholt wrote:
> Ian Romanick<idr at freedesktop.org> writes:
>
>> From: Ian Romanick<ian.d.romanick at intel.com>
>>
>> The server does not want GL extension prototypes. It never links with
>> anything that could possibly provide implementations of these functions. It
>> *is* the provide, and it does not provde these symbols. All this does is
>> create hundreds of warnings like:
>>
>> incude/GL/glext.h:5570:45: warning: redundant redeclaration of 'glMultTransposeMatrixd' [-Wredundant-decls]
>> include/GL/gl.h:1940:45: note: previous declaration of 'glMultTransposeMatrixd' was here
>
>
> Huh? When I look at the commit quoted, I see:
>
> -#define GL_GLEXT_PROTOTYPES /* we want prototypes */
> +#define GL_GLEXT_PROTOTYPES /* we want prototypes */
Yeah... So I misread this as moving the /* from the beginning of the
line to somewhere else in the line. I only went looking for this after
the tidalwave of warnings appeared in the build. I assumed this was due
to a recent change in GLX, but it seems that it was due to a recent
change in the build. Is -Wredundant-decls a useful warning?
Either way, the rest of the commit message is correct. The only case
where GL_GLEXT_PROTOTYPES should be defined is when linking with
something that will provide those symbols. The xserver does not link
with anything that will provide these symbols. Since the xserver is the
driver loader, it is the thing that would / could provide these symbols,
and it does not.
I think that it used to (and this define used to be necessary) back
before the AIGLX work, but that was ages ago.
> The #define has actually been present since:
>
> commit 9508a382f8a9f241dab097d921b6d290c1c3a776
> Author: Kaleb Keithley<kaleb at freedesktop.org>
> Date: Fri Nov 14 16:48:57 2003 +0000
>
> Initial revision
More information about the xorg-devel
mailing list