[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna created this revision. jessesna added a reviewer: klimek. jessesna requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Extend AllowShortBlocksOnASingleLine for else blocks. See https://bugs.llvm.org/show_bug.cgi?id=49722 Repository:

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna updated this revision to Diff 388924. jessesna added a comment. use verifyFormat instead of EXPECT_EQ (as this does some extra stuff to ensure its stable) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114320/new/ https://reviews.llvm.org/

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna updated this revision to Diff 388961. jessesna added a comment. use the more dense "verifyFormat(after,Style)" calls Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114320/new/ https://reviews.llvm.org/D114320 Files: clang/lib/Format/Unwr

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna added a comment. In D114320#3148046 , @MyDeveloperDay wrote: > Ok now I'm a little puzzled, shouldn't this be covered by > `AllowShortIfStatementsOnASingleLine`? That's exactly what i thought when i first stumbled over the behavior :D But I t

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna added a comment. In D114320#3153795 , @MyDeveloperDay wrote: > Thank you for the patch, Do you need help committing this? if so we need your > name and email address to do so > > https://llvm.org/docs/DeveloperPolicy.html#commit-messages Thank

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna added a comment. Great. Thanks a lot. It's Jesses Gott and jesses.gott.na+l...@gmail.com Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114320/new/ https://reviews.llvm.org/D114320 ___ cfe-commit

[PATCH] D114320: [clang-format] Extend AllowShortBlocksOnASingleLine for else blocks

jessesna added a comment. Thanks a lot for your time and help @MyDeveloperDay Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114320/new/ https://reviews.llvm.org/D114320 ___ cfe-commits mailing list cfe-c