Skip to content

Commit 83ea561

Browse files
committed
Fix issue with lowering to Groovy AST
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
1 parent c793a42 commit 83ea561

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

modules/compiler/src/main/java/config/parser/ConfigAstBuilder.java

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -186,15 +186,8 @@ private ModuleNode compilationUnit(CompilationUnitContext ctx) {
186186
for( var stmt : ctx.configStatement() )
187187
configStatement(stmt);
188188

189-
var scriptClassNode = moduleNode.getScriptClassDummy();
190-
var statements = moduleNode.getConfigStatements();
191-
if( scriptClassNode != null && !statements.isEmpty() ) {
192-
var first = statements.get(0);
193-
var last = statements.get(statements.size() - 1);
194-
scriptClassNode.setSourcePosition(first);
195-
scriptClassNode.setLastColumnNumber(last.getLastColumnNumber());
196-
scriptClassNode.setLastLineNumber(last.getLastLineNumber());
197-
}
189+
// NOTE: required to produce a valid script class
190+
moduleNode.addStatement(EmptyStatement.INSTANCE);
198191

199192
if( numberFormatError != null )
200193
throw createParsingFailedException(numberFormatError.getV2().getMessage(), numberFormatError.getV1());

modules/compiler/src/main/java/script/parser/ScriptAstBuilder.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,8 @@ private ModuleNode compilationUnit(CompilationUnitContext ctx) {
234234
moduleNode.setEntry(workflowNode);
235235
}
236236

237-
var scriptClassNode = moduleNode.getScriptClassDummy();
237+
// NOTE: required to produce a valid script class
238+
moduleNode.addStatement(EmptyStatement.INSTANCE);
238239

239240
if( numberFormatError != null )
240241
throw createParsingFailedException(numberFormatError.getV2().getMessage(), numberFormatError.getV1());

0 commit comments

Comments
 (0)