[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 13:03:55 CEST 2021


@kfessel commented on this pull request.



> +    uint8_t *wasm_buf = NULL;
+    unsigned wasm_buf_size = 0;

I read the cpp-check and thought WTF declaration on top of the function is normal in c and having uninitialized values around is a huge Problem for much code, that suggestion seems plain stupid to me -forcing errors.


-- 
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#discussion_r714687275
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210923/c3a2890f/attachment.htm>


More information about the notifications mailing list