LorenzoMauro wrote: Thank you @HazardyKnusperkeks for the review.
@owenca I agree — I also don't see why anyone would want the current behavior, so it definitely seems like a bug to me as well. It's absolutely possible to fix this without introducing a new option: the check I wrote could simply apply unconditionally when ``BinPackParameters`` is set to ``AlwaysOnePerLine``. If we take that approach, most of the changes in the PR would also become unnecessary. Let me know how you'd like to proceed, and I’ll update the patch accordingly. https://github.com/llvm/llvm-project/pull/137544 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits