kotlinpoet Smarter wrapping logic for long parameter lists

Fixes #274

Nov 14 '17 10:11 Egorand

I think I'll update this PR to cover #291 as well. Any feedback here so far?

Dec 01 '17 21:12 Egorand

I certainly like the results. When do you emit a parameter spec with wrapping=False?

Dec 15 '17 22:12 JakeWharton

When you've got a parameterized LambdaTypeName as the type of the param:

fun veryLongFunctionName(
        veryLongParameterName: (java.io.Serializable, java.lang.Appendable, kotlin.Cloneable) -> kotlin.Unit,
        i: kotlin.Int
) = kotlin.Unit

Dec 15 '17 22:12 Egorand

Ah, nice. I think I'd prefer if we flipped the boolean so that it wraps by default and we opt-out only in the lambda special case.

Dec 15 '17 22:12 JakeWharton

Can do

Dec 15 '17 22:12 Egorand

Fixes #291

Dec 23 '17 17:12 Egorand

Any updates on this request?

Dec 05 '18 18:12 gilgoldzweig

@gilgoldzweig we'll most likely go with a solution for #532, which should handle this use-case as well.

Dec 05 '18 19:12 Egorand

Cool, is there any way to make it work now?

Dec 05 '18 20:12 gilgoldzweig

There's no workaround unfortunately, but if your function/constructor has more than 2 parameters we'll auto-wrap it.

Dec 05 '18 20:12 Egorand