test: silence a Wmaybe-uninitialized report on ret variable
authorDjalal Harouni <tixxdz@opendz.org>
Sat, 4 Jan 2014 19:46:46 +0000 (20:46 +0100)
committerKay Sievers <kay@vrfy.org>
Sun, 5 Jan 2014 15:01:49 +0000 (19:01 +0400)
Silence the gcc Wmaybe-uninitialized report on ret variable, and avoid
an extra check since run_tests() already returns EXIT_FAILURE or
EXIT_SUCCESS

Signed-off-by: Djalal Harouni <tixxdz@opendz.org>
test/test-kdbus.c

index 42fb29a28c5b9722ee4a80e9bb0b4a6ce5571824..de1cefc12dbd31144242490cb3392b68a8b922f5 100644 (file)
@@ -1198,7 +1198,7 @@ static int arg_loop = 0;
 
 int main(int argc, char *argv[])
 {
-       int c, ret;
+       int c, ret = -EINVAL;
 
        enum {
                ARG_VERSION = 0x100,
@@ -1224,7 +1224,7 @@ int main(int argc, char *argv[])
 
                default:
                        printf("Unknown option code %c", c);
-                       return -EINVAL;
+                       return ret;
                }
        }
 
@@ -1235,5 +1235,5 @@ int main(int argc, char *argv[])
        for (c = 0; c < arg_count; c++)
                ret = run_tests();
 
-       return ret > 0 ? EXIT_FAILURE : EXIT_SUCCESS;
+       return ret;
 }