ir_print_visitor: Remove unnecessary parenthesis around type names.
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 7 Apr 2010 21:41:13 +0000 (14:41 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Wed, 28 Apr 2010 22:34:52 +0000 (15:34 -0700)
Parenthesis should only be present for compound types (i.e. arrays
or structures).  For atomic types, simply print the symbol.

ir_print_visitor.cpp

index 99dbacca11231a08176960cbeb71a027baf53ebb..cbe7d976ce79706704a9bb07aff9f6d80ceb529a 100644 (file)
@@ -28,11 +28,11 @@ static void
 print_type(const glsl_type *t)
 {
    if (t->base_type == GLSL_TYPE_ARRAY) {
-      printf("array (");
+      printf("(array ");
       print_type(t->fields.array);
-      printf(") (%u))", t->length);
+      printf(" (%u))", t->length);
    } else if (t->base_type == GLSL_TYPE_STRUCT) {
-      printf("struct (%s %u ", t->name ? t->name : "@", t->length);
+      printf("(struct (%s %u ", t->name ? t->name : "@", t->length);
       printf("(FINISHME: structure fields go here) ");
       printf(")");
    } else {
@@ -56,9 +56,7 @@ void ir_print_visitor::visit(ir_variable *ir)
       printf("(%s%s%s%s) ",
             cent, inv, mode[ir->mode], interp[ir->interpolation]);
 
-      printf("(");
       print_type(ir->type);
-      printf(") ");
       printf("(%s)) ", ir->name);
    }
 }
@@ -237,9 +235,9 @@ void ir_print_visitor::visit(ir_constant *ir)
 {
    const glsl_type *const base_type = ir->type->get_base_type();
 
-   printf("(constant (");
+   printf("(constant ");
    print_type(ir->type);
-   printf(") (");
+   printf(" (");
 
    for (unsigned i = 0; i < ir->type->components(); i++) {
       if (i != 0)