[riot-notifications] [RIOT-OS/RIOT] WIP: pkg/wamr: add WAMR to provide WASM support in RIOT (#15329)

Karl Fessel notifications at github.com
Thu Sep 23 12:57:25 CEST 2021


> Would you mind moving the convenience functions out of the example? E.g. you could create a `wamr` module (which depends on the `wamr` package) that adds RIOT specific convenience stuff. Or you could just add a `contrib` and a `include` folder into the package and add code and headers there. Whatever you consider cleaner.

I wouldn't consider the convenience functions in the example an API -> I don't think they should go into a module or into the pkg.
The API to wamr is defined in RIOT/build/pkg/wamr/core/iwasm/include/*.h.

But i can split them into a seperate .c and .h inside the example

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15329#issuecomment-925706451
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210923/23d5bddf/attachment.htm>


More information about the notifications mailing list