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

feat(c-api) Implement wasm_func_type #1725

Merged
merged 3 commits into from
Oct 17, 2020
Merged

Conversation

Hywan
Copy link
Contributor

@Hywan Hywan commented Oct 15, 2020

This patch creates a wasm_functype_t::new constructor, that is used
by the new wasm_func_type function, and the existing
wasm_functype_new_inner function.

Note: wasm_func_type is defined in

WASM_API_EXTERN own wasm_functype_t* wasm_func_type(const wasm_func_t*);
.

Review

  • Add a short description of the the change to the CHANGELOG.md file

This patch creates a `wasm_functype_t::new` constructor, that is used
by the new `wasm_func_type` function, and the existing
`wasm_functype_new_inner` function.
@Hywan Hywan added the 🎉 enhancement New feature! label Oct 15, 2020
@Hywan
Copy link
Contributor Author

Hywan commented Oct 16, 2020

bors r+

bors bot added a commit that referenced this pull request Oct 16, 2020
1725: feat(c-api) Implement `wasm_func_type` r=Hywan a=Hywan

This patch creates a `wasm_functype_t::new` constructor, that is used
by the new `wasm_func_type` function, and the existing
`wasm_functype_new_inner` function.

Note: `wasm_func_type` is defined in https://github.com/wasmerio/wasmer/blob/193d7c8ce1b744e6078269ac7a77dd53d1e4cbd6/lib/c-api/tests/wasm_c_api/wasm-c-api/include/wasm.h#L421.

# Review

- [x] Add a short description of the the change to the CHANGELOG.md file


Co-authored-by: Ivan Enderlin <ivan@mnt.io>
@Hywan Hywan added the 📦 lib-c-api About wasmer-c-api label Oct 16, 2020
@bors
Copy link
Contributor

bors bot commented Oct 16, 2020

Build failed:

@MarkMcCaskey
Copy link
Contributor

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 17, 2020

@bors bors bot merged commit 1d4bf9b into wasmerio:master Oct 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎉 enhancement New feature! 📦 lib-c-api About wasmer-c-api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants