aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Liebler <stli@linux.vnet.ibm.com>2014-12-12 11:13:09 +0100
committerAndreas Krebbel <krebbel@linux.vnet.ibm.com>2014-12-12 11:14:00 +0100
commit9d9c0019e750b6ef2f6d941e40a4b959c736653c (patch)
tree5a9df2c0f1785f65d4a0c1c2e44ab9a355bbdc03
parent9173840b4dbcd1a76993f22fd9595f0ae30d5a42 (diff)
downloadglibc-9d9c0019e750b6ef2f6d941e40a4b959c736653c.tar
glibc-9d9c0019e750b6ef2f6d941e40a4b959c736653c.tar.gz
glibc-9d9c0019e750b6ef2f6d941e40a4b959c736653c.tar.bz2
glibc-9d9c0019e750b6ef2f6d941e40a4b959c736653c.zip
Get rid of warning comparision will always evaluate as true
-rw-r--r--ChangeLog9
-rw-r--r--nptl/tst-mutex6.c8
-rw-r--r--nptl/tst-mutexpp6.c1
3 files changed, 15 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index 53995f9857..9f18f4e95e 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 292e3bf06d..9c11a20632 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 2ddf6b45c6..87a6ebd74c 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"