From 7d4dc0456f4cb8233597a0e2d62dfe9b803dd56c Mon Sep 17 00:00:00 2001
From: Alexandre Abadie <alexandre.abadie@inria.fr>
Date: Tue, 7 Nov 2017 19:11:44 +0100
Subject: [PATCH] tests/thread_flags_xtimer: minor improvements in test
 application

---
 tests/thread_flags_xtimer/main.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/tests/thread_flags_xtimer/main.c b/tests/thread_flags_xtimer/main.c
index 2c20f19657..410d14caec 100644
--- a/tests/thread_flags_xtimer/main.c
+++ b/tests/thread_flags_xtimer/main.c
@@ -24,7 +24,7 @@
 #include "thread_flags.h"
 
 #define TIMEOUT         (100UL * US_PER_MS)
-#define REPEAT          (5)
+#define REPEAT          (5U)
 #define RUNTIME         (TIMEOUT * REPEAT)
 
 static void time_evt(void *arg)
@@ -34,30 +34,30 @@ static void time_evt(void *arg)
 
 int main(void)
 {
-
+    puts("START");
     xtimer_t timer;
     timer.callback = time_evt;
     timer.arg = (void *)sched_active_thread;
     uint32_t last = xtimer_now_usec();
 
-    puts("\nTest setting thread flags from (x)timer callback");
+    puts("Test setting thread flags from (x)timer callback");
     printf("You should see the message '+++ timeout XX +++' printed to the \n"
-           "screen %i times, once every %lu milliseconds\n\n",
+           "screen %u times, once every %lu milliseconds\n\n",
             REPEAT, (TIMEOUT / US_PER_MS));
 
-    for (int i = 1; i <= REPEAT; i++) {
+    for (unsigned i = 1; i <= REPEAT; i++) {
         xtimer_set(&timer, TIMEOUT);
         thread_flags_wait_any(0x1);
-        printf("+++ timeout %2i +++\n", i);
+        printf("+++ timeout %2u +++\n", i);
     }
 
     /* we consider the test successful, if the runtime was above 500ms */
     uint32_t runtime = xtimer_now_usec() - last;
     if (runtime > RUNTIME) {
-        puts("Test finished: SUCCESS\n");
+        puts("SUCCESS");
     }
     else {
-        puts("Test finished: FAILED\n");
+        puts("FAILURE");
     }
 
     return 0;