[PATCH] D51036: clang-format: Fix formatting C++ namespaces with preceding 'inline' or 'export' specifier
Owen Pan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 24 02:50:48 PDT 2018
owenpan added inline comments.
================
Comment at: lib/Format/UnwrappedLineFormatter.cpp:533-535
+ !(Line.startsWith(tok::kw_namespace) ||
+ Line.startsWith(tok::kw_inline, tok::kw_namespace) ||
+ Line.startsWith(tok::kw_export, tok::kw_namespace)) &&
----------------
Maybe add a test case (or modify an existing one) for this fix, with a C/C++ style comment before `namespace`, `inline`, and/or `export`?
Repository:
rC Clang
https://reviews.llvm.org/D51036
More information about the cfe-commits
mailing list