[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