Mesa (gallium-0.1): scons: Specify C99 throughout all the tree.

Jose Fonseca jrfonseca at kemper.freedesktop.org
Mon Jan 5 15:32:20 UTC 2009


Module: Mesa
Branch: gallium-0.1
Commit: 8ee1df065234eaf27c53c03bdf6e4ad487bc326d
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8ee1df065234eaf27c53c03bdf6e4ad487bc326d

Author: José Fonseca <jfonseca at vmware.com>
Date:   Tue Dec 30 17:13:38 2008 +0000

scons: Specify C99 throughout all the tree.

MSVC may not support full C99, but supports more than plain C90. And
-pedantic without -std=c99 generates too many spurious warnings
(specially C++ style comments) to be of any use.

Note that using certain C99 features in the cross-platform parts of Gallium
is still not possible; namely mid-of-scope variable declarations and named
structure initializers will break MSVC builds.

---

 scons/gallium.py    |    1 +
 src/mesa/SConscript |    6 +-----
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/scons/gallium.py b/scons/gallium.py
index 1136d26..19eeb18 100644
--- a/scons/gallium.py
+++ b/scons/gallium.py
@@ -311,6 +311,7 @@ def generate(env):
             '-Wmissing-prototypes',
             '-Wno-long-long',
             '-ffast-math',
+            '-std=c99',
             '-pedantic',
             '-fmessage-length=0', # be nice to Eclipse
         ]
diff --git a/src/mesa/SConscript b/src/mesa/SConscript
index a690d57..72ddc34 100644
--- a/src/mesa/SConscript
+++ b/src/mesa/SConscript
@@ -12,11 +12,6 @@ if env['platform'] != 'winddk':
 		'#/src/mesa',
 	])
 	
-	if gcc:
-		env.Append(CFLAGS = [
-			'-std=c99',
-		])
-
 	#
 	# Source files
 	#
@@ -164,6 +159,7 @@ if env['platform'] != 'winddk':
 		'state_tracker/st_context.c',
 		'state_tracker/st_debug.c',
 		'state_tracker/st_draw.c',
+		'state_tracker/st_draw_feedback.c',
 		'state_tracker/st_extensions.c',
 		'state_tracker/st_format.c',
 		'state_tracker/st_framebuffer.c',




More information about the mesa-commit mailing list