[Piglit] [PATCH 4/4] util: change default window size to 160 x 160

Daniel Vetter daniel at ffwll.ch
Wed May 7 04:27:27 PDT 2014


On Fri, May 02, 2014 at 03:53:48PM -0600, Brian Paul wrote:
> To avoid "Got spurious window resize" warnings and SKIP result
> on Windows 8.  A while back we had changed the default window
> size for Windows 7.
> ---
>  tests/util/piglit-framework-gl.h |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/util/piglit-framework-gl.h b/tests/util/piglit-framework-gl.h
> index 9824372..d0de8e8 100644
> --- a/tests/util/piglit-framework-gl.h
> +++ b/tests/util/piglit-framework-gl.h
> @@ -269,8 +269,8 @@ piglit_gl_test_run(int argc, char *argv[],
>                  /* seems to be 116 pixels.  When the window size is */       \
>                  /* unexpectedly resized, tests are marked as "WARN". */      \
>                  /* Let's use a larger default to avoid that. */              \

Shouldn't the comment be updated since now the limit isn't 116 pixels any
more, at least on Win8?
-Daniel

> -                config.window_width = 150;                                   \
> -                config.window_height = 150;                                  \
> +                config.window_width = 160;                                   \
> +                config.window_height = 160;                                  \
>                                                                               \
>                  /* Open a new scope so that tests can declare locals */      \
>                  /* between here and PIGLIT_GL_TEST_CONFIG_END. */            \
> -- 
> 1.7.10.4
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch


More information about the Piglit mailing list