mirror of
https://sourceware.org/git/glibc.git
synced 2024-11-21 20:40:05 +00:00
Fix -Waddress warnings in nptl/tst-mutex1.c.
This patch fixes -Waddress warnings in nptl/tst-mutex1.c from comparing the address of an object with NULL (ATTR may either be NULL, or the address of an object when included from other tests, and the warning arises in the latter case). A macro ATTR_NULL is defined alongside ATTR and used for the tests. Tested for x86_64. * nptl/tst-mutex1.c: Include <stdbool.h>. [!ATTR] (ATTR_NULL): New macro. (do_test): Test !ATTR_NULL instead of ATTR != NULL. * nptl/tst-mutexpi1.c (ATTR_NULL): New macro. * nptl/tst-mutexpp1.c (ATTR_NULL): New macro.
This commit is contained in:
parent
b1aff6a411
commit
4e0b901601
@ -1,5 +1,11 @@
|
|||||||
2014-12-02 Joseph Myers <joseph@codesourcery.com>
|
2014-12-02 Joseph Myers <joseph@codesourcery.com>
|
||||||
|
|
||||||
|
* nptl/tst-mutex1.c: Include <stdbool.h>.
|
||||||
|
[!ATTR] (ATTR_NULL): New macro.
|
||||||
|
(do_test): Test !ATTR_NULL instead of ATTR != NULL.
|
||||||
|
* nptl/tst-mutexpi1.c (ATTR_NULL): New macro.
|
||||||
|
* nptl/tst-mutexpp1.c (ATTR_NULL): New macro.
|
||||||
|
|
||||||
* posix/tst-getopt_long1.c (do_test): Cast elements of argv array
|
* posix/tst-getopt_long1.c (do_test): Cast elements of argv array
|
||||||
to char *.
|
to char *.
|
||||||
|
|
||||||
|
@ -19,10 +19,12 @@
|
|||||||
#include <pthread.h>
|
#include <pthread.h>
|
||||||
#include <stdio.h>
|
#include <stdio.h>
|
||||||
#include <errno.h>
|
#include <errno.h>
|
||||||
|
#include <stdbool.h>
|
||||||
|
|
||||||
|
|
||||||
#ifndef ATTR
|
#ifndef ATTR
|
||||||
# define ATTR NULL
|
# define ATTR NULL
|
||||||
|
# define ATTR_NULL true
|
||||||
#endif
|
#endif
|
||||||
|
|
||||||
|
|
||||||
@ -32,7 +34,7 @@ do_test (void)
|
|||||||
pthread_mutex_t m;
|
pthread_mutex_t m;
|
||||||
|
|
||||||
int e = pthread_mutex_init (&m, ATTR);
|
int e = pthread_mutex_init (&m, ATTR);
|
||||||
if (ATTR != NULL && e == ENOTSUP)
|
if (!ATTR_NULL && e == ENOTSUP)
|
||||||
{
|
{
|
||||||
puts ("cannot support selected type of mutexes");
|
puts ("cannot support selected type of mutexes");
|
||||||
return 0;
|
return 0;
|
||||||
@ -43,7 +45,7 @@ do_test (void)
|
|||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ATTR != NULL && pthread_mutexattr_destroy (ATTR) != 0)
|
if (!ATTR_NULL && pthread_mutexattr_destroy (ATTR) != 0)
|
||||||
{
|
{
|
||||||
puts ("mutexattr_destroy failed");
|
puts ("mutexattr_destroy failed");
|
||||||
return 1;
|
return 1;
|
||||||
|
@ -24,4 +24,5 @@ prepare (void)
|
|||||||
|
|
||||||
|
|
||||||
#define ATTR &a
|
#define ATTR &a
|
||||||
|
#define ATTR_NULL false
|
||||||
#include "tst-mutex1.c"
|
#include "tst-mutex1.c"
|
||||||
|
@ -42,4 +42,5 @@ do_test_wrapper (void)
|
|||||||
#define TEST_FUNCTION do_test_wrapper ()
|
#define TEST_FUNCTION do_test_wrapper ()
|
||||||
|
|
||||||
#define ATTR &a
|
#define ATTR &a
|
||||||
|
#define ATTR_NULL false
|
||||||
#include "tst-mutex1.c"
|
#include "tst-mutex1.c"
|
||||||
|
Loading…
Reference in New Issue
Block a user