glsl: Drop bad ASSERT_TRUE in gl_CullDistance link_varyings test.
authorKenneth Graunke <kenneth@whitecape.org>
Sat, 14 May 2016 02:26:37 +0000 (19:26 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Mon, 16 May 2016 06:49:19 +0000 (23:49 -0700)
I don't know what the intention was here, but this function returns
void.  We can't assert anything about its return value.

Fixes "make check" failures.

v2: Also fix prototype for the function (caught by Jordan).

Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
src/compiler/glsl/tests/varyings_test.cpp

index 936f49581d77298e590032dd0858b4b9bc4e51e7..cabda7a01d4d176fa0f661e6bedcf03d8050902a 100644 (file)
@@ -35,7 +35,7 @@
  */
 
 namespace linker {
-bool
+void
 populate_consumer_input_sets(void *mem_ctx, exec_list *ir,
                              hash_table *consumer_inputs,
                              hash_table *consumer_interface_inputs,
@@ -210,11 +210,11 @@ TEST_F(link_varyings, gl_CullDistance)
 
    ir.push_tail(culldistance);
 
-   ASSERT_TRUE(linker::populate_consumer_input_sets(mem_ctx,
-                                                    &ir,
-                                                    consumer_inputs,
-                                                    consumer_interface_inputs,
-                                                    junk));
+   linker::populate_consumer_input_sets(mem_ctx,
+                                        &ir,
+                                        consumer_inputs,
+                                        consumer_interface_inputs,
+                                        junk);
 
    EXPECT_EQ(culldistance, junk[VARYING_SLOT_CULL_DIST0]);
    EXPECT_TRUE(is_empty(consumer_inputs));