[PATCH] D114521: [clang-format] [PR47936] AfterControlStatement: MultiLine breaks AllowShortFunctionsOnASingleLine

2021-11-24 Thread Kyrylo Bohdanenko via Phabricator via cfe-commits
KyrBoh added a comment. In D114521#3151431 , @curdeius wrote: > LGTM, but I have one question. You mentioned in the bug ticket comment that > "this exposes a greater issue in AllowShortXXX". Have you found other cases > that misbehave? If so, then it wo

[PATCH] D114521: [clang-format] [PR47936] AfterControlStatement: MultiLine breaks AllowShortFunctionsOnASingleLine

2021-11-24 Thread Kyrylo Bohdanenko via Phabricator via cfe-commits
KyrBoh added a comment. @MyDeveloperDay > What it seems is "AllowShortIfStatementsOnASingleLine" ONLY seems to work > when there are no {} AFAIR, there are 2 settings (in case of control statements): - AllowShortBlocksOnASingleLine --> whether to treat (curly-)braced blocks as a single statem