From 2eafbf2b98e56ef992ba0ea47886e97c9567886e Mon Sep 17 00:00:00 2001 From: Aaron O'Mullan Date: Mon, 12 Apr 2021 23:38:26 +0200 Subject: perf(core/ops): avoid allocs when returning primitives (#10149) --- core/benches/op_baseline.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'core/benches') diff --git a/core/benches/op_baseline.rs b/core/benches/op_baseline.rs index 5287a7a8b..132d92f00 100644 --- a/core/benches/op_baseline.rs +++ b/core/benches/op_baseline.rs @@ -3,10 +3,10 @@ use bencher::{benchmark_group, benchmark_main, Bencher}; use deno_core::error::AnyError; use deno_core::op_async; use deno_core::op_sync; +use deno_core::serialize_op_result; use deno_core::v8; use deno_core::JsRuntime; use deno_core::Op; -use deno_core::OpResponse; use deno_core::OpState; use deno_core::ZeroCopyBuf; @@ -17,8 +17,9 @@ fn create_js_runtime() -> JsRuntime { let mut runtime = JsRuntime::new(Default::default()); runtime.register_op("pi_json", op_sync(|_, _: (), _| Ok(314159))); runtime.register_op("pi_async", op_async(op_pi_async)); - runtime - .register_op("nop", |_, _, _| Op::Sync(OpResponse::Value(Box::new(9)))); + runtime.register_op("nop", |state, _, _| { + Op::Sync(serialize_op_result(Ok(9), state)) + }); // Init ops runtime -- cgit v1.2.3