A top-level function declaration may not have a body

openrewrite · Sep 10, 2023 · 25abc16 · 25abc16
1 parent 8b210b7
commit 25abc16
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/main/kotlin/org/openrewrite/kotlin/internal/KotlinParserVisitor.kt b/src/main/kotlin/org/openrewrite/kotlin/internal/KotlinParserVisitor.kt
@@ -2671,7 +2671,7 @@ class KotlinParserVisitor(
             returnTypeExpression = visitElement(simpleFunction.returnTypeRef, data) as TypeTree?
             saveCursor = cursor
             before = whitespace()
-            if (source[cursor] == '?') {
+            if (at('?')) {
                 returnTypeExpression = returnTypeExpression!!.withMarkers(
                     returnTypeExpression.markers.addIfAbsent(IsNullable(randomId(), before))
                 )
@@ -4878,6 +4878,8 @@ class KotlinParserVisitor(
         return if (currentFile == null) null else currentFile!!.symbol
     }
 
+    private fun at(c: Char) = cursor < source.length && source[cursor] == c
+
     private fun skip(token: String?): Boolean {
         if (token != null && source.startsWith(token, cursor)) {
             cursor += token.length

diff --git a/src/test/java/org/openrewrite/kotlin/tree/MethodDeclarationTest.java b/src/test/java/org/openrewrite/kotlin/tree/MethodDeclarationTest.java
@@ -322,4 +322,15 @@ fun size(): Int = (-1)
           )
         );
     }
+
+    @Test
+    void multiplatformExpectDeclaration() {
+        rewriteRun(
+          kotlin(
+            """
+              expect suspend fun Any.executeAsync(): Any
+              """
+          )
+        );
+    }
 }