[PATCH] D121370: [clang-format] SortIncludes should support "@import" lines in Objective-C
Konrad Wilhelm Kleine via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Mar 16 03:52:21 PDT 2022
kwk marked an inline comment as done.
kwk added a comment.
Addressed review comments.
================
Comment at: clang/lib/Format/Format.cpp:2779
/*CheckMainHeader=*/!MainIncludeFound && FirstIncludeBlock);
- int Priority = Categories.getSortIncludePriority(
- IncludeName, !MainIncludeFound && FirstIncludeBlock);
+ int Priority = WithSemicolon ? INT_MAX
+ : Categories.getSortIncludePriority(
----------------
HazardyKnusperkeks wrote:
> I don't know about the others, but I prefer C++: `std::numeric_limits`
Sure. That's done now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121370/new/
https://reviews.llvm.org/D121370
More information about the cfe-commits
mailing list