Primary constructor formatting

[hick209]

For primary constructors ktfmt currently format this code

A

class MyView @JvmOverloads constructor(
    context: Context,
    attrs: AttributeSet? = null,
    @AttrRes defaultAttr = ...,
    @StyleRes defaultStyle = ...,
) : View(context, attrs, defaultAttr, defaultStyle) {
  ...

into this

B

class MyView 
    @JvmOverloads
    constructor(
        context: Context,
        attrs: AttributeSet? = null,
        @AttrRes defaultAttr = ...,
        @StyleRes defaultStyle = ...,
    ) : View(context, attrs, defaultAttr, defaultStyle) {
  ...

It does not seem like A would violate the rectangle rule since, but in case it does, I believe I would prefer something more similar to this

C

class MyView 
@JvmOverloads
constructor(
    context: Context,
    attrs: AttributeSet? = null,
    @AttrRes defaultAttr = ...,
    @StyleRes defaultStyle = ...,
) : View(context, attrs, defaultAttr, defaultStyle) {
  ...

since it gets closer to what we have defined in the guidelines.

This is something that seems like it's still being debated (Kotlin/kotlin-style-guide#2) so there are no clear guidelines on what we should do here, still I say my personal preference is A > C >> B.

[hick209]

Closing as this was resolved in a96140e