[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