ioeric accepted this revision. ioeric added a comment. This revision is now accepted and ready to land.
lg ================ Comment at: unittests/clang-move/ClangMoveTests.cpp:122 @@ +121,3 @@ + +const char ExpectedNewCC[] = "#include \"foo.h\"\n" + "namespace a {\n" ---------------- It's fine for this patch, but I think we also want to add newlines be between moved declarations, especially if there were newlines in the original code. https://reviews.llvm.org/D24243 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits