Mesa (master): mesa: remove redundant running of check_symbol_table()

Tapani Pälli tpalli at kemper.freedesktop.org
Wed Apr 2 16:56:23 UTC 2014


Module: Mesa
Branch: master
Commit: 0f641b2d5025a4306c6e5f9d639def4061f69667
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=0f641b2d5025a4306c6e5f9d639def4061f69667

Author: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
Date:   Wed Apr  2 16:13:59 2014 +0300

mesa: remove redundant running of check_symbol_table()

Nested for loops running through tables against which they
finally do an assert were ran also with optimized builds.

Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/mesa/program/symbol_table.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mesa/program/symbol_table.c b/src/mesa/program/symbol_table.c
index 2f41322..9462978 100644
--- a/src/mesa/program/symbol_table.c
+++ b/src/mesa/program/symbol_table.c
@@ -115,7 +115,7 @@ struct _mesa_symbol_table {
 static void
 check_symbol_table(struct _mesa_symbol_table *table)
 {
-#if 1
+#if !defined(NDEBUG)
     struct scope_level *scope;
 
     for (scope = table->current_scope; scope != NULL; scope = scope->next) {
@@ -134,7 +134,9 @@ check_symbol_table(struct _mesa_symbol_table *table)
             }
         }
     }
-#endif
+#else
+    (void) table;
+#endif /* !defined(NDEBUG) */
 }
 
 void




More information about the mesa-commit mailing list