rmarker wrote: It seems like `BinPackParameters` is currently applying to both regular parameters and also template parameters. I think it would be inconsistent if it stopped applying to template parameters only when set to `AlwaysOnePerLine`.
What about separating it out into `BinPackTemplateParameters` so that the two cases can be controlled independently? There are already `BinPackArguments` and `BinPackLongBracedList` also, so having multiple options for bin packing different situations is already established. I imagine that backwards compatibility could be maintained by having `BinPackTemplateParameters` use the `BinPackParameters` if not explicitly set. 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