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 return type into the ABI #633

Closed
kevaundray opened this issue Jan 12, 2023 · 1 comment
Closed

Add return type into the ABI #633

kevaundray opened this issue Jan 12, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@kevaundray
Copy link
Contributor

Problem

The ABI currenlty only specifies parameters and not the return types from the functions. This is not a big problem as we use the "return" parameter to denote the output. Assuming there are no other parameters named "return" this is safe.

Solution

We can add another field in the ABI Parameter field called "return" and put the return type there instead.

Alternatives considered

(Describe any alternative solutions you have considered.)

Additional context

(If applicable.)

@kevaundray kevaundray added the enhancement New feature or request label Jan 12, 2023
@kevaundray kevaundray changed the title Add return types into the ABI Add return type into the ABI Jan 12, 2023
@TomAFrench
Copy link
Member

This is a duplicate of #678. Closing this issue as the other has more detail.

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

No branches or pull requests

2 participants