When printing the status of test is OK or FAIL, check if stdout is
attached to a terminal device. This way the output is not cluttered with
useless escape characters. Some use cases where we don't want colors:
$ ./sometest | less
$ ./sometest > a.log
Signed-off-by: Christian Babeux <christian.babeux@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
#define BRIGHT 1
#define GREEN 32
#define RED 31
#define BRIGHT 1
#define GREEN 32
#define RED 31
-#define PRINT_OK() printf("%c[%d;%dmOK%c[%dm\n", 0x1B, BRIGHT, GREEN, 0x1B, 0);
-#define PRINT_FAIL() printf("%c[%d;%dmFAIL%c[%dm\n", 0x1B, BRIGHT, RED, 0x1B, 0);
+#define PRINT_OK() \
+do { \
+ /* Check for color support */ \
+ if (isatty(STDOUT_FILENO)) { \
+ printf("%c[%d;%dmOK%c[%dm\n", 0x1B, BRIGHT, GREEN, 0x1B, 0); \
+ } else { \
+ printf("OK\n"); \
+ } \
+} while (0)
+
+#define PRINT_FAIL() \
+do { \
+ /* Check for color support */ \
+ if (isatty(STDOUT_FILENO)) { \
+ printf("%c[%d;%dmFAIL%c[%dm\n", 0x1B, BRIGHT, RED, 0x1B, 0); \
+ } else { \
+ printf("FAIL\n"); \
+ } \
+} while (0)