-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GROOVY-7909 Calling parents method from trait using Parent.super.method() fail depending on trait declaration order #432
Conversation
@@ -0,0 +1,57 @@ | |||
package groovy.bugs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add the license header to this file, should be able to grab it from another file in the same package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has been added.
/** | ||
* This transformer is used to transform calls to <code>SomeTrait.super.foo()</code> into the appropriate trait call. | ||
* | ||
* @author Cédric Champeau | ||
* @since 2.3.0 | ||
*/ | ||
class SuperCallTraitTransformer extends ClassCodeExpressionTransformer { | ||
public static final String UNRESOLVED_HELPER_CLASS = "UNRESOLVED_HELPER_CLASS"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is only used in the same package, I think package-private would be better than public.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I haven't realize it. Done as you say.
+1 |
Merged, thanks! |
No description provided.