Add a limit assert.

JerryScript-DCO-1.0-Signed-off-by: Zoltan Herczeg [email protected]

Author: zherczeg

jerry-core/ecma/base/ecma-globals.h

@@ -676,6 +676,9 @@ typedef struct
 
 /**
  * Identifier for ecma-string's actual data container
+ *
+ * Important: ECMA_STRING_CONTAINER__MAX must be updated
+ *            below if a new constant is appended to this list
  */
 typedef enum
 {
@@ -689,6 +692,11 @@ typedef enum
   ECMA_STRING_CONTAINER_MAGIC_STRING_EX /**< the ecma-string is equal to one of external magic strings */
 } ecma_string_container_t;
 
+/**
+ * Maximum value of ecma string container types.
+ */
+#define ECMA_STRING_CONTAINER__MAX ECMA_STRING_CONTAINER_MAGIC_STRING_EX
+
 /**
  * Mask for getting the container of a string.
  */

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

@@ -52,6 +52,12 @@ JERRY_STATIC_ASSERT ((int32_t) ECMA_STRING_MAX_CONCATENATION_LENGTH ==
 JERRY_STATIC_ASSERT (ECMA_STRING_CONTAINER_MASK + 1 == ECMA_STRING_REF_ONE,
                      ecma_string_ref_counter_should_start_after_the_container_field);
 
+/**
+ * The ecma string container types must be lower than the container mask.
+ */
+JERRY_STATIC_ASSERT (ECMA_STRING_CONTAINER_MASK >= ECMA_STRING_CONTAINER__MAX,
+                     ecma_string_container_types_must_be_lower_than_the_container_mask);
+
 /**
  * The ecma string ref and container fields should fill the 16 bit field.
  */
@@ -506,10 +512,10 @@ ecma_deref_ecma_string (ecma_string_t *string_p) /**< ecma-string */
     case ECMA_STRING_CONTAINER_MAGIC_STRING_EX:
     {
       /* only the string descriptor itself should be freed */
+      break;
     }
   }
 
-
   ecma_dealloc_string (string_p);
 } /* ecma_deref_ecma_string */