Repository: groovy Updated Branches: refs/heads/GROOVY_2_6_X a238fc99a -> 611b21fa6
Minor refactoring Project: http://git-wip-us.apache.org/repos/asf/groovy/repo Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/611b21fa Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/611b21fa Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/611b21fa Branch: refs/heads/GROOVY_2_6_X Commit: 611b21fa6bedb72ddfe18c7f698abc75c4fc54ea Parents: a238fc9 Author: sunlan <[email protected]> Authored: Fri Aug 25 08:29:23 2017 +0800 Committer: sunlan <[email protected]> Committed: Fri Aug 25 08:29:42 2017 +0800 ---------------------------------------------------------------------- .../apache/groovy/parser/antlr4/AstBuilder.java | 37 ++++++-------------- 1 file changed, 11 insertions(+), 26 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/groovy/blob/611b21fa/subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java ---------------------------------------------------------------------- diff --git a/subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java b/subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java index 7b7bddd..e47655d 100644 --- a/subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java +++ b/subprojects/parser-antlr4/src/main/java/org/apache/groovy/parser/antlr4/AstBuilder.java @@ -1839,17 +1839,7 @@ public class AstBuilder extends GroovyParserBaseVisitor<Object> implements Groov this.createMethodCallExpression(baseExpr, arguments), arguments); } else { // e.g. a[x] b, new A() b, etc. - methodCallExpression = - this.configureAST( - new MethodCallExpression( - baseExpr, - CALL_STR, - arguments - ), - arguments - ); - - methodCallExpression.setImplicitThis(false); + methodCallExpression = this.createCallMethodCallExpression(baseExpr, arguments); } if (!asBoolean(ctx.commandArgument())) { @@ -2055,16 +2045,7 @@ public class AstBuilder extends GroovyParserBaseVisitor<Object> implements Groov this.configureAST(argumentsExpr, ctx); if (isInsideParentheses(baseExpr)) { // e.g. (obj.x)(), (obj.@x)() - MethodCallExpression methodCallExpression = - new MethodCallExpression( - baseExpr, - CALL_STR, - argumentsExpr - ); - - methodCallExpression.setImplicitThis(false); - - return this.configureAST(methodCallExpression, ctx); + return this.configureAST(createCallMethodCallExpression(baseExpr, argumentsExpr), ctx); } if (baseExpr instanceof AttributeExpression) { // e.g. obj.@a(1, 2) @@ -2143,11 +2124,7 @@ public class AstBuilder extends GroovyParserBaseVisitor<Object> implements Groov } // e.g. 1(), 1.1(), ((int) 1 / 2)(1, 2), {a, b -> a + b }(1, 2), m()() - MethodCallExpression methodCallExpression = - new MethodCallExpression(baseExpr, CALL_STR, argumentsExpr); - methodCallExpression.setImplicitThis(false); - - return this.configureAST(methodCallExpression, ctx); + return this.configureAST(createCallMethodCallExpression(baseExpr, argumentsExpr), ctx); } if (asBoolean(ctx.closure())) { @@ -2254,6 +2231,14 @@ public class AstBuilder extends GroovyParserBaseVisitor<Object> implements Groov throw createParsingFailedException("Unsupported path element: " + ctx.getText(), ctx); } + private MethodCallExpression createCallMethodCallExpression(Expression baseExpr, Expression argumentsExpr) { + MethodCallExpression methodCallExpression = + new MethodCallExpression(baseExpr, CALL_STR, argumentsExpr); + + methodCallExpression.setImplicitThis(false); + + return methodCallExpression; + } @Override public GenericsType[] visitNonWildcardTypeArguments(NonWildcardTypeArgumentsContext ctx) {
