[PATCH] D96744: clang-format IncludeBlocks: Regroup determination of "main" for framework-style includes fix

[Björn Schäpers via Phabricator via cfe-commits]

HazardyKnusperkeks added inline comments.


================
Comment at: clang/unittests/Tooling/HeaderIncludesTest.cpp:106
+        << "for source file " << FileName;
+    EXPECT_EQ(Manager.getIncludePriority("<baz.h>", true), 0)
+        << "for source file " << FileName;
----------------
I would also add `"baz.h"`, since the test is named `IsMainHeader`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96744/new/

https://reviews.llvm.org/D96744

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits