[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