[PATCH] D126934: [clang-format] NFC Sort names of format token types
Owen Pan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 2 19:07:47 PDT 2022
owenpan added inline comments.
================
Comment at: clang/lib/Format/FormatToken.h:39-45
+ TYPE(CSharpGenericTypeConstraint) \
+ TYPE(CSharpGenericTypeConstraintColon) \
+ TYPE(CSharpGenericTypeConstraintComma) \
+ TYPE(CSharpNamedArgumentColon) \
+ TYPE(CSharpNullConditionalLSquare) \
+ TYPE(CSharpNullable) \
+ TYPE(CSharpStringLiteral) \
----------------
Also use case-insensitive sorting and move them to just below `CppCastLParen` on line 55.
================
Comment at: clang/lib/Format/FormatToken.h:43-44
+ TYPE(CSharpNamedArgumentColon) \
+ TYPE(CSharpNullConditionalLSquare) \
+ TYPE(CSharpNullable) \
+ TYPE(CSharpStringLiteral) \
----------------
Can you swap them back? I think we want case-insensitive sorting.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126934/new/
https://reviews.llvm.org/D126934
More information about the cfe-commits
mailing list