[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

2021-11-29 Thread C. Rayroud via Phabricator via cfe-commits
crayroud added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:14126-14127 + // FIXME these tests regressed behaviour. + // verifyFormat("T A::operator() ();", Space); + // verifyFormat("X A::operator++ (T);", Space); verifyFormat("auto lambda = [] () { r

[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

2021-11-29 Thread MyDeveloperDay via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG814aabae3775: [clang-format] regressed default behavior for operator parentheses (authored by MyDeveloperDay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

2021-11-29 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay added a comment. @crayroud I'm going to commit this change to get past the regression, but we can continue to discuss here how you'd like to proceed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114696/new/ https://reviews.llvm.org

[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

2021-11-29 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius accepted this revision. curdeius added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114696/new/ https://reviews.llvm.org/D114696 ___ cfe-commits mailing list cfe-commits@lists.l

[PATCH] D114696: [clang-format] regressed default behavior for operator parentheses

2021-11-29 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay created this revision. MyDeveloperDay added reviewers: HazardyKnusperkeks, crayroud, curdeius, owenpan. MyDeveloperDay added projects: clang, clang-format. MyDeveloperDay requested review of this revision. D110833: [clang-format] Refactor SpaceBeforeParens to add options