Space after property accessor #56
Conversation
| node.nextLeaf()?.delete() | ||
| } | ||
| } | ||
| } else if (noLFBeforeSet.contains(node.elementType)) { |
There was a problem hiding this comment.
@arturbosch I think we have a regression here. Previously this block would be executed regardless of the preceding if clause (there was no else) (i.e. there could be 2 LintError's emitted for the same node: Unexpected newline before "else" & Missing spacing after "else".) but no anymore.
There was a problem hiding this comment.
Yes your right, my fault - the else keyword is in both sets.
|
Thank you the PR, Artur! This PR makes sure there is no extra space between set/get and the following "(", right? PR description implies extra space after "private set" but I don't think it's inserted. |
|
Argh yes, did not check my spelling. There should be no space after a property accessor. But if the property accessor has no block statement, then it means, the users indention was to change the visibility of the getter/setter which needs no check for spacing. KtLint expected a space which was a false positiv. :) |
|
Thank you 🙇♂️ |
And no space if property accessor has no block body. See https://kotlinlang.org/docs/reference/properties.html