webassembly/proxy_c: Don't return value of a void function.

dpgeorge · dpgeorge · commit ed2885facbf0 · 2024-05-21T15:14:14.000+10:00
Signed-off-by: Damien George &lt;damien@micropython.org&gt;
diff --git a/ports/webassembly/proxy_c.c b/ports/webassembly/proxy_c.c
@@ -210,10 +210,10 @@ void proxy_c_to_js_dir(uint32_t c_ref, uint32_t *out) {
             dir = mp_builtin_dir_obj.fun.var(1, args);
         }
         nlr_pop();
-        return proxy_convert_mp_to_js_obj_cside(dir, out);
+        proxy_convert_mp_to_js_obj_cside(dir, out);
     } else {
         // uncaught exception
-        return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);
+        proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);
     }
 }
 
@@ -255,10 +255,10 @@ void proxy_c_to_js_lookup_attr(uint32_t c_ref, const char *attr_in, uint32_t *ou
             member = mp_load_attr(obj, attr);
         }
         nlr_pop();
-        return proxy_convert_mp_to_js_obj_cside(member, out);
+        proxy_convert_mp_to_js_obj_cside(member, out);
     } else {
         // uncaught exception
-        return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);
+        proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);
     }
 }
 
@@ -482,9 +482,9 @@ void proxy_c_to_js_resume(uint32_t c_ref, uint32_t *args) {
         mp_obj_t reject = proxy_convert_js_to_mp_obj_cside(args + 2 * 3);
         mp_obj_t ret = proxy_resume_execute(obj, mp_const_none, mp_const_none, resolve, reject);
         nlr_pop();
-        return proxy_convert_mp_to_js_obj_cside(ret, args);
+        proxy_convert_mp_to_js_obj_cside(ret, args);
     } else {
         // uncaught exception
-        return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, args);
+        proxy_convert_mp_to_js_exc_cside(nlr.ret_val, args);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -210,10 +210,10 @@ void proxy_c_to_js_dir(uint32_t c_ref, uint32_t *out) {`
`210`	`210`	`dir = mp_builtin_dir_obj.fun.var(1, args);`
`211`	`211`	`}`
`212`	`212`	`nlr_pop();`
`213`		`- return proxy_convert_mp_to_js_obj_cside(dir, out);`
	`213`	`+ proxy_convert_mp_to_js_obj_cside(dir, out);`
`214`	`214`	`} else {`
`215`	`215`	`// uncaught exception`
`216`		`- return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);`
	`216`	`+ proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);`
`217`	`217`	`}`
`218`	`218`	`}`
`219`	`219`
`@@ -255,10 +255,10 @@ void proxy_c_to_js_lookup_attr(uint32_t c_ref, const char attr_in, uint32_t ou`
`255`	`255`	`member = mp_load_attr(obj, attr);`
`256`	`256`	`}`
`257`	`257`	`nlr_pop();`
`258`		`- return proxy_convert_mp_to_js_obj_cside(member, out);`
	`258`	`+ proxy_convert_mp_to_js_obj_cside(member, out);`
`259`	`259`	`} else {`
`260`	`260`	`// uncaught exception`
`261`		`- return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);`
	`261`	`+ proxy_convert_mp_to_js_exc_cside(nlr.ret_val, out);`
`262`	`262`	`}`
`263`	`263`	`}`
`264`	`264`
`@@ -482,9 +482,9 @@ void proxy_c_to_js_resume(uint32_t c_ref, uint32_t *args) {`
`482`	`482`	`mp_obj_t reject = proxy_convert_js_to_mp_obj_cside(args + 2 * 3);`
`483`	`483`	`mp_obj_t ret = proxy_resume_execute(obj, mp_const_none, mp_const_none, resolve, reject);`
`484`	`484`	`nlr_pop();`
`485`		`- return proxy_convert_mp_to_js_obj_cside(ret, args);`
	`485`	`+ proxy_convert_mp_to_js_obj_cside(ret, args);`
`486`	`486`	`} else {`
`487`	`487`	`// uncaught exception`
`488`		`- return proxy_convert_mp_to_js_exc_cside(nlr.ret_val, args);`
	`488`	`+ proxy_convert_mp_to_js_exc_cside(nlr.ret_val, args);`
`489`	`489`	`}`
`490`	`490`	`}`