[Piglit] default piglit window size change
Brian Paul
brianp at vmware.com
Fri Nov 2 09:32:05 PDT 2012
I think I've brought this up in the past. On Windows, the minimum
width of a window is 116 pixels (at least in my current Win7 setup).
A lot of piglit tests create 100x100 pixels and when piglit creates
the window, Win7 resizes it to 116x100. The framework code detects
this change and reports:
Got spurious window resize in automatic run (100,100 to 116,100)
PIGLIT: {'result': 'warn'}
So a bunch of tests get the 'warn' result instead of being run
properly. I think this changed at some point during the recent
framework/waffle overhaul.
My proposal for this is to set a default window size of 150x150 in the
piglit-framework-gl.h header:
--- a/tests/util/piglit-framework-gl.h
+++ b/tests/util/piglit-framework-gl.h
@@ -216,6 +216,13 @@ piglit_gl_test_run(int argc, char *argv[],
\
config.init = piglit_init;
\
config.display = piglit_display;
\
+
\
+ /* Default window size. Note: Win7's min window
width */ \
+ /* 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. */
\
+ config.window_width = 150;
\
+ config.window_height = 150;
#define PIGLIT_GL_TEST_CONFIG_END
\
And then remove these config lines from a bunch of tests:
config.window_width = 100;
config.window_height = 100;
Of course, I'll have to inspect these tests to make sure they don't
rely on that specific window size, but I bet few if any of them do.
This change fixes things here.
Sound OK?
-Brian
More information about the Piglit
mailing list