diff options
| author | Stepan Kasal <skasal@redhat.com> | 2009-11-23 12:14:06 +0100 |
|---|---|---|
| committer | Rafael Garcia-Suarez <rgs@consttype.org> | 2009-11-23 12:14:06 +0100 |
| commit | fabb67aac748c1bf94bbc4a173a67dbd9a00b4df (patch) | |
| tree | 611e1cca1d86de2018bfcaf7e506a41622478b2a /ext | |
| parent | 1cfccccd91dcb8384e07fecd6d80c2f1956492ac (diff) | |
| download | perl-fabb67aac748c1bf94bbc4a173a67dbd9a00b4df.tar.gz | |
fix logic for a workaround in POSIX.xs
There is the assignment "RETVAL = 0" to silence the compiler in case
it incorrectly reports return of an undefined value.
This assignment should be present whenever *any* of the symbols is
not defined. But the code was written so that the assignment was
compiled only when *none* of the symbols was defined.
Diffstat (limited to 'ext')
| -rw-r--r-- | ext/POSIX/POSIX.xs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ext/POSIX/POSIX.xs b/ext/POSIX/POSIX.xs index d30a7529b6..7bdd6339b7 100644 --- a/ext/POSIX/POSIX.xs +++ b/ext/POSIX/POSIX.xs @@ -887,8 +887,8 @@ WEXITSTATUS(status) POSIX::WSTOPSIG = 4 POSIX::WTERMSIG = 5 CODE: -#if !(defined(WEXITSTATUS) || defined(WIFEXITED) || defined(WIFSIGNALED) \ - || defined(WIFSTOPPED) || defined(WSTOPSIG) || defined (WTERMSIG)) +#if !defined(WEXITSTATUS) || !defined(WIFEXITED) || !defined(WIFSIGNALED) \ + || !defined(WIFSTOPPED) || !defined(WSTOPSIG) || !defined(WTERMSIG) RETVAL = 0; /* Silence compilers that notice this, but don't realise that not_here() can't return. */ #endif |