[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