summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2018-10-09 08:43:08 +0200
committerSebastian Huber <sebastian.huber@embedded-brains.de>2018-10-09 08:48:25 +0200
commitacb53d53c900f21b5940554dfba8d7ad94f4bcba (patch)
treed32d39814572bf671afdb946a219ecc99d266480
parentftpd: Remove superfluous temporary buffer (diff)
downloadrtems-acb53d53c900f21b5940554dfba8d7ad94f4bcba.tar.bz2
posix: Remove static assertion
Standard C does not allow pointer operands in an integer constant expressions. GCC issues only a -Wpedantic warning in the removed static assertion. However, clang is pedantic by default and issues an error. Rely on the test case instead.
-rw-r--r--cpukit/posix/src/seminit.c2
-rw-r--r--testsuites/psxtests/psxsem01/init.c1
2 files changed, 1 insertions, 2 deletions
diff --git a/cpukit/posix/src/seminit.c b/cpukit/posix/src/seminit.c
index 65104ad2c7..e752a2752b 100644
--- a/cpukit/posix/src/seminit.c
+++ b/cpukit/posix/src/seminit.c
@@ -22,8 +22,6 @@
#include <limits.h>
-RTEMS_STATIC_ASSERT(NULL == SEM_FAILED, sem_failed);
-
/*
* 11.2.1 Initialize an Unnamed Semaphore, P1003.1b-1993, p.219
*/
diff --git a/testsuites/psxtests/psxsem01/init.c b/testsuites/psxtests/psxsem01/init.c
index dfc2ed5670..899e324fa0 100644
--- a/testsuites/psxtests/psxsem01/init.c
+++ b/testsuites/psxtests/psxsem01/init.c
@@ -193,6 +193,7 @@ static void test_sem_null(void)
int val;
struct timespec to;
+ /* This equality is important for POSIX_SEMAPHORE_VALIDATE_OBJECT() */
rtems_test_assert( NULL == SEM_FAILED );
errno = 0;