[Piglit] [PATCH 2/2] util/wfl: Re-enable GL 3.1 log messages in debug paths

Chad Versace chad.versace at linux.intel.com
Mon Jun 30 17:11:15 PDT 2014


Commit 1fdb953 removed some noisy debug messages with preprocessor
guards. This patch re-enables them, but only if the environment variable
PIGLIT_DEBUG is "1" or "true".

Fixes some unused variable warnings.

Signed-off-by: Chad Versace <chad.versace at linux.intel.com>
---
 tests/util/piglit-framework-gl/piglit_wfl_framework.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/tests/util/piglit-framework-gl/piglit_wfl_framework.c b/tests/util/piglit-framework-gl/piglit_wfl_framework.c
index a63f809..818625b 100644
--- a/tests/util/piglit-framework-gl/piglit_wfl_framework.c
+++ b/tests/util/piglit-framework-gl/piglit_wfl_framework.c
@@ -438,11 +438,9 @@ special_case_gl31(struct piglit_wfl_framework *wfl_fw,
 		 * promise that the returned context will have the requested
 		 * profile.  So Piglit has no need to check the profile here.
 		 */
-#if 0
-		piglit_logi("Requested an %s, and received a matching "
+		piglit_logd("Requested an %s, and received a matching "
 			    "%d.%d context\n", context_description,
 			    actual_gl_version / 10, actual_gl_version % 10);
-#endif
 		return true;
 	}
 
@@ -453,14 +451,12 @@ special_case_gl31(struct piglit_wfl_framework *wfl_fw,
 		return true;
 	}
 
-#if 0
-	piglit_logi("Requested an %s, and the returned context is exactly a 3.1 "
+	piglit_logd("Requested an %s, and the returned context is exactly a 3.1 "
 		    "context. But it has the wrong profile because it %s the "
 		    "GL_ARB_compatibility extension. Fallback to requesting a "
 		    "3.2 context, which is guaranteed to have the correct "
 		    "profile if context creation succeeds.",
 		    context_description, error_verb);
-#endif
 
 	waffle_config_destroy(wfl_fw->config);
 	waffle_context_destroy(wfl_fw->context);
-- 
2.0.0



More information about the Piglit mailing list