[PATCH] D21323: [clang-format] do not add existing includes.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 14 06:31:18 PDT 2016


djasper added a comment.

Lets get this in for now. We can always remove it if we think it is no longer useful.


================
Comment at: unittests/Format/CleanupTest.cpp:692
@@ +691,3 @@
+TEST_F(CleanUpReplacementsTest, SkipExistingHeaders) {
+  std::string Code = "#include \"a.h\"\n#include <vector>\n";
+  std::string Expected = "#include \"a.h\"\n#include <vector>\n";
----------------
Please also add a test with quoted #includes as well as combined forms (inserting <vector> when there already is "vector" and vice versa).


http://reviews.llvm.org/D21323





More information about the cfe-commits mailing list