[PATCH] D95017: [clang-format] add case aware include sorting
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jan 24 14:43:56 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2039
+**IncludeSortCaseAware** (``bool``)
+ Specify if sorting should be done in a case aware fashion.
----------------
curdeius wrote:
> The name is somewhat awkward IMO (in the sense it doesn't read nicely), but it's nice to be (alphabetically) closer to other Include-related options in the documentation.
> I have no better name though. Just noticing.
I agree how about `IncludeSortCaseSensitive`
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