[PATCH] D138358: [clang-format] Match all block braces

2022-12-03 Thread Owen Pan via Phabricator via cfe-commits
owenpan added a comment. In D138358#3968759 , @HazardyKnusperkeks wrote: > In D138358#3968610 , @owenpan wrote: > >> Please see D139257 for a different >> approach. > > All my

[PATCH] D138358: [clang-format] Match all block braces

2022-12-03 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks abandoned this revision. HazardyKnusperkeks added a comment. In D138358#3968610 , @owenpan wrote: > Please see D139257 for a different > approach. All my hard work... *sniff*. Repository: rG LLVM

[PATCH] D138358: [clang-format] Match all block braces

2022-12-03 Thread Owen Pan via Phabricator via cfe-commits
owenpan added a comment. Please see D139257 for a different approach. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D138358/new/ https://reviews.llvm.org/D138358 ___ cfe

[PATCH] D138358: [clang-format] Match all block braces

2022-12-03 Thread Owen Pan via Phabricator via cfe-commits
owenpan added a comment. This wouldn't work for deeply nested loops: $ cat test.cpp for (;;) { for (;;) { for (;;) { a; } } } $ clang-format -style='{RemoveBracesLLVM: true}' test.cpp for (;;) for (;;) for (;;) a; Expected output: for (;

[PATCH] D138358: [clang-format] Match all block braces

2022-11-19 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks created this revision. HazardyKnusperkeks added reviewers: owenpan, MyDeveloperDay, rymiel. HazardyKnusperkeks added a project: clang-format. Herald added a project: All. HazardyKnusperkeks requested review of this revision. Herald added a project: clang. Herald added a subscribe