From: Stefan Liebler <stli@linux.vnet.ibm.com>
Date: Fri, 12 Dec 2014 10:13:09 +0000 (+0100)
Subject: Get rid of warning comparision will always evaluate as true
X-Git-Tag: glibc-2.21~238
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9d9c0019e750b6ef2f6d941e40a4b959c736653c;p=thirdparty%2Fglibc.git

Get rid of warning comparision will always evaluate as true
---

diff --git a/ChangeLog b/ChangeLog
index 53995f98571..9f18f4e95e4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2014-12-12  Stefan Liebler  <stli@linux.vnet.ibm.com>
+
+	* nptl/tst-mutex6.c
+	(ATTR_NULL): New define checks ATTR against NULL.
+	(do_test): Use !ATTR_NULL instead of ATTR != NULL.
+	* nptl/tst-mutexpp6.c (ATTR_NULL): New define.
+
 2014-12-11  James Lemke  <jwlemke@codesourcery.com>
 
 	[BZ #17581]
@@ -117,7 +124,7 @@
 
 2014-12-11  Stefan Liebler  <stli@linux.vnet.ibm.com>
 
-	sysdeps/s390/s390-32/dl-machine.h (elf_machine_load_address):
+	* sysdeps/s390/s390-32/dl-machine.h (elf_machine_load_address):
 	Truncating assembler expression to a .long expression.
 
 2014-12-11  Andreas Schwab  <schwab@suse.de>
diff --git a/nptl/tst-mutex6.c b/nptl/tst-mutex6.c
index 292e3bf06df..9c11a20632d 100644
--- a/nptl/tst-mutex6.c
+++ b/nptl/tst-mutex6.c
@@ -21,6 +21,7 @@
 #include <stdio.h>
 #include <unistd.h>
 #include <errno.h>
+#include <stdbool.h>
 
 
 #ifndef ATTR
@@ -28,6 +29,9 @@ pthread_mutexattr_t *attr;
 # define ATTR attr
 #endif
 
+#ifndef ATTR_NULL
+# define ATTR_NULL (ATTR == NULL)
+#endif
 
 static int
 do_test (void)
@@ -35,7 +39,7 @@ do_test (void)
   pthread_mutex_t m;
 
   int e = pthread_mutex_init (&m, ATTR);
-  if (ATTR != NULL && e == ENOTSUP)
+  if (!ATTR_NULL && e == ENOTSUP)
     {
       puts ("cannot support selected type of mutexes");
       e = pthread_mutex_init (&m, NULL);
@@ -46,7 +50,7 @@ do_test (void)
       return 1;
     }
 
-  if (ATTR != NULL && pthread_mutexattr_destroy (ATTR) != 0)
+  if (!ATTR_NULL && pthread_mutexattr_destroy (ATTR) != 0)
     {
       puts ("mutexattr_destroy failed");
       return 1;
diff --git a/nptl/tst-mutexpp6.c b/nptl/tst-mutexpp6.c
index 2ddf6b45c68..87a6ebd74c6 100644
--- a/nptl/tst-mutexpp6.c
+++ b/nptl/tst-mutexpp6.c
@@ -42,4 +42,5 @@ do_test_wrapper (void)
 #define TEST_FUNCTION do_test_wrapper ()
 
 #define ATTR &a
+#define ATTR_NULL false
 #include "tst-mutex6.c"