From 298700108a286fa7154e4a638017c9fbebf5c3ff Mon Sep 17 00:00:00 2001 From: Michael Williamson Date: Thu, 2 Nov 2023 23:01:34 +0000 Subject: Parse return statements without expression --- .../org/shedlang/compiler/ast/untyped/UntypedFunctionReturnNode.java | 4 +++- src/main/java/org/shedlang/compiler/parser/Parser.java | 4 +++- src/main/java/org/shedlang/compiler/typechecker/TypeChecker.java | 2 +- 3 files changed, 7 insertions(+), 3 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/org/shedlang/compiler/ast/untyped/UntypedFunctionReturnNode.java b/src/main/java/org/shedlang/compiler/ast/untyped/UntypedFunctionReturnNode.java index 94eb7a65..9809ce1f 100644 --- a/src/main/java/org/shedlang/compiler/ast/untyped/UntypedFunctionReturnNode.java +++ b/src/main/java/org/shedlang/compiler/ast/untyped/UntypedFunctionReturnNode.java @@ -2,8 +2,10 @@ package org.shedlang.compiler.ast.untyped; import org.shedlang.compiler.sources.Source; +import java.util.Optional; + public record UntypedFunctionReturnNode( - UntypedExpressionNode expression, + Optional expression, Source source ) implements UntypedFunctionStatementNode { @Override diff --git a/src/main/java/org/shedlang/compiler/parser/Parser.java b/src/main/java/org/shedlang/compiler/parser/Parser.java index 329a5283..272c66c0 100644 --- a/src/main/java/org/shedlang/compiler/parser/Parser.java +++ b/src/main/java/org/shedlang/compiler/parser/Parser.java @@ -413,7 +413,9 @@ public class Parser { tokens.skip(TokenType.KEYWORD_RETURN); - var expression = parseTopLevelExpression(tokens); + var expression = tokens.isNext(TokenType.SYMBOL_SEMICOLON) + ? Optional.empty() + : Optional.of(parseTopLevelExpression(tokens)); tokens.skip(TokenType.SYMBOL_SEMICOLON); diff --git a/src/main/java/org/shedlang/compiler/typechecker/TypeChecker.java b/src/main/java/org/shedlang/compiler/typechecker/TypeChecker.java index 00797708..d00cc352 100644 --- a/src/main/java/org/shedlang/compiler/typechecker/TypeChecker.java +++ b/src/main/java/org/shedlang/compiler/typechecker/TypeChecker.java @@ -768,7 +768,7 @@ public class TypeChecker { TypeCheckerContext context ) { // TODO: pass return type as hint if available - var expressionResult = typeCheckExpression(untyped.expression(), Optional.empty(), context); + var expressionResult = typeCheckExpression(untyped.expression().orElseThrow(), Optional.empty(), context); var expression = expressionResult.value(); context = expressionResult.context(); -- cgit v1.2.3