-
Bug
-
Resolution: Unresolved
-
P2: Important
-
None
-
5.12.7
-
None
-
f0039bd51 (dev), ef8019af7 (6.5), 8ac712968 (6.6)
There is at least one distribution that add GCC's C++ headers on CPLUS_INCLUDE_PATH.
When 'moc' encounters the _GLIBCXX_VISIBILITY macro, it trips and fails with a "Parse error before <token>" where token is the word just before _GLIBCXX_VISIBILITY.
An easy fix for this problem is:
diff --git a/src/tools/moc/main.cpp b/src/tools/moc/main.cpp --- a/src/tools/moc/main.cpp +++ b/src/tools/moc/main.cpp @@ -188,6 +188,7 @@ int runMoc(int argc, char **argv) dummyVariadicFunctionMacro.arguments += Symbol(0, PP_IDENTIFIER, "__VA_ARGS__"); pp.macros["__attribute__"] = dummyVariadicFunctionMacro; pp.macros["__declspec"] = dummyVariadicFunctionMacro; + pp.macros["_GLIBCXX_VISIBILITY"] = dummyVariadicFunctionMacro; QString filename; QString output
GLIBCXX_VISIBILITY is a preprocessor macro that expands to _attribute_ which is already ignored by moc.
Should I send this patch through Gerrit?
- relates to
-
-
- Reported
-
-
-
- Open
-
-
-
- Closed
-
-
-
- Closed
-
| For Gerrit Dashboard: QTBUG-83160 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 492580,3 | moc: handle "L" integer suffix | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 493093,2 | moc: handle "L" integer suffix | 6.6 | qt/qtbase | Status: MERGED | +2 | 0 |
| 493094,2 | moc: handle "L" integer suffix | 6.5 | qt/qtbase | Status: MERGED | +2 | 0 |
| 493134,1 | moc: handle "L" integer suffix | tqtc/lts-6.2 | qt/tqtc-qtbase | Status: ABANDONED | -1 | 0 |