[PATCH] D143691: Fix clang-formats IncludeCategory to match the documentation
Fabian Keßler via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Feb 10 18:38:23 PST 2023
Febbe added a comment.
I think I am done, and you can review it now.
With the latest changes, SortPriority is no longer required to match Priority, it can always be 0, since Priority is now used in the sorting algorithm.
This is what the documentation tends to depict:
"and #includes are sorted first according to increasing category number and then alphabetically within each category."
" There is a third and optional field SortPriority which can used while IncludeBlocks = IBS_Regroup to define the priority in which #includes should be ordered."
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143691/new/
https://reviews.llvm.org/D143691
More information about the cfe-commits
mailing list