[
https://issues.apache.org/jira/browse/GROOVY-8834?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Paul King closed GROOVY-8834.
-----------------------------
Resolution: Not A Problem
As per last comment. I'll close this and we can re-open if we have specific
improvement suggestions to consider.
> Making "public" - BinaryExpression.leftExpression and similar fields
> --------------------------------------------------------------------
>
> Key: GROOVY-8834
> URL: https://issues.apache.org/jira/browse/GROOVY-8834
> Project: Groovy
> Issue Type: Wish
> Reporter: Anton Pryamostanov
> Priority: Major
> Labels: ast
>
> This is a general request related to AST API.
>
> Currently BinaryExpression.leftExpression and similar fields are having
> Private access.
> So to change the value, we need to replace the whole BinaryExpression with
> new one.
> This is not useful as it requires coding outside of Visitor methods, and I do
> not see reason why it should be private. It will greatly simplify visitor
> methods and transformations, if all Expression fields will have Public access.
> Alternatively please add setter methods with Public access.
> List of fields to be changed from Private to Public:
> {code:java}
> ArrayExpression->expressions
> ArrayExpression->sizeExpression
> BitwiseNegationExpression->expression
> BooleanExpression->expression
> CastExpression->expression
> ConstructorCallExpression->arguments
> GStringExpression->strings
> GStringExpression->values
> ListExpression->expressions
> MapExpression->mapEntryExpressions
> MethodPointerExpression->expression
> MethodPointerExpression->methodName
> PropertyExpression->property
> RangeExpression->from
> RangeExpression->to
> SpreadExpression->expression
> SpreadMapExpression->expression
> StaticMethodCallExpression->arguments
> TernaryExpression->booleanExpression
> TernaryExpression->trueExpression
> TernaryExpression->falseExpression
> TupleExpression->expressions
> UnaryMinusExpression->expression
> UnaryPlusExpression->expression{code}
> Note: those fields with "final" modifier should also be made re-assignable.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
