0.0.5

Guy Bedford · Guy Bedford · commit 305202cc89c0 · 2023-04-17T20:08:25.000-07:00
diff --git a/lib/spidermonkey_embedding.wasm b/lib/spidermonkey_embedding.wasm
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@bytecodealliance/componentize-js",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "type": "module",
   "exports": "./src/componentize.js",
   "devDependencies": {
diff --git a/spidermonkey_embedding/spidermonkey_embedding.cpp b/spidermonkey_embedding/spidermonkey_embedding.cpp
@@ -819,11 +819,8 @@ check_init()
   return R.init_err;
 }
 
-bool first_call = true;
-
 __attribute__((export_name("call"))) uint32_t call(uint32_t fn_idx, void *argptr)
 {
-  R.free_list.clear();
   Runtime::CoreFn *fn = &R.fns[fn_idx];
   R.cur_fn_idx = fn_idx;
   if (DEBUG)
@@ -1007,11 +1004,12 @@ __attribute__((export_name("post_call"))) void post_call(uint32_t fn_idx)
   //   abort();
   // }
   R.cur_fn_idx = -1;
+  // TODO: pending https://github.com/bytecodealliance/preview2-prototyping/issues/145
+  R.free_list.clear();
   for (void *ptr : R.free_list)
   {
     cabi_free(ptr);
   }
-  R.free_list.clear();
   log("(post_call) jobs");
   js::RunJobs(R.cx);
   log("(post_call) maybe gc");

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@bytecodealliance/componentize-js",`
`3`		`- "version": "0.0.4",`
	`3`	`+ "version": "0.0.5",`
`4`	`4`	`"type": "module",`
`5`	`5`	`"exports": "./src/componentize.js",`
`6`	`6`	`"devDependencies": {`
Original file line number	Diff line number	Diff line change
`@@ -819,11 +819,8 @@ check_init()`
`819`	`819`	`return R.init_err;`
`820`	`820`	`}`
`821`	`821`
`822`		`-bool first_call = true;`
`823`		`-`
`824`	`822`	`__attribute__((export_name("call"))) uint32_t call(uint32_t fn_idx, void *argptr)`
`825`	`823`	`{`
`826`		`- R.free_list.clear();`
`827`	`824`	`Runtime::CoreFn *fn = &R.fns[fn_idx];`
`828`	`825`	`R.cur_fn_idx = fn_idx;`
`829`	`826`	`if (DEBUG)`
`@@ -1007,11 +1004,12 @@ __attribute__((export_name("post_call"))) void post_call(uint32_t fn_idx)`
`1007`	`1004`	`// abort();`
`1008`	`1005`	`// }`
`1009`	`1006`	`R.cur_fn_idx = -1;`
	`1007`	`+ // TODO: pending https://github.com/bytecodealliance/preview2-prototyping/issues/145`
	`1008`	`+ R.free_list.clear();`
`1010`	`1009`	`for (void *ptr : R.free_list)`
`1011`	`1010`	`{`
`1012`	`1011`	`cabi_free(ptr);`
`1013`	`1012`	`}`
`1014`		`- R.free_list.clear();`
`1015`	`1013`	`log("(post_call) jobs");`
`1016`	`1014`	`js::RunJobs(R.cx);`
`1017`	`1015`	`log("(post_call) maybe gc");`