[Mesa-dev] [PATCH 12/19] glsl: Fix all the whitespace errors in link_uniform_block_active_visitor.cpp

Ian Romanick idr at freedesktop.org
Fri Dec 16 04:10:24 UTC 2016


From: Ian Romanick <ian.d.romanick at intel.com>

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
---
 .../glsl/link_uniform_block_active_visitor.cpp     | 46 ++++++++++------------
 1 file changed, 21 insertions(+), 25 deletions(-)

diff --git a/src/compiler/glsl/link_uniform_block_active_visitor.cpp b/src/compiler/glsl/link_uniform_block_active_visitor.cpp
index df8b221..cd1baf7 100644
--- a/src/compiler/glsl/link_uniform_block_active_visitor.cpp
+++ b/src/compiler/glsl/link_uniform_block_active_visitor.cpp
@@ -40,7 +40,7 @@ process_block(void *mem_ctx, struct hash_table *ht, ir_variable *var)
     */
    if (existing_block == NULL) {
       link_uniform_block_active *const b =
-	 rzalloc(mem_ctx, struct link_uniform_block_active);
+         rzalloc(mem_ctx, struct link_uniform_block_active);
 
       b->type = block_type;
       b->has_instance_name = var->is_interface_instance();
@@ -58,13 +58,13 @@ process_block(void *mem_ctx, struct hash_table *ht, ir_variable *var)
       return b;
    } else {
       link_uniform_block_active *const b =
-	 (link_uniform_block_active *) existing_block->data;
+         (link_uniform_block_active *) existing_block->data;
 
       if (b->type != block_type
-	  || b->has_instance_name != var->is_interface_instance())
-	 return NULL;
+          || b->has_instance_name != var->is_interface_instance())
+         return NULL;
       else
-	 return b;
+         return b;
    }
 
    assert(!"Should not get here.");
@@ -89,8 +89,8 @@ process_block(void *mem_ctx, struct hash_table *ht, ir_variable *var)
  *   }
  *
  * There are only 2 active blocks above but for the sake of indirect indexing
- * and not over complicating the code we will end up with a count of 8.
- * Here each dimension has 2 different indices counted so we end up with 2*2*2
+ * and not over complicating the code we will end up with a count of 8.  Here
+ * each dimension has 2 different indices counted so we end up with 2*2*2
  */
 static struct uniform_block_array_elements **
 process_arrays(void *mem_ctx, ir_dereference_array *ir,
@@ -107,8 +107,8 @@ process_arrays(void *mem_ctx, ir_dereference_array *ir,
       struct uniform_block_array_elements *ub_array = *ub_array_ptr;
       ir_constant *c = ir->array_index->as_constant();
       if (c) {
-         /* Index is a constant, so mark just that element used,
-          * if not already.
+         /* Index is a constant, so mark just that element used, if not
+          * already.
           */
          const unsigned idx = c->get_uint_component(0);
 
@@ -131,9 +131,7 @@ process_arrays(void *mem_ctx, ir_dereference_array *ir,
             ub_array->num_array_elements++;
          }
       } else {
-         /* The array index is not a constant,
-          * so mark the entire array used.
-          */
+         /* The array index is not a constant, so mark the entire array used. */
          assert(ir->array->type->is_array());
          if (ub_array->num_array_elements < ir->array->type->length) {
             ub_array->num_array_elements = ir->array->type->length;
@@ -147,6 +145,7 @@ process_arrays(void *mem_ctx, ir_dereference_array *ir,
             }
          }
       }
+
       return &ub_array->array;
    } else {
       return &block->array;
@@ -170,8 +169,7 @@ link_uniform_block_active_visitor::visit(ir_variable *var)
    if (var->get_interface_type_packing() == GLSL_INTERFACE_PACKING_PACKED)
       return visit_continue;
 
-   /* Process the block.  Bail if there was an error.
-    */
+   /* Process the block.  Bail if there was an error. */
    link_uniform_block_active *const b =
       process_block(this->mem_ctx, this->ht, var);
    if (b == NULL) {
@@ -236,14 +234,13 @@ link_uniform_block_active_visitor::visit_enter(ir_dereference_array *ir)
        || !var->is_interface_instance())
       return visit_continue;
 
-   /* Process the block.  Bail if there was an error.
-    */
+   /* Process the block.  Bail if there was an error. */
    link_uniform_block_active *const b =
       process_block(this->mem_ctx, this->ht, var);
    if (b == NULL) {
       linker_error(prog,
-		   "uniform block `%s' has mismatching definitions",
-		   var->get_interface_type()->name);
+                   "uniform block `%s' has mismatching definitions",
+                   var->get_interface_type()->name);
       this->success = false;
       return visit_stop;
    }
@@ -253,9 +250,9 @@ link_uniform_block_active_visitor::visit_enter(ir_dereference_array *ir)
    assert(b->has_instance_name);
    assert(b->type != NULL);
 
-   /* If the block array was declared with a shared or
-    * std140 layout qualifier, all its instances have been already marked
-    * as used in link_uniform_block_active_visitor::visit(ir_variable *).
+   /* If the block array was declared with a shared or std140 layout
+    * qualifier, all its instances have been already marked as used in
+    * link_uniform_block_active_visitor::visit(ir_variable *).
     */
    if (var->get_interface_type_packing() == GLSL_INTERFACE_PACKING_PACKED) {
       b->var = var;
@@ -275,14 +272,13 @@ link_uniform_block_active_visitor::visit(ir_dereference_variable *ir)
 
    assert(!var->is_interface_instance() || !var->type->is_array());
 
-   /* Process the block.  Bail if there was an error.
-    */
+   /* Process the block.  Bail if there was an error. */
    link_uniform_block_active *const b =
       process_block(this->mem_ctx, this->ht, var);
    if (b == NULL) {
       linker_error(this->prog,
-		   "uniform block `%s' has mismatching definitions",
-		   var->get_interface_type()->name);
+                   "uniform block `%s' has mismatching definitions",
+                   var->get_interface_type()->name);
       this->success = false;
       return visit_stop;
    }
-- 
2.7.4



More information about the mesa-dev mailing list