[Piglit] [PATCH 06/25] arb_direct_state_access: Use piglit_reset_gl_error in texunits
Pohjolainen, Topi
topi.pohjolainen at intel.com
Wed May 20 01:09:46 PDT 2015
On Mon, May 18, 2015 at 01:49:59PM -0700, Ian Romanick wrote:
> From: Ian Romanick <ian.d.romanick at intel.com>
>
> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
This is also:
Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>
Unfortunately I don't have much input for the rest of the patches
in the series.
> ---
> tests/spec/arb_direct_state_access/texunits.c | 16 ++++------------
> 1 file changed, 4 insertions(+), 12 deletions(-)
>
> diff --git a/tests/spec/arb_direct_state_access/texunits.c b/tests/spec/arb_direct_state_access/texunits.c
> index 54bd886..d4710e3 100644
> --- a/tests/spec/arb_direct_state_access/texunits.c
> +++ b/tests/spec/arb_direct_state_access/texunits.c
> @@ -91,20 +91,12 @@ report4v(const GLfloat exp[4], const GLfloat act[4])
> }
>
>
> -static void
> -clear_errors()
> -{
> - while (glGetError())
> - ;
> -}
> -
> -
> static bool
> test_rasterpos(void)
> {
> int i;
>
> - clear_errors();
> + piglit_reset_gl_error();
>
> /* set current texcoords */
> for (i = 0; i < MaxTextureCoordUnits; i++) {
> @@ -169,7 +161,7 @@ test_texture_matrix(void)
> {
> int i;
>
> - clear_errors();
> + piglit_reset_gl_error();
>
> /* set tex matrices */
> for (i = 0; i < MaxTextureCoordUnits; i++) {
> @@ -221,7 +213,7 @@ test_texture_params(void)
> int i;
> int maxUnit;
>
> - clear_errors();
> + piglit_reset_gl_error();
>
> glCreateTextures(GL_TEXTURE_2D, MaxTextureCombinedUnits, tex);
>
> @@ -269,7 +261,7 @@ test_texture_env(void)
> /* Texture Environment state is fixed-function; not used by shaders */
> int i;
>
> - clear_errors();
> + piglit_reset_gl_error();
>
> /* set per-unit state */
> for (i = 0; i < MaxTextureCombinedUnits; i++) {
> --
> 2.1.0
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
More information about the Piglit
mailing list