Fix precommit issues.

JerryScript-DCO-1.0-Signed-off-by: László Langó [email protected]

Author: LaszloLango

jerry-core/ecma/base/ecma-helpers-value.c

@@ -229,7 +229,7 @@ ecma_is_value_object (ecma_value_t value) /**< ecma value */
 bool __attr_pure___ __attr_always_inline___
 ecma_is_value_error (ecma_value_t value) /**< ecma value */
 {
-  return (value & (1 << ECMA_VALUE_ERROR_POS)) != 0;
+  return (value & (1u << ECMA_VALUE_ERROR_POS)) != 0;
 } /* ecma_is_value_error */
 
 /**
@@ -327,14 +327,14 @@ ecma_make_error_value (ecma_value_t value) /**< original ecma value */
   /* Error values cannot be converted. */
   JERRY_ASSERT (!ecma_is_value_error (value));
 
-  return value | (1 << ECMA_VALUE_ERROR_POS);
+  return value | (1u << ECMA_VALUE_ERROR_POS);
 } /* ecma_make_error_value */
 
 ecma_value_t __attr_const___
 ecma_make_error_obj_value (const ecma_object_t *object_p) /**< object to reference in value */
 {
   return ecma_make_error_value (ecma_make_object_value (object_p));
-} /* ecma_make_error_value */
+} /* ecma_make_error_obj_value */
 
 /**
  * Get pointer to ecma-number from ecma value

jerry-core/ecma/base/ecma-helpers.c

@@ -1340,7 +1340,7 @@ ecma_bytecode_ref (ecma_compiled_code_t *bytecode_p) /**< byte code pointer */
     jerry_fatal (ERR_REF_COUNT_LIMIT);
   }
 
-  bytecode_p->status_flags = (uint16_t) (bytecode_p->status_flags + (1 << ECMA_BYTECODE_REF_SHIFT));
+  bytecode_p->status_flags = (uint16_t) (bytecode_p->status_flags + (1u << ECMA_BYTECODE_REF_SHIFT));
 } /* ecma_bytecode_ref */
 
 /**
@@ -1352,9 +1352,9 @@ ecma_bytecode_deref (ecma_compiled_code_t *bytecode_p) /**< byte code pointer */
 {
   JERRY_ASSERT ((bytecode_p->status_flags >> ECMA_BYTECODE_REF_SHIFT) > 0);
 
-  bytecode_p->status_flags = (uint16_t) (bytecode_p->status_flags - (1 << ECMA_BYTECODE_REF_SHIFT));
+  bytecode_p->status_flags = (uint16_t) (bytecode_p->status_flags - (1u << ECMA_BYTECODE_REF_SHIFT));
 
-  if (bytecode_p->status_flags >= (1 << ECMA_BYTECODE_REF_SHIFT))
+  if (bytecode_p->status_flags >= (1u << ECMA_BYTECODE_REF_SHIFT))
   {
     /* Non-zero reference counter. */
     return;

jerry-core/ecma/builtin-objects/ecma-builtin-array-prototype.c

@@ -59,7 +59,7 @@ static ecma_value_t
 ecma_builtin_array_prototype_helper_set_length (ecma_object_t *object, /**< object*/
                                                 ecma_number_t length) /**< new length */
 {
-  ecma_value_t ret_value = ecma_make_simple_value (ECMA_SIMPLE_VALUE_EMPTY);
+  ecma_value_t ret_value;
   ecma_string_t *magic_string_length_p = ecma_get_magic_string (LIT_MAGIC_STRING_LENGTH);
 
   ecma_number_t *len_p = ecma_alloc_number ();

jerry-core/ecma/builtin-objects/ecma-builtin-global.c

@@ -708,7 +708,7 @@ ecma_builtin_global_object_character_is_in (uint32_t character, /**< character *
                                             const uint8_t *bitset) /**< character set */
 {
   JERRY_ASSERT (character < 128);
-  return (bitset[character >> 3] & (1 << (character & 0x7))) != 0;
+  return (bitset[character >> 3] & (1u << (character & 0x7))) != 0;
 } /* ecma_builtin_global_object_character_is_in */
 
 /*

jerry-core/ecma/operations/ecma-regexp-object.h

@@ -32,9 +32,9 @@
 /**
  * RegExp flags
  */
-#define RE_FLAG_GLOBAL              (1 << 1) /* ECMA-262 v5, 15.10.7.2 */
-#define RE_FLAG_IGNORE_CASE         (1 << 2) /* ECMA-262 v5, 15.10.7.3 */
-#define RE_FLAG_MULTILINE           (1 << 3) /* ECMA-262 v5, 15.10.7.4 */
+#define RE_FLAG_GLOBAL              (1u << 1) /* ECMA-262 v5, 15.10.7.2 */
+#define RE_FLAG_IGNORE_CASE         (1u << 2) /* ECMA-262 v5, 15.10.7.3 */
+#define RE_FLAG_MULTILINE           (1u << 3) /* ECMA-262 v5, 15.10.7.4 */
 
 /**
  * RegExp executor context

jerry-core/jerry.c

@@ -1431,7 +1431,7 @@ jerry_api_invoke_function (bool is_invoke_as_constructor, /**< true - invoke fun
     is_successful = false;
   }
 
-  if (is_successful && retval_p != NULL)
+  if (retval_p != NULL)
   {
     jerry_api_convert_ecma_value_to_api_value (retval_p, call_completion);
   }
@@ -1987,8 +1987,8 @@ jerry_snapshot_set_offsets (uint8_t *buffer_p, /**< buffer */
     ecma_compiled_code_t *bytecode_p = (ecma_compiled_code_t *) buffer_p;
 
     /* Set reference counter to 1. */
-    bytecode_p->status_flags &= (1 << ECMA_BYTECODE_REF_SHIFT) - 1;
-    bytecode_p->status_flags |= 1 << ECMA_BYTECODE_REF_SHIFT;
+    bytecode_p->status_flags &= (1u << ECMA_BYTECODE_REF_SHIFT) - 1;
+    bytecode_p->status_flags |= 1u << ECMA_BYTECODE_REF_SHIFT;
 
     if (bytecode_p->status_flags & CBC_CODE_FLAGS_FUNCTION)
     {

jerry-core/mem/mem-allocator.h

@@ -41,7 +41,7 @@ typedef uint16_t mem_cpointer_t;
 /**
  * Required alignment for allocated units/blocks
  */
-#define MEM_ALIGNMENT       (1 << MEM_ALIGNMENT_LOG)
+#define MEM_ALIGNMENT       (1u << MEM_ALIGNMENT_LOG)
 
 /**
  * Width of compressed memory pointer

jerry-core/parser/js/js-parser.c

@@ -1462,7 +1462,7 @@ parser_post_processing (parser_context_t *context_p) /**< context */
   compiled_code_p = (ecma_compiled_code_t *) parser_malloc (context_p, total_size);
 
   byte_code_p = (uint8_t *) compiled_code_p;
-  compiled_code_p->status_flags = CBC_CODE_FLAGS_FUNCTION | (1 << ECMA_BYTECODE_REF_SHIFT);
+  compiled_code_p->status_flags = CBC_CODE_FLAGS_FUNCTION | (1u << ECMA_BYTECODE_REF_SHIFT);
 
   if (needs_uint16_arguments)
   {

jerry-core/parser/regexp/re-compiler.c

@@ -680,7 +680,7 @@ re_compile_bytecode (re_compiled_code_t **out_bytecode_p, /**< out:pointer to by
     /* 3. Insert extra informations for bytecode header */
     re_compiled_code_t re_compiled_code;
 
-    re_compiled_code.flags = re_ctx.flags | (1 << ECMA_BYTECODE_REF_SHIFT);
+    re_compiled_code.flags = re_ctx.flags | (1u << ECMA_BYTECODE_REF_SHIFT);
     ECMA_SET_NON_NULL_POINTER (re_compiled_code.pattern_cp,
                                ecma_copy_or_ref_ecma_string (pattern_str_p));
     re_compiled_code.num_of_captures = re_ctx.num_of_captures * 2;