Skip to content
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

fix: Getter now calls method from parent even if getter starts with is #241

Merged
merged 2 commits into from
Jun 15, 2021

Conversation

piiertho
Copy link
Member

Generated getters were not calling parent method if method was starting with is.
This resolves #216

@piiertho piiertho self-assigned this Jun 10, 2021
@piiertho piiertho added the bug Something isn't working label Jun 10, 2021
@piiertho piiertho force-pushed the feature/api-gen/getter-call-parent-method-if-exists branch from edff85c to 8bd2d00 Compare June 10, 2021 16:58
@piiertho piiertho changed the title fix: Getter now calls method from parent even if getter stats with is fix: Getter now calls method from parent even if getter starts with is Jun 11, 2021
@CedNaru
Copy link
Member

CedNaru commented Jun 11, 2021

@JvmName should be your friend.

@piiertho piiertho force-pushed the feature/api-gen/getter-call-parent-method-if-exists branch from 8bd2d00 to e66dbe3 Compare June 12, 2021 10:41
…ter which rely on super implementation will not be visible from java
@piiertho piiertho force-pushed the feature/api-gen/getter-call-parent-method-if-exists branch from e66dbe3 to b77d587 Compare June 12, 2021 10:42
@piiertho piiertho requested a review from chippmann June 12, 2021 10:43
Copy link
Contributor

@chippmann chippmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah with @JvmName it's fine now :-)

Edit: hoh @JvmName is a user on github xD i guess he'll get's mentioned a lot because of this xD

@piiertho piiertho merged commit 33f74e8 into master Jun 15, 2021
@piiertho piiertho deleted the feature/api-gen/getter-call-parent-method-if-exists branch June 15, 2021 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

InputEventMouseButton has no pressed property getter
3 participants