YourWishes/jerryscript
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reduce the scope of variable str_buf in jerry-main (#2457)

Browse Source 
Also reduce the number of magic constants used in the code.

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss akiss@inf.u-szeged.hu
This commit is contained in:
Akos Kiss
2018-08-09 10:11:25 +02:00
committed by László Langó
parent 29f6ffc35b
commit f6b99b60d2
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
jerry-main/main-unix.c
+5 -3
View File
@@ -745,14 +745,16 @@ main (int argc,
jerry_value_t abort_value = jerry_get_value_from_error (ret_value, false); jerry_value_t abort_value = jerry_get_value_from_error (ret_value, false);
if (jerry_value_is_string (abort_value)) if (jerry_value_is_string (abort_value))
{ {
jerry_char_t str_buf[5]; static const char restart_str[] = "r353t";
jerry_value_t str_val = jerry_value_to_string (abort_value); jerry_value_t str_val = jerry_value_to_string (abort_value);
jerry_size_t str_size = jerry_get_string_size (str_val); jerry_size_t str_size = jerry_get_string_size (str_val);
if (str_size == 5) if (str_size == sizeof (restart_str) - 1)
{ {
JERRY_VLA (jerry_char_t, str_buf, str_size);
jerry_string_to_char_buffer (str_val, str_buf, str_size); jerry_string_to_char_buffer (str_val, str_buf, str_size);
if (memcmp ("r353t", (char *) (str_buf), 5) == 0) if (memcmp (restart_str, (char *) (str_buf), str_size) == 0)
{ {
jerry_release_value (ret_value); jerry_release_value (ret_value);
restart = true; restart = true;