Browse Source

wasm_c_api.c: improve unimplemented cases (#1355)

ASSERT_NOT_IMPLEMENTED is bh_assert, which might be no-op.
in that case, it's better to fall back to the "default" case,
which reports an error properly.
YAMAMOTO Takashi 3 years ago
parent
commit
425efb875f
1 changed files with 2 additions and 4 deletions
  1. 2 4
      core/iwasm/common/wasm_c_api.c

+ 2 - 4
core/iwasm/common/wasm_c_api.c

@@ -3955,9 +3955,8 @@ interp_link(const wasm_instance_t *inst, const WASMModule *module_interp,
             }
             case IMPORT_KIND_MEMORY:
             case IMPORT_KIND_TABLE:
-                ASSERT_NOT_IMPLEMENTED();
-                break;
             default:
+                ASSERT_NOT_IMPLEMENTED();
                 LOG_WARNING("%s meets unsupported kind: %d", __FUNCTION__,
                             import_rt->kind);
                 goto failed;
@@ -4175,9 +4174,8 @@ aot_link(const wasm_instance_t *inst, const AOTModule *module_aot,
                 break;
             case WASM_EXTERN_MEMORY:
             case WASM_EXTERN_TABLE:
-                ASSERT_NOT_IMPLEMENTED();
-                break;
             default:
+                ASSERT_NOT_IMPLEMENTED();
                 goto failed;
         }
     }