1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

moc: handle "L" integer suffix

Browse Source 
This commit adds some initial support for handling the 'L' suffix after
numbers. This one is especially important given that the __cplusplus
define is using it.
Other suffixes will be handled in some later commit, which should also
unify the already divergent parse behavior between DIGIT and PP_DIGIT
parsing (e.g. when it comes to byte prefixes).

Task-number: QTBUG-83160
Task-number: QTBUG-115558
Change-Id: Ie61eae49c468abfaee80e7e4f7097917a254dc0e
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
(cherry picked from commit f0039bd5170ad84d972a023316e8d153b89f841a)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
This commit is contained in:
Fabian Kosmale 2023-07-25 10:33:25 +02:00 committed by Qt Cherry-pick Bot
parent 9758650a37
commit 8ac712968a
2 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/tools/moc/preprocessor.cpp
View File

@ -223,7 +223,8 @@ Symbols Preprocessor::tokenize(const QByteArray& input, int lineNum, Preprocesso
++data;
while (is_hex_char(*data) || *data == '\'')
++data;
}
} else if (*data == 'L') // TODO: handle other suffixes
++data;
break;
}
token = FLOATING_LITERAL;
@ -400,7 +401,8 @@ Symbols Preprocessor::tokenize(const QByteArray& input, int lineNum, Preprocesso
++data;
while (is_hex_char(*data) || *data == '\'')
++data;
}
} else if (*data == 'L') // TODO: handle other suffixes
++data;
break;
}
token = PP_FLOATING_LITERAL;
@ -930,7 +932,11 @@ int PP_Expression::primary_expression()
test(PP_RPAREN);
} else {
next();
value = lexem().toInt(nullptr, 0);
const QByteArray &lex = lexem();
auto lexView = QByteArrayView(lex);
if (lex.endsWith('L'))
lexView.chop(1);
value = lexView.toInt(nullptr, 0);
}
return value;
}

5
tests/auto/tools/moc/tst_moc.cpp
View File

@ -617,6 +617,10 @@ private slots:
QT_WARNING_POP
// quick test to verify that moc handles the L suffix
// correctly in the preprocessor
#if 2000L < 1
#else
class PropertyTestClass : public QObject
{
Q_OBJECT
@ -626,6 +630,7 @@ public:
Q_ENUM(TestEnum)
};
#endif
class PropertyUseClass : public QObject
{