[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