Skip to content

Conversation

@robertsipka
Copy link
Contributor

This patch contains an example in which you can run a small js on RIOT-OS with STM32F4-Discovery.
Check the README.md for a more detailed explanation of how to compile and run it.

JerryScript-DCO-1.0-Signed-off-by: Robert Sipka rsipka.uszeged@partner.samsung.com

@LaszloLango LaszloLango added development Feature implementation jerry-port Related to the port API or the default port implementation labels Jun 9, 2016
@LaszloLango
Copy link
Contributor

LGTM

/* Suppress compiler errors */
(void)argc;
(void)argv;
const jerry_api_char_t script[] ="print ('Hello, World!');";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

space after =
space after (void)

@robertsipka
Copy link
Contributor Author

I've updated the patch, based on your comments.


/* start the shell */
char line_buf[SHELL_DEFAULT_BUFSIZE];
shell_run(shell_commands, line_buf, SHELL_DEFAULT_BUFSIZE);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Space after shell_run

@robertsipka
Copy link
Contributor Author

Thanks, I've updated these.

@zherczeg
Copy link
Member

zherczeg commented Jun 9, 2016

LGTM

This patch contains an example in which you can run a small js on RIOT-OS with STM32F4-Discovery.
Check the README.md for a more detailed explanation of how to compile and run it.

JerryScript-DCO-1.0-Signed-off-by: Robert Sipka rsipka.uszeged@partner.samsung.com
@robertsipka
Copy link
Contributor Author

Thanks, I've rebased it with master.

@bzsolt bzsolt merged commit 0dd1082 into jerryscript-project:master Jun 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development Feature implementation jerry-port Related to the port API or the default port implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants