summaryrefslogtreecommitdiff
path: root/include/wchar.h
diff options
context:
space:
mode:
authorA. Wilcox2018-06-23 17:57:48 -0500
committersin2018-07-24 11:00:30 +0100
commita9ffac8596b094da8563aa5dd5d81c946670afe5 (patch)
tree7e8ab469005400603d788c1291fdd6dfb5ba1b4e /include/wchar.h
parent6e7e43ff992fa232b186cec9fc2920ca260c27a6 (diff)
Don't use __extension__ in C++ code
A few important notes: * __extension__ is a GNU C "alternate" keyword, not a C++ keyword.[1] * __extension__ is designed to work on "expressions"; it does work on #include_next in C mode, but it has no effect in C++ mode; the warning will still appear, if enabled, even with __extension__ preceding #include_next. This is because #include_next is not considered an expression in C++, so the compiler attaches __extension__ to the first expression of the header. All of this leads us to a build failure while building at least all Mozilla software. Moz has an alternate -isystem dir searched before /usr/include that overrides some headers, including <features.h>. The first statement in each of these headers is a #pragma, and since __extension__ is looking for an expression, and #pragma is a "null" expression, we end up with the following error: dist/system_wrappers/features.h:1:9: error: '#pragma' is not allowed here Since __extension__ has no effect on #include_next in C++ mode anyway, and since it can cause breakage, this commit omits __extension__ in C++ mode. [1]: https://gcc.gnu.org/onlinedocs/gcc-6.4.0/gcc/Alternate-Keywords.html
Diffstat (limited to 'include/wchar.h')
-rw-r--r--include/wchar.h8
1 files changed, 8 insertions, 0 deletions
diff --git a/include/wchar.h b/include/wchar.h
index 7394598..3cb6f92 100644
--- a/include/wchar.h
+++ b/include/wchar.h
@@ -16,13 +16,21 @@
16#ifndef _FORTIFY_WCHAR_H 16#ifndef _FORTIFY_WCHAR_H
17#define _FORTIFY_WCHAR_H 17#define _FORTIFY_WCHAR_H
18 18
19#ifndef __cplusplus
19__extension__ 20__extension__
21#endif
20#include_next <limits.h> 22#include_next <limits.h>
23#ifndef __cplusplus
21__extension__ 24__extension__
25#endif
22#include_next <stdlib.h> 26#include_next <stdlib.h>
27#ifndef __cplusplus
23__extension__ 28__extension__
29#endif
24#include_next <string.h> 30#include_next <string.h>
31#ifndef __cplusplus
25__extension__ 32__extension__
33#endif
26#include_next <wchar.h> 34#include_next <wchar.h>
27 35
28#if defined(_FORTIFY_SOURCE) && _FORTIFY_SOURCE > 0 && defined(__OPTIMIZE__) && __OPTIMIZE__ > 0 36#if defined(_FORTIFY_SOURCE) && _FORTIFY_SOURCE > 0 && defined(__OPTIMIZE__) && __OPTIMIZE__ > 0