X11R6.8.2rc1 build problem

Kevin E Martin kem at freedesktop.org
Sat Dec 18 14:19:42 PST 2004


I've been running build and run tests for RC1, and found a build problem
with gcc 2.96 in Mesa.  The problem is that "(void)rmesa" was added to
LOCAL_VARS to eliminate compiler warnings; however, in t_dd_dmatmp.h and
t_dd_dmatmp2.h, there are additional declarations after LOCAL_VARS.
This causes the build to fail on gcc 2.96.

Note this is _not_ a problem with gcc 3.4.2.

Below is a patch to workaround the problem, but it might better to add a
ELIMINATE_WARNINGS define with the list of "(void)var;"'s that need to
be used to eliminate compiler warnings, and then add ELIMINATE_WARNINGS
to the appropriate functions in t_dd_dmatmp.h and t_dd_dmatmp2.h.

I will continue my testing of RC1 this evening and report back any other
problems that I find.


Index: extras/Mesa/src/mesa/drivers/dri/r200/r200_tcl.c
===================================================================
RCS file: /cvs/xorg/xc/extras/Mesa/src/mesa/drivers/dri/r200/r200_tcl.c,v
retrieving revision 1.1.1.2.2.1
diff -u -r1.1.1.2.2.1 r200_tcl.c
--- extras/Mesa/src/mesa/drivers/dri/r200/r200_tcl.c	15 Dec 2004 01:50:58 -0000	1.1.1.2.2.1
+++ extras/Mesa/src/mesa/drivers/dri/r200/r200_tcl.c	18 Dec 2004 04:51:47 -0000
@@ -101,7 +101,7 @@
 };
    
 
-#define LOCAL_VARS r200ContextPtr rmesa = R200_CONTEXT(ctx); (void)rmesa
+#define LOCAL_VARS r200ContextPtr rmesa = R200_CONTEXT(ctx)
 #define ELT_TYPE  GLushort
 
 #define ELT_INIT(prim, hw_prim) \
Index: extras/Mesa/src/mesa/drivers/dri/radeon/radeon_swtcl.c
===================================================================
RCS file: /cvs/xorg/xc/extras/Mesa/src/mesa/drivers/dri/radeon/radeon_swtcl.c,v
retrieving revision 1.1.1.3.2.1
diff -u -r1.1.1.3.2.1 radeon_swtcl.c
--- extras/Mesa/src/mesa/drivers/dri/radeon/radeon_swtcl.c	15 Dec 2004 01:50:58 -0000	1.1.1.3.2.1
+++ extras/Mesa/src/mesa/drivers/dri/radeon/radeon_swtcl.c	18 Dec 2004 04:51:47 -0000
@@ -508,7 +508,7 @@
 
 
 
-#define LOCAL_VARS radeonContextPtr rmesa = RADEON_CONTEXT(ctx); (void)rmesa
+#define LOCAL_VARS radeonContextPtr rmesa = RADEON_CONTEXT(ctx)
 #define ELTS_VARS( buf )  GLushort *dest = buf
 #define INIT( prim ) radeonDmaPrimitive( rmesa, prim )
 #define ELT_INIT(prim) radeonEltPrimitive( rmesa, prim )
Index: extras/Mesa/src/mesa/drivers/dri/radeon/radeon_tcl.c
===================================================================
RCS file: /cvs/xorg/xc/extras/Mesa/src/mesa/drivers/dri/radeon/radeon_tcl.c,v
retrieving revision 1.1.1.2.2.1
diff -u -r1.1.1.2.2.1 radeon_tcl.c
--- extras/Mesa/src/mesa/drivers/dri/radeon/radeon_tcl.c	15 Dec 2004 01:50:58 -0000	1.1.1.2.2.1
+++ extras/Mesa/src/mesa/drivers/dri/radeon/radeon_tcl.c	18 Dec 2004 04:51:47 -0000
@@ -105,7 +105,7 @@
 };
    
 
-#define LOCAL_VARS radeonContextPtr rmesa = RADEON_CONTEXT(ctx); (void)rmesa
+#define LOCAL_VARS radeonContextPtr rmesa = RADEON_CONTEXT(ctx)
 #define ELT_TYPE  GLushort
 
 #define ELT_INIT(prim, hw_prim) \


More information about the release-wranglers mailing list