Current tinderbox regression (xserver)

[Paulo César Pereira de Andrade]

Chris Ball wrote:
> http://tinderbox.x.org/builds/2009-01-30-0005/logs/xserver/#build
>
> In file included from ../Xext/panoramiX.h:43, from dispatch.c:134:
> /home/cjb/xorg-build/include/X11/extensions/panoramiXext.h:48:
>   error: expected ')' before '*' token
> /home/cjb/xorg-build/include/X11/extensions/panoramiXext.h:53:
>   error: expected '=', ',', ';', 'asm' or '__attribute__'
>   before 'XPanoramiXQueryVersion'

  It is me striking again :-)  Since there was already one
missing prototype declared in panoramiXext.h, I just added the
remaining ones, but that requires defining "Display" to something
(yes, I did not remember the X Server build could include that
header...)

  The X Server build only needs the macros PANORAMIX_MAJOR_VERSION
and PANORAMIX_MINOR_VERSION from <X11/extensions/panoramiXext.h>

  One option is to use the schema used by some other related
cases, and protect the prototypes there with something like:
#ifndef _PANORAMIX_SERVER
...
#endif

  Or move those macros to X11/extensions/panoramiXproto.h like
is done in other similar cases.

  There is also some code doing weird things to get Display
defined, see xserver/Xext/securitysrv.h

  To avoid the risk of breaking things elsewhere, the proper
correction should be to protect the prototypes with:
#ifndef _PANORAMIX_SERVER
and add _XFUNCPROTOBEGIN/_XFUNCPROTOEND while at it.

> --
> Chris Ball   <cjb at laptop.org>

Paulo