[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