[PATCH] D37813: clang-format: better handle namespace macros
Manuel Klimek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 27 00:44:12 PDT 2019
klimek added inline comments.
================
Comment at: unittests/Format/NamespaceEndCommentsFixerTest.cpp:526
EXPECT_EQ("namespace A {\n"
" int i;\n"
"} // namespace A",
----------------
Typz wrote:
> Should I also fix these tests?
>
> They already existed before this patch, but do not follow LLVM namespace indent style either.
If you don't mind, that would be useful - doing it in a separate CL is fine, so we keep this one focused (and no review needed).
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D37813/new/
https://reviews.llvm.org/D37813
More information about the cfe-commits
mailing list