diff options
author | Francois Ferrand <thetypz@gmail.com> | 2018-10-02 16:37:51 +0000 |
---|---|---|
committer | Francois Ferrand <thetypz@gmail.com> | 2018-10-02 16:37:51 +0000 |
commit | 6f40e21a1601aa1b83a13282918ddae47e58b7d5 (patch) | |
tree | e913d7ae400a1074b73dee48d3a0f01eb48911de /clang/lib/Format/UnwrappedLineParser.h | |
parent | 59500f7a0b8a492fec8b571a4a47cf163204247e (diff) | |
download | bcm5719-llvm-6f40e21a1601aa1b83a13282918ddae47e58b7d5.tar.gz bcm5719-llvm-6f40e21a1601aa1b83a13282918ddae47e58b7d5.zip |
clang-format: better handle statement macros
Summary:
Some macros are used in the body of function, and actually contain the trailing semicolon: they should thus be automatically followed by a new line, and not get merged with the next line. This is for example the case with Qt's Q_UNUSED macro:
void foo(int a, int b) {
Q_UNUSED(a)
return b;
}
This patch deals with these cases by introducing a new option to specify list of statement macros. This re-uses the system already in place for foreach macros, to ensure there is no impact on performance.
Reviewers: krasimir, djasper, klimek
Reviewed By: krasimir
Subscribers: acoomans, mgrang, alexfh, klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D33440
llvm-svn: 343602
Diffstat (limited to 'clang/lib/Format/UnwrappedLineParser.h')
-rw-r--r-- | clang/lib/Format/UnwrappedLineParser.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/clang/lib/Format/UnwrappedLineParser.h b/clang/lib/Format/UnwrappedLineParser.h index 87254832c63..55d60dff915 100644 --- a/clang/lib/Format/UnwrappedLineParser.h +++ b/clang/lib/Format/UnwrappedLineParser.h @@ -126,6 +126,7 @@ private: void parseObjCInterfaceOrImplementation(); bool parseObjCProtocol(); void parseJavaScriptEs6ImportExport(); + void parseStatementMacro(); bool tryToParseLambda(); bool tryToParseLambdaIntroducer(); void tryToParseJSFunction(); |