[groovy] branch master updated: Trivial refactoring for `PositionConfigureUtils.configureAST`

[su...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sunlan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git


The following commit(s) were added to refs/heads/master by this push:
     new 8d1a38b  Trivial refactoring for `PositionConfigureUtils.configureAST`
8d1a38b is described below

commit 8d1a38b6a8d09e888688baf99bbb0a233e6dd2d6
Author: Daniel Sun <su...@apache.org>
AuthorDate: Mon Jul 26 14:09:48 2021 +0800

    Trivial refactoring for `PositionConfigureUtils.configureAST`
---
 .../org/apache/groovy/parser/antlr4/util/PositionConfigureUtils.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/main/java/org/apache/groovy/parser/antlr4/util/PositionConfigureUtils.java b/src/main/java/org/apache/groovy/parser/antlr4/util/PositionConfigureUtils.java
index ded5a59..abd79cc 100644
--- a/src/main/java/org/apache/groovy/parser/antlr4/util/PositionConfigureUtils.java
+++ b/src/main/java/org/apache/groovy/parser/antlr4/util/PositionConfigureUtils.java
@@ -91,8 +91,6 @@ public class PositionConfigureUtils {
 
     public static <T extends ASTNode> T configureAST(T astNode, GroovyParser.GroovyParserRuleContext ctx, ASTNode initialStop) {
         Token start = ctx.getStart();
-        Token stop = ctx.getStop();
-
         astNode.setLineNumber(start.getLine());
         astNode.setColumnNumber(start.getCharPositionInLine() + 1);
 
@@ -100,7 +98,8 @@ public class PositionConfigureUtils {
             astNode.setLastLineNumber(initialStop.getLastLineNumber());
             astNode.setLastColumnNumber(initialStop.getLastColumnNumber());
         } else {
-            configureEndPosition(astNode, ctx.getStop());
+            Token stop = ctx.getStop();
+            configureEndPosition(astNode, stop);
         }
 
         return astNode;