ITStudio_OUC/OxideJS
Archived
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: 将同步原生调用参数限制错误消息对齐 ECMAScript 规范措辞

Browse Source
This commit is contained in:
爱特每羊队
2026-06-26 16:41:21 +08:00
parent 02c7de6e09
commit 417368dd5f
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/oxide_vm/src/vm.rs
+2 -5
View File
@@ -252,10 +252,7 @@ impl Vm {
&mut self, receiver: JsValue, callee: JsValue, args: &[JsValue], &mut self, receiver: JsValue, callee: JsValue, args: &[JsValue],
) -> Result<Vec<u8>, String> { ) -> Result<Vec<u8>, String> {
if args.len() > Self::SYNC_NATIVE_ARG_LIMIT { if args.len() > Self::SYNC_NATIVE_ARG_LIMIT {
return Err(format!( return Err("RangeError: Maximum call stack size exceeded".to_string());
"RangeError: synchronous native call supports at most {} arguments",
Self::SYNC_NATIVE_ARG_LIMIT
));
} }
self.regs[253] = receiver; self.regs[253] = receiver;
@@ -1450,7 +1447,7 @@ mod tests {
.call_function_sync(callee, JsValue::undefined(), &args) .call_function_sync(callee, JsValue::undefined(), &args)
.expect_err("too many args should fail cleanly"); .expect_err("too many args should fail cleanly");
assert!(err.contains("at most 253 arguments"), "unexpected error: {err}"); assert!(err.contains("Maximum call stack size exceeded"), "unexpected error: {err}");
assert_eq!(vm.reg(1), JsValue::int(7)); assert_eq!(vm.reg(1), JsValue::int(7));
assert_eq!(vm.reg(253), JsValue::int(11)); assert_eq!(vm.reg(253), JsValue::int(11));
assert_eq!(vm.reg(254), JsValue::int(12)); assert_eq!(vm.reg(254), JsValue::int(12));