Fixing issue where ktlint would erroneously want to insert a newline …

…if parameter comments were used Fixes pinterest#433
shashachu · May 22, 2019 · 9dd6b69 · 9dd6b69
1 parent cc6c59b
commit 9dd6b69
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/...experimental/src/main/kotlin/com/pinterest/ktlint/ruleset/experimental/IndentationRule.kt b/...experimental/src/main/kotlin/com/pinterest/ktlint/ruleset/experimental/IndentationRule.kt
@@ -207,7 +207,7 @@ class IndentationRule : Rule("indent"), Rule.Modifier.RestrictToRootLast {
         ) {
             return
         }
-        if (!node.nextCodeLeaf()?.prevLeaf().isWhiteSpaceWithNewline() &&
+        if (!node.nextCodeLeaf()?.prevLeaf { !it.isPartOfComment() && it.textLength > 0 }.isWhiteSpaceWithNewline() &&
             // IDEA quirk:
             // if (true &&
             //     true

diff --git a/...rimental/src/test/kotlin/com/pinterest/ktlint/ruleset/experimental/IndentationRuleTest.kt b/...rimental/src/test/kotlin/com/pinterest/ktlint/ruleset/experimental/IndentationRuleTest.kt
@@ -254,6 +254,22 @@ class IndentationRuleTest {
         ).isEmpty()
     }
 
+    @Test(description = "https://github.com/pinterest/ktlint/issues/433")
+    fun testLintParameterListWithComments() {
+        assertThat(
+            IndentationRule().lint(
+                """
+                fun main() {
+                    foo(
+                        /*param1=*/param1,
+                        /*param2=*/param2
+                    )
+                }
+                """.trimIndent()
+            )
+        ).isEmpty()
+    }
+
     @Test
     fun testUnexpectedTabCharacter() {
         val ktScript = "fun main() {\n\t\treturn 0\n\t}"