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

Reply via email to