JohelEGP added a comment. In D120806#3354119 <https://reviews.llvm.org/D120806#3354119>, @MyDeveloperDay wrote:
> LFTM > > Do we also need to consider this in the context of "AllowShortIfStatments"? > (that would be in a different review in my view) Definitely. Since it didn't work at all, I reported it without a `.clang-format`. I was hoping it'd play well with `AllowShortIfStatments` out of the box. I'm currently building `HEAD` with this patch to reformat my code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120806/new/ https://reviews.llvm.org/D120806 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
