Precommit fails after expanding magic string API

[bzsolt]

After [723924f]](seanshpark@723924f) the precommit fails with the following message:

.../tests/unit/test_api.cpp:452:65: error: conversion to ‘uint32_t {aka unsigned int}’ from ‘long unsigned int’ may alter its value [-Werror=conversion]
   uint32_t num_magic_string_items = sizeof (magic_string_items) / sizeof (jerry_api_char_ptr_t);

Here is the mentoined line: L452
Refer: #136

[bzsolt]

CC: @seanshpark

[ruben-ayrapetyan]

@bzsolt, could you, please, check the following patch?

diff --git a/tests/unit/test_api.cpp b/tests/unit/test_api.cpp
index f002f35..ada2e6d 100644
--- a/tests/unit/test_api.cpp
+++ b/tests/unit/test_api.cpp
@@ -449,7 +449,7 @@ main (void)
   // External Magic String
   jerry_init (JERRY_FLAG_SHOW_OPCODES);

-  uint32_t num_magic_string_items = sizeof (magic_string_items) / sizeof (jerry_api_char_ptr_t);
+  uint32_t num_magic_string_items = (uint32_t) (sizeof (magic_string_items) / sizeof (jerry_api_char_ptr_t));
   jerry_register_external_magic_strings (magic_string_items,
                                          num_magic_string_items,
                                          magic_string_lengths);

[bzsolt]

@ruben-ayrapetyan It is works. Thanks for the quick response.

[ruben-ayrapetyan]

@bzsolt, thank you for checking.

[ruben-ayrapetyan]

The issue occured because precommit testing wasn't performed on x86_64 machine with g++ 4.8.2 compiler.

[ruben-ayrapetyan]

Fixed (#162).

[seanshpark]

@bzsolt , Thank you.