[PATCH] D118873: [clang-format] Revert a feature in RemoveBracesLLVM

2022-02-03 Thread Owen Pan 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 rGeaef54f21388: [clang-format] Revert a feature in RemoveBracesLLVM (authored by owenpan). Changed prior to commit: https://reviews.llvm.org/D118873

[PATCH] D118873: [clang-format] Revert a feature in RemoveBracesLLVM

2022-02-03 Thread Owen Pan via Phabricator via cfe-commits
owenpan added a comment. In D118873#3292956 , @curdeius wrote: > Ok, so now (until a new fix), it will always remove the braces for a single > statement even if it's multi-line. That's fine. Yep. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D118873: [clang-format] Revert a feature in RemoveBracesLLVM

2022-02-03 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius accepted this revision. curdeius added a comment. Ok, so now (until a new fix), it will always remove the braces for a single statement even if it's multi-line. That's fine. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118873/new/ https:

[PATCH] D118873: [clang-format] Revert a feature in RemoveBracesLLVM

2022-02-02 Thread Owen Pan via Phabricator via cfe-commits
owenpan created this revision. owenpan added reviewers: curdeius, MyDeveloperDay, HazardyKnusperkeks. owenpan added a project: clang-format. owenpan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Revert the handling of a single-statement b