[PATCH] D97927: [clang-format] Rename case sorting

2021-03-05 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. In D97927#2605914 , @kentsommer wrote: > LGTM > > Sorry for the slow response! No problem, we all have some phases where we don't have or find the time. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-05 Thread Björn Schäpers via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG7b02794f0aeb: [clang-format] Rename case sorting (authored by HazardyKnusperkeks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-05 Thread Kent Sommer via Phabricator via cfe-commits
kentsommer accepted this revision. kentsommer added a comment. LGTM Sorry for the slow response! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97927/new/ https://reviews.llvm.org/D97927 ___ cfe-commits

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-04 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. In D97927#2603089 , @curdeius wrote: > LGTM. It's indeed more logical this way. > Hopefully the original author will confirm. I'll give him one week, if he or nobody else comes along to object I'll push it. Repository

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-04 Thread Chris Johnson via Phabricator via cfe-commits
PragmaNull accepted this revision. PragmaNull added a comment. Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97927/new/ https://reviews.llvm.org/D97927 ___ cfe-commits mailing list cfe-commits

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-04 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius accepted this revision. curdeius added a comment. This revision is now accepted and ready to land. LGTM. It's indeed more logical this way. Hopefully the original author will confirm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97927/new/

[PATCH] D97927: [clang-format] Rename case sorting

2021-03-04 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks created this revision. HazardyKnusperkeks added reviewers: kentsommer, curdeius, MyDeveloperDay, PragmaNull. HazardyKnusperkeks added a project: clang-format. HazardyKnusperkeks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-comm