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

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 16 00:03:00 PST 2021


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



More information about the cfe-commits mailing list