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

Add method overloading #1683

Closed
roman-khimov opened this issue Jan 21, 2021 · 2 comments · Fixed by #1689
Closed

Add method overloading #1683

roman-khimov opened this issue Jan 21, 2021 · 2 comments · Fixed by #1689
Assignees
Milestone

Comments

@roman-khimov
Copy link
Member

neo-project/neo#2237, we now can have methods with identical names, but different number of parameters and we need to choose an appropriate one for the call.

@fyrchik
Copy link
Contributor

fyrchik commented Jan 26, 2021

I haven't come up with how we can support this in compiler and it's not Go-way, so may be only interop handling should be changed. Essentially, ...interface{} varargs do the same work.

@roman-khimov
Copy link
Member Author

The only thing we need from the compiler is to generate correct code for deploy(1,2) and deploy(1,2,3) calls, of course our contracts won't expose any overloaded methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants