On Mon, 21 Nov 2011 16:27:57 +0800, Yuanhan Liu <yuanhan....@linux.intel.com> 
wrote:
> Add a draw-pixel-with-texture testcase to check if texture
> sampling is happened while drawing pixels by glDrawPixels.
> 
> v2: use piglit_probe_rect_rgba instead of just sampling a
>     set of pixels(comments from Eric)
> 
> Signed-off-by: Yuanhan Liu <yuanhan....@linux.intel.com>
> ---
>  tests/all.tests                         |    1 +
>  tests/general/CMakeLists.gl.txt         |    1 +
>  tests/general/draw-pixel-with-texture.c |   77 
> +++++++++++++++++++++++++++++++
>  3 files changed, 79 insertions(+), 0 deletions(-)
>  create mode 100644 tests/general/draw-pixel-with-texture.c
> 
> diff --git a/tests/all.tests b/tests/all.tests
> index 48ce2cb..851db11 100644
> --- a/tests/all.tests
> +++ b/tests/all.tests
> @@ -242,6 +242,7 @@ general['draw-elements-user'] = 
> PlainExecTest(['draw-elements', '-auto', 'user']
>  add_plain_test(general, 'draw-elements-vs-inputs')
>  add_plain_test(general, 'draw-instanced')
>  add_plain_test(general, 'draw-instanced-divisor')
> +add_plain_test(general, 'draw-pixel-with-texture')
>  add_plain_test(general, 'draw-vertices')
>  general['draw-vertices-user'] = PlainExecTest(['draw-vertices', '-auto', 
> 'user'])
>  add_plain_test(general, 'draw-vertices-half-float')

> diff --git a/tests/general/draw-pixel-with-texture.c 
> b/tests/general/draw-pixel-with-texture.c
> new file mode 100644
> index 0000000..73e04c5
> --- /dev/null
> +++ b/tests/general/draw-pixel-with-texture.c

> +     glutSwapBuffers();

Similarly to the other test, I think with a piglit_present_results()
here, you could make this an add_concurrent_test instead of
add_plain_test.  Other than that, looks great!

Reviewed-by: Eric Anholt <e...@anholt.net>

Attachment: pgpCr7tH4EmFX.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to