Do not crash on empty KDoc.

Fixes #5
hovinen · Nov 2, 2020 · 4970d60 · 4970d60
1 parent 57affe8
commit 4970d60
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/formatter/src/main/kotlin/org/kotlin/formatter/scanning/KDocScanner.kt b/formatter/src/main/kotlin/org/kotlin/formatter/scanning/KDocScanner.kt
@@ -46,7 +46,7 @@ internal class KDocScanner(private val kotlinScanner: KotlinScanner) : NodeScann
 
     private fun String.trimOneWhitespace(): String =
         if (startsWith(' ') && endsWith(' ')) {
-            substring(1, length - 1)
+            substring(1, Integer.max(length - 1, 1))
         } else {
             this
         }

diff --git a/formatter/src/test/kotlin/org/kotlin/formatter/KotlinFormatterTest.kt b/formatter/src/test/kotlin/org/kotlin/formatter/KotlinFormatterTest.kt
@@ -3839,6 +3839,25 @@ class KotlinFormatterTest {
         )
     }
 
+    @Test
+    fun `accepts empty KDoc`() {
+        val subject = KotlinFormatter(maxLineLength = 40)
+
+        val result = subject.format(
+            """
+                /** */
+                class MyClass
+            """.trimIndent()
+        )
+
+        assertThat(result).isEqualTo(
+            """
+                /**  */
+                class MyClass
+            """.trimIndent()
+        )
+    }
+
     @Test
     fun `maintains a line break between KDoc and object declaration`() {
         val subject = KotlinFormatter(maxLineLength = 40)