Mesa (glsl2): glsl2: Don' t declare a variable called sig that shadows the other one

Ian Romanick idr at kemper.freedesktop.org
Thu Aug 12 00:08:54 UTC 2010


Module: Mesa
Branch: glsl2
Commit: 202604e8160157e4e80b3458175e0170d168e557
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=202604e8160157e4e80b3458175e0170d168e557

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Wed Aug 11 16:58:25 2010 -0700

glsl2: Don't declare a variable called sig that shadows the other one

Accidentally having a variable called 'sig' within an if-statement
cause the higher scope 'sig' to always be NULL.  As a result a new
function signature was created for a function definition even when one
already existed from a prototype declaration.

Fixes piglit test case glsl-function-prototype (bugzilla #29520).

---

 src/glsl/ast_to_hir.cpp |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index f14341c..9d4448f 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@ -2128,7 +2128,7 @@ ast_function::hir(exec_list *instructions,
     */
    f = state->symbols->get_function(name);
    if (f != NULL) {
-      ir_function_signature *sig = f->exact_matching_signature(&hir_parameters);
+      sig = f->exact_matching_signature(&hir_parameters);
       if (sig != NULL) {
 	 const char *badvar = sig->qualifiers_match(&hir_parameters);
 	 if (badvar != NULL) {




More information about the mesa-commit mailing list