diff options
author | Jim Meyering <meyering@fb.com> | 2014-06-05 10:42:05 -0700 |
---|---|---|
committer | Jim Meyering <meyering@fb.com> | 2016-12-18 01:30:51 -0800 |
commit | e077349ce589466eecd47213db4fae6b80ec18c4 (patch) | |
tree | 3f8dc06410c398f69426bf115e643284a133436b /stdio-common/ftrylockfile.c | |
parent | 009ba649b4999ea269de8d0b301e659f075df495 (diff) | |
download | glibc-e077349ce589466eecd47213db4fae6b80ec18c4.tar.gz glibc-e077349ce589466eecd47213db4fae6b80ec18c4.tar.xz glibc-e077349ce589466eecd47213db4fae6b80ec18c4.zip |
assert.h: allow gcc to detect assert(a = 1) errors
* assert/assert.h (assert): Rewrite assert's definition so that a s/==/=/ typo, e.g., assert(errno = ENOENT) is not hidden from gcc's -Wparentheses by assert-added parentheses. The new definition uses "if (expr) /* empty */; else __assert_fail...", so gcc -Wall will now detect that type of error in an assert, too. The __STRICT_ANSI__ disjunct is to make this work also with both -ansi and -pedantic, which would reject the use of ({...}). I would have preferred to use __extension__ to mark that, but doing so would mistakenly suppress warnings about any extension in the user-supplied "expr". E.g., "assert ( ({1;}) )" must continue to evoke a warning.
Diffstat (limited to 'stdio-common/ftrylockfile.c')
0 files changed, 0 insertions, 0 deletions