Fix build warnings in nptl/tst-eintr1.c
authorStefan Liebler <stli@linux.ibm.com>
Wed, 26 Jun 2019 10:29:50 +0000 (12:29 +0200)
committerStefan Liebler <stli@linux.ibm.com>
Wed, 26 Jun 2019 10:29:50 +0000 (12:29 +0200)
This patch fixes the gcc warnings seen with gcc 9.1 -O3 on s390x:
tst-eintr1.c: In function ‘tf1’:
tst-eintr1.c:46:1: error: no return statement in function returning non-void [-Werror=return-type]
   46 | }
      | ^
tst-eintr1.c: In function ‘do_test’:
tst-eintr1.c:57:17: error: unused variable ‘th’ [-Werror=unused-variable]
   57 |       pthread_t th = xpthread_create (NULL, tf1, NULL);
      |                 ^~

ChangeLog:

* nptl/tst-eintr1.c (tf1): Add return statement.
(do_test): Remove unused th variable.

ChangeLog
nptl/tst-eintr1.c

index adc58e4a14a8c4e84e761f7f8d2842abd5578413..2e54659370faf49213efadca80bd6daec646bdf1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2019-06-26  Stefan Liebler  <stli@linux.ibm.com>
+
+       * nptl/tst-eintr1.c (tf1): Add return statement.
+       (do_test): Remove unused th variable.
+
 2019-06-26  Stefan Liebler  <stli@linux.ibm.com>
 
        * locale/programs/ld-ctype.c (charclass_symbolic_ellipsis):
index b60b796d619713f0e6f293974b2e0b85589b94e7..11404ff7ff5e76262fb87e076e7737bf8d7cbee7 100644 (file)
@@ -43,6 +43,7 @@ tf1 (void *arg)
       pthread_t th = xpthread_create (NULL, tf2, NULL);
       xpthread_join (th);
     }
+  return NULL;
 }
 
 
@@ -53,9 +54,7 @@ do_test (void)
 
   int i;
   for (i = 0; i < 10; ++i)
-    {
-      pthread_t th = xpthread_create (NULL, tf1, NULL);
-    }
+    xpthread_create (NULL, tf1, NULL);
 
   delayed_exit (3);
   /* This call must never return.  */