djasper added inline comments. ================ Comment at: unittests/Format/CleanupTest.cpp:693 @@ +692,3 @@ + std::string Code = "#include \"a.h\"\n#include <vector>\n"; + std::string Expected = "#include \"a.h\"\n#include <vector>\n"; + tooling::Replacements Replaces = {createInsertion("#include <vector>"), ---------------- Please add a line break after each "\n", i.e.:
... = "#include \"a.h\"\n" "#include <vector>"; ================ Comment at: unittests/Format/CleanupTest.cpp:701 @@ +700,3 @@ + std::string Code = "#include \"a.h\"\n#include <vector>\n"; + std::string Expected = "#include \"a.h\"\n#include \"vector\"\n" + "#include <vector>\n#include <a.h>\n"; ---------------- Is this really what we want? http://reviews.llvm.org/D21323 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits