summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorPaulo Zanoni <paulo.r.zanoni@intel.com>2015-08-13 13:49:42 -0300
committerPaulo Zanoni <paulo.r.zanoni@intel.com>2015-08-14 15:26:28 -0300
commita1fce7490c72c45c79cff89a39dee0b59a8f4c71 (patch)
treeecce15515ffb4a76295adb12f90daaeb04a0ee4b /lib
parent87a5f2d7762b5b8398c5f5899d796bb78dc827b7 (diff)
lib/igt_core: use print("%s", #expr) instead of print(#expr)
If I have a program with the following: igt_skip_on(i % 2 == 0); igt_skip_on_f(i % 2 == 0, "i:%d\n", i); igt_require(i % 2 == 0); igt_require_f(i % 2 == 0, "i:%d\n", i); then I'll get compiler error messages complaining about format conversions related to the '%' character used in the mod operation. So put the whole string as a %s argument to avoid interpreting '%' and any other possible chars. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/igt_core.h8
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/igt_core.h b/lib/igt_core.h
index 1d77f452..f8bfbf01 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -491,7 +491,7 @@ void igt_exit(void) __attribute__((noreturn));
*/
#define igt_require(expr) do { \
if (!(expr)) igt_skip_check(#expr , NULL); \
- else igt_debug("Test requirement passed: "#expr"\n"); \
+ else igt_debug("Test requirement passed: %s\n", #expr); \
} while (0)
/**
@@ -506,7 +506,7 @@ void igt_exit(void) __attribute__((noreturn));
*/
#define igt_skip_on(expr) do { \
if ((expr)) igt_skip_check("!(" #expr ")" , NULL); \
- else igt_debug("Test requirement passed: !("#expr")\n"); \
+ else igt_debug("Test requirement passed: !(%s)\n", #expr); \
} while (0)
/**
@@ -525,7 +525,7 @@ void igt_exit(void) __attribute__((noreturn));
*/
#define igt_require_f(expr, f...) do { \
if (!(expr)) igt_skip_check(#expr , f); \
- else igt_debug("Test requirement passed: "#expr"\n"); \
+ else igt_debug("Test requirement passed: %s\n", #expr); \
} while (0)
/**
@@ -544,7 +544,7 @@ void igt_exit(void) __attribute__((noreturn));
*/
#define igt_skip_on_f(expr, f...) do { \
if ((expr)) igt_skip_check("!("#expr")", f); \
- else igt_debug("Test requirement passed: !("#expr")\n"); \
+ else igt_debug("Test requirement passed: !(%s)\n", #expr); \
} while (0)
/* fork support code */