[Piglit] [PATCH 1/2] glx_oml_sync_control: report FAIL if glXGetMscRateOML returns an error

Daniel Kurtz djkurtz at chromium.org
Wed Feb 5 22:30:44 PST 2014


The glx_oml_sync_control does not parse parameters.  In particular, it
does not parse the "-auto" flag to set piglit_automatic.

So, when we return FAIL from draw(), it returns to the
piglit_glx_event_loop, which just continues to spin waiting for events.
Instead, just report the FAIL result, which quits the program.

Signed-off-by: Daniel Kurtz <djkurtz at chromium.org>
---
 tests/spec/glx_oml_sync_control/getmscrate.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/spec/glx_oml_sync_control/getmscrate.c b/tests/spec/glx_oml_sync_control/getmscrate.c
index 9f0a43b..c18ea49 100644
--- a/tests/spec/glx_oml_sync_control/getmscrate.c
+++ b/tests/spec/glx_oml_sync_control/getmscrate.c
@@ -41,7 +41,7 @@ draw(Display *dpy)
 
 	if (!glXGetMscRateOML(dpy, win, &numerator, &denominator)) {
 		printf("glXGetMscRateOML returned failure.\n");
-		return PIGLIT_FAIL;
+		piglit_report_result(PIGLIT_FAIL);
 	}
 
 	if (numerator == 0xDEADBEEF) {
-- 
1.8.5.3



More information about the Piglit mailing list