[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