On 10/09/2012 06:26 PM, Marek Olšák wrote:
It doesn't test anything.
---
  tests/all.tests |    1 -
  1 file changed, 1 deletion(-)

diff --git a/tests/all.tests b/tests/all.tests
index 964c84c..2c99171 100644
--- a/tests/all.tests
+++ b/tests/all.tests
@@ -115,7 +115,6 @@ glean['rgbTriStrip'] = GleanTest('rgbTriStrip', res_dir)
  glean['scissor'] = GleanTest('scissor', res_dir)
  glean['shaderAPI'] = GleanTest('shaderAPI', res_dir)
  glean['stencil2'] = GleanTest('stencil2', res_dir)
-glean['teapot'] = GleanTest('teapot', res_dir)
  glean['texCombine'] = GleanTest('texCombine', res_dir)
  glean['texCube'] = GleanTest('texCube', res_dir)
  glean['texEnv'] = GleanTest('texEnv', res_dir)

I believe that teapot was added as a performance test (several glean tests are perf tests).

The focus of piglit has been correctness but I think it would be nice to develop a suite of performance measurement tests too. But that's a whole other discussion.

I'm fine with this patch though.

Reviewed-by: Brian Paul <[email protected]>
_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to