On 12/16/2014 11:07 PM, Vinson Lee wrote:
Fix Coverity "big parameter passed by value" defect.

Signed-off-by: Vinson Lee <v...@freedesktop.org>
---
  tests/spec/arb_sample_shading/execution/builtin-gl-num-samples.cpp | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/spec/arb_sample_shading/execution/builtin-gl-num-samples.cpp 
b/tests/spec/arb_sample_shading/execution/builtin-gl-num-samples.cpp
index 620c12e..08bbea9 100644
--- a/tests/spec/arb_sample_shading/execution/builtin-gl-num-samples.cpp
+++ b/tests/spec/arb_sample_shading/execution/builtin-gl-num-samples.cpp
@@ -118,7 +118,7 @@ piglit_init(int argc, char **argv)
        }
  }

-bool test_builtin_num_samples(Fbo ms_fbo)
+bool test_builtin_num_samples(const Fbo& ms_fbo)
  {
        GLint samples;
          GLfloat expected[4] = {0.0, 1.0, 0.0, 1.0};


For both arb_sample_shading patches,
Reviewed-by: Brian Paul <bri...@vmware.com>

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to