Commit 4c24f7faa8bc71589538e8b07ea8b64c84c11b5c

Ran Benita 2016-03-15T20:42:21

test: assert/ignore some warn_unused_result's Signed-off-by: Ran Benita <ran234@gmail.com>

diff --git a/bench/bench.c b/bench/bench.c
index 9bb535c..37e41f5 100644
--- a/bench/bench.c
+++ b/bench/bench.c
@@ -33,6 +33,7 @@
 #include <sys/time.h>
 #endif
 
+#include <assert.h>
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
@@ -77,7 +78,7 @@ bench_timer_start(struct bench_timer *self)
 #if defined(USE_CLOCK_GETTIME)
     struct timespec val;
 
-    (void)clock_gettime(CLOCK_MONOTONIC, &val);
+    (void) clock_gettime(CLOCK_MONOTONIC, &val);
 
     /* With conversion from nanosecond to millisecond */
     set_bench_time(&self->start, val.tv_sec, val.tv_nsec / 1000);
@@ -93,7 +94,7 @@ bench_timer_start(struct bench_timer *self)
 #else
     struct timeval val;
 
-    (void)gettimeofday(&val, NULL);
+    (void) gettimeofday(&val, NULL);
 
     set_bench_time(&self->start, val.tv_sec, val.tv_usec);
 #endif
@@ -105,7 +106,7 @@ bench_timer_stop(struct bench_timer *self)
 #if defined(USE_CLOCK_GETTIME)
     struct timespec val;
 
-    (void)clock_gettime(CLOCK_MONOTONIC, &val);
+    (void) clock_gettime(CLOCK_MONOTONIC, &val);
 
     /* With conversion from nanosecond to millisecond */
     set_bench_time(&self->stop, val.tv_sec, val.tv_nsec / 1000);
@@ -121,7 +122,7 @@ bench_timer_stop(struct bench_timer *self)
 #else
     struct timeval val;
 
-    (void)gettimeofday(&val, NULL);
+    (void) gettimeofday(&val, NULL);
 
     set_bench_time(&self->stop, val.tv_sec, val.tv_usec);
 #endif
@@ -139,10 +140,12 @@ bench_timer_get_elapsed_time_str(struct bench_timer *self)
 {
     struct bench_time elapsed;
     char *buf;
+    int ret;
 
     bench_timer_get_elapsed_time(self, &elapsed);
     normalize_bench_time(&elapsed);
-    asprintf(&buf, "%ld.%06ld", elapsed.seconds, elapsed.milliseconds);
+    ret = asprintf(&buf, "%ld.%06ld", elapsed.seconds, elapsed.milliseconds);
+    assert(ret >= 0);
 
     return buf;
 }
diff --git a/test/interactive-evdev.c b/test/interactive-evdev.c
index 04a347f..7fdc069 100644
--- a/test/interactive-evdev.c
+++ b/test/interactive-evdev.c
@@ -483,14 +483,14 @@ main(int argc, char *argv[])
     sigaction(SIGTERM, &act, NULL);
 
     /* Instead of fiddling with termios.. */
-    system("stty -echo");
+    (void) system("stty -echo");
 
     ret = loop(kbds);
     if (ret)
         goto err_stty;
 
 err_stty:
-    system("stty echo");
+    (void) system("stty echo");
     free_keyboards(kbds);
 err_compose:
     xkb_compose_table_unref(compose_table);
diff --git a/test/interactive-x11.c b/test/interactive-x11.c
index d720004..34ad654 100644
--- a/test/interactive-x11.c
+++ b/test/interactive-x11.c
@@ -372,9 +372,9 @@ main(int argc, char *argv[])
         goto err_core_kbd;
     }
 
-    system("stty -echo");
+    (void) system("stty -echo");
     ret = loop(conn, &core_kbd);
-    system("stty echo");
+    (void) system("stty echo");
 
 err_core_kbd:
     deinit_kbd(&core_kbd);