Typz marked 2 inline comments as done.
Typz added inline comments.
================
Comment at: unittests/Format/NamespaceEndCommentsFixerTest.cpp:526
EXPECT_EQ("namespace A {\n"
" int i;\n"
"} // namespace A",
----------------
Should I also fix these tests?
They already existed before this patch, but do not follow LLVM namespace indent
style either.
================
Comment at: unittests/Format/NamespaceEndCommentsFixerTest.cpp:582
+ EXPECT_EQ("TESTSUITE() {\n"
+ " int i;\n"
+ "} // TESTSUITE()",
----------------
klimek wrote:
> All of the fixNamespaceEndComments tests are indented, but standard llvm
> style doesn't indent in namespaces at all iiuc.
Ok, understood. I can fix that.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D37813/new/
https://reviews.llvm.org/D37813
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
