[riot-notifications] [RIOT-OS/RIOT] pkg: add support for Elk Tiny Javascript engine (#16887)

benpicco notifications at github.com
Fri Sep 24 15:14:08 CEST 2021


@benpicco commented on this pull request.



> + * @{
+ *
+ * @file
+ * @brief       Test application for the Elk Javascript engine
+ *
+ *              See README at https://github.com/cesanta/elk
+ *
+ * @author      Alexandre Abadie <alexandre.abadie at inria.fr>
+ *
+ * @}
+ */
+#include <stdio.h>
+#include "elk.h"
+
+/* C function that adds two numbers. Will be called from JS */
+int sum(int a, int b) {

I'd give that a more unique name so it's clearer where this is coming from

```suggestion
int c_sum_from_riot(int a, int b) {
```

-- 
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/16887#pullrequestreview-763063848
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210924/dc247e2f/attachment-0001.htm>


More information about the notifications mailing list