[PATCH] D95017: [clang-format] Add case aware include sorting.
Chris Johnson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 18 12:40:22 PST 2021
PragmaNull added a comment.
In D95017#2572238 <https://reviews.llvm.org/D95017#2572238>, @curdeius wrote:
> Do you have an idea for better names?
> I see that e.g. MS documentation uses ascending order and case-sensitive order.
I'm okay with the names, it just seems to me that they are reversed, where CaseSensitive should sort using ASCIIbetic order, and CaseInsensitive should ignore case entirely in the same way that most case-insensitive string compares would.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95017/new/
https://reviews.llvm.org/D95017
More information about the cfe-commits
mailing list