[PATCH] D95017: [clang-format] Add case aware include sorting.
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 1 12:00:30 PST 2021
HazardyKnusperkeks added a comment.
In D95017#2572578 <https://reviews.llvm.org/D95017#2572578>, @PragmaNull wrote:
> 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.
Do you make a change? Otherwise I will do, but that will take some time, because I'm rather busy.
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