================ @@ -306,6 +311,12 @@ TEST(IncludeCleaner, GenerateMissingHeaderDiags) { withFix({Fix(MainFile.range("insert_b"), "#include \"b.h\"\n", "#include \"b.h\""), FixMessage("add all missing includes")})), + AllOf(Diag(MainFile.range("b_angled"), + "No header providing \"b_angled\" is directly included"), + withFix( + {Fix(MainFile.range("insert_b_angled"), + "#include <b_angled.h>\n", "#include \"b_angled.h\""), ---------------- HighCommander4 wrote:
Shouldn't the message (third parameter to `Fix()`) also be `#include <b_angled.h>`? https://github.com/llvm/llvm-project/pull/140594 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits