[PATCH] D51036: clang-format: Fix formatting C++ namespaces with preceding 'inline' or 'export' specifier

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 24 01:30:43 PDT 2018


sammccall added inline comments.


================
Comment at: unittests/Format/FormatTest.cpp:7582
                Style);
+  verifyFormat("export namespace Foo\n"
+               "{};",
----------------
you may want to add tests for other modules TS syntax (e.g. non-namespace export decls).
It seems this works well today, but without tests it could regress.


Repository:
  rC Clang

https://reviews.llvm.org/D51036





More information about the cfe-commits mailing list