Now a block gets recognized as a block if it contains a block.
The special cases we added don't have additional tests because they
are already tested for example in FormatTest.FormatsLambdas and
FormatTestJS.ContainerLiterals.
Differential D150452
[clang-format] Recognize nested blocks sstwcw on May 12 2023, 8:37 AM. Authored by
Details Now a block gets recognized as a block if it contains a block. The special cases we added don't have additional tests because they
Diff Detail
Event TimelineComment Actions Coincidentally I also sent out a review to fix this issue yesterday, but went with a different approach of trying to scope the ProbablyBracedList logic by just looking at the lbrace previous token. Comment Actions On this change I like the token annotator tests, and reusing the same stack instead of adding a second one which always has the same size.
|