optimize code-style and update tests

Author: hvwyl

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

@@ -300,9 +300,8 @@ parser_emit_unary_lvalue_opcode (parser_context_t *context_p, /**< context */
   }
   else if (opcode == CBC_DELETE_PUSH_RESULT)
   {
+Invalid_LeftHandSide:
     /* Invalid LeftHandSide expression. */
-    Invalid_LeftHandSide:
-
     if (context_p->last_cbc_opcode == PARSER_TO_EXT_OPCODE (CBC_EXT_PUSH_SUPER_PROP_LITERAL)
         || context_p->last_cbc_opcode == PARSER_TO_EXT_OPCODE (CBC_EXT_PUSH_SUPER_PROP))
     {

jerry-core/vm/vm.c

@@ -2955,43 +2955,45 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
         case VM_OC_PROP_PRE_DECR:
         case VM_OC_PROP_POST_INCR:
         case VM_OC_PROP_POST_DECR:
-        if (JERRY_UNLIKELY (byte_code_start_p - 3 >= frame_ctx_p->byte_code_p
-                            && byte_code_start_p[-3] == CBC_EXT_OPCODE
-                            && byte_code_start_p[-2] == CBC_EXT_PUSH_PRIVATE_PROP_LITERAL_REFERENCE))
         {
-          if (opcode < CBC_PRE_INCR)
+          if (JERRY_UNLIKELY (byte_code_start_p - 3 >= frame_ctx_p->byte_code_p
+                              && byte_code_start_p[-3] == CBC_EXT_OPCODE
+                              && byte_code_start_p[-2] == CBC_EXT_PUSH_PRIVATE_PROP_LITERAL_REFERENCE))
           {
-            break;
+            if (opcode < CBC_PRE_INCR)
+            {
+              break;
+            }
+            result = right_value;
+            stack_top_p += 1;
+            left_value = right_value;
+            /* Use right_value as the marker for private field */
+            right_value = ECMA_VALUE_EMPTY;
           }
-          result = right_value;
-          stack_top_p += 1;
-          left_value = right_value;
-          /* Use right_value as the marker for private field */
-          right_value = ECMA_VALUE_EMPTY;
-        }
-        else
-        {
-          result = vm_op_get_value (left_value, right_value);
-
-          if (opcode < CBC_PRE_INCR)
+          else
           {
-            left_value = ECMA_VALUE_UNDEFINED;
-            right_value = ECMA_VALUE_UNDEFINED;
-          }
+            result = vm_op_get_value (left_value, right_value);
 
-          if (ECMA_IS_VALUE_ERROR (result))
-          {
-            goto error;
-          }
+            if (opcode < CBC_PRE_INCR)
+            {
+              left_value = ECMA_VALUE_UNDEFINED;
+              right_value = ECMA_VALUE_UNDEFINED;
+            }
 
-          if (opcode < CBC_PRE_INCR)
-          {
-            break;
-          }
+            if (ECMA_IS_VALUE_ERROR (result))
+            {
+              goto error;
+            }
 
-          stack_top_p += 2;
-          left_value = result;
-          right_value = ECMA_VALUE_UNDEFINED;
+            if (opcode < CBC_PRE_INCR)
+            {
+              break;
+            }
+
+            stack_top_p += 2;
+            left_value = result;
+            right_value = ECMA_VALUE_UNDEFINED;
+          }
           /* FALLTHRU */
         }
         case VM_OC_PRE_INCR:
@@ -3115,7 +3117,7 @@ vm_loop (vm_frame_ctx_t *frame_ctx_p) /**< frame context */
           {
             result = ecma_update_float_number (result, result_number + increase);
           }
-          unary_arithmetic_operation_break:
+unary_arithmetic_operation_break:
           if (JERRY_UNLIKELY (right_value == ECMA_VALUE_EMPTY))
           {
             right_value = ECMA_VALUE_UNDEFINED;

tests/jerry/private_fields.js

@@ -53,7 +53,6 @@ check_syntax_error("class A { static *#bar(x) { } #bar }");
 check_syntax_error("class A { async *#bar(x) { } #bar }");
 check_syntax_error("class A { async #bar(x) { } #bar }");
 check_syntax_error("class A { *#bar(x) { } #bar }");
-check_syntax_error("class A { #a; foo(){ ++this.#a; --this.#a; this.#a++; this.#a--; }}");
 
 
 class A {