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",
----------------
klimek wrote:
> 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).
Done


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

Reply via email to