Suggestion: Needless blank line(s) additions

[vanniktech]

The check, Needless blank line(s), flags the following code:

}


private fun something() { }

although I'd also expect it to flag the one below as well:

  }

}

since the newline between the } is not necessary.

[shyiko]

Hi @vanniktech.

While I like the idea, I'm note sure Intellij built-in formatter supports this (there is a setting for Java (Code Style -> Java -> Blank Lines -> Before '}') which can be set to 0 to enforce this kind of style, but not for Kotlin) (at least not in IDEA 2016.3). It's pretty easy to implement but I'd prefer not to add anything that can't be enforced by the IDEA's formatter (if we can).

[vanniktech]

There will be an option with 1.1.4.

[shyiko]

@vanniktech 👍

[vanniktech]

Will this mean this functionality will be added now or once 1.1.4 is out?

[shyiko]

I think we should hold off until 1.1.4 is out (just to be on the safe side).

[vanniktech]

Kotlin 1.1.4 is out

[shyiko]

Change contributed by @Ilya-Gh 🎉.
Keeping this issue open until ktlint-intellij-idea-integration is updated & a new release is published.

[vanniktech]

This can be closed now. Thanks :)