From 929eb00b32dfea9b1ac58923c88d5573872adea1 Mon Sep 17 00:00:00 2001
From: Brian Paul <brian.paul@tungstengraphics.com>
Date: Thu, 11 Dec 2008 18:49:28 -0700
Subject: mesa: glsl clean-ups

---
 src/mesa/shader/slang/slang_codegen.c | 95 ++++++++++++++++++-----------------
 1 file changed, 48 insertions(+), 47 deletions(-)

(limited to 'src/mesa/shader/slang/slang_codegen.c')

diff --git a/src/mesa/shader/slang/slang_codegen.c b/src/mesa/shader/slang/slang_codegen.c
index 9d6a9d7d1b..2030cc03fa 100644
--- a/src/mesa/shader/slang/slang_codegen.c
+++ b/src/mesa/shader/slang/slang_codegen.c
@@ -2490,61 +2490,62 @@ _slang_gen_var_decl(slang_assemble_ctx *A, slang_variable *var)
    var->declared = GL_TRUE;
 
    n = new_node0(IR_VAR_DECL);
-   if (n) {
-      _slang_attach_storage(n, var);
-      assert(var->store);
-      assert(n->Store == var->store);
-      assert(n->Store);
-      assert(n->Store->Index < 0);
+   if (!n)
+      return NULL;
 
-      if (is_sampler_type(&var->type)) {
-         n->Store->File = PROGRAM_SAMPLER;
-      }
-      else {
-         n->Store->File = PROGRAM_TEMPORARY;
-      }
+   _slang_attach_storage(n, var);
+   assert(var->store);
+   assert(n->Store == var->store);
+   assert(n->Store);
+   assert(n->Store->Index < 0);
+
+   if (is_sampler_type(&var->type)) {
+      n->Store->File = PROGRAM_SAMPLER;
+   }
+   else {
+      n->Store->File = PROGRAM_TEMPORARY;
+   }
 
-      n->Store->Size = _slang_sizeof_type_specifier(&n->Var->type.specifier);
+   n->Store->Size = _slang_sizeof_type_specifier(&n->Var->type.specifier);
 
-      if (n->Store->Size <= 0) {
-         slang_info_log_error(A->log, "invalid declaration for '%s'",
-                              (char*) var->a_name);
-         return NULL;
-      }
+   if (n->Store->Size <= 0) {
+      slang_info_log_error(A->log, "invalid declaration for '%s'",
+                           (char*) var->a_name);
+      return NULL;
+   }
 #if 0
-      printf("%s var %p %s  store=%p index=%d size=%d\n",
-             __FUNCTION__, (void *) var, (char *) var->a_name,
-             (void *) n->Store, n->Store->Index, n->Store->Size);
+   printf("%s var %p %s  store=%p index=%d size=%d\n",
+          __FUNCTION__, (void *) var, (char *) var->a_name,
+          (void *) n->Store, n->Store->Index, n->Store->Size);
 #endif
 
-      if (var->array_len > 0) {
-         /* this is an array */
-         /* round up the element size to a multiple of 4 */
-         GLint sz = (n->Store->Size + 3) & ~3;
-         /* total size = element size * array length */
-         sz *= var->array_len;
-         n->Store->Size = sz;
-      }
-
-      assert(n->Store->Size > 0);
+   if (var->array_len > 0) {
+      /* this is an array */
+      /* round up the element size to a multiple of 4 */
+      GLint sz = (n->Store->Size + 3) & ~3;
+      /* total size = element size * array length */
+      sz *= var->array_len;
+      n->Store->Size = sz;
+   }
 
-      /* setup default swizzle for storing the variable */
-      switch (n->Store->Size) {
-      case 2:
-         n->Store->Swizzle = MAKE_SWIZZLE4(SWIZZLE_X, SWIZZLE_Y,
-                                           SWIZZLE_NIL, SWIZZLE_NIL);
-         break;
-      case 3:
-         n->Store->Swizzle = MAKE_SWIZZLE4(SWIZZLE_X, SWIZZLE_Y,
-                                           SWIZZLE_Z, SWIZZLE_NIL);
-         break;
-      default:
-         /* Note that float-sized vars may be allocated in any x/y/z/w
-          * slot, but that won't be determined until code emit time.
-          */
-         n->Store->Swizzle = SWIZZLE_NOOP;
-      }
+   /* setup default swizzle for storing the variable */
+   /* XXX this may not be needed anymore - remove & test */
+   switch (n->Store->Size) {
+   case 2:
+      n->Store->Swizzle = MAKE_SWIZZLE4(SWIZZLE_X, SWIZZLE_Y,
+                                        SWIZZLE_NIL, SWIZZLE_NIL);
+      break;
+   case 3:
+      n->Store->Swizzle = MAKE_SWIZZLE4(SWIZZLE_X, SWIZZLE_Y,
+                                        SWIZZLE_Z, SWIZZLE_NIL);
+      break;
+   default:
+      /* Note that float-sized vars may be allocated in any x/y/z/w
+       * slot, but that won't be determined until code emit time.
+       */
+      n->Store->Swizzle = SWIZZLE_NOOP;
    }
+
    return n;
 }
 
-- 
cgit v1.2.3