From 3487fde236d0852a8b0672c293fa41a741f471e8 Mon Sep 17 00:00:00 2001 From: Matt Mastracci Date: Fri, 17 Mar 2023 16:15:27 -0600 Subject: perf(core) Reduce copying and cloning in extension initialization (#18252) Follow-up to #18210: * we are passing the generated `cfg` object into the state function rather than passing individual config fields * reduce cloning dramatically by making the state_fn `FnOnce` * `take` for `ExtensionBuilder` to avoid more unnecessary copies * renamed `config` to `options` --- ext/broadcast_channel/lib.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'ext/broadcast_channel') diff --git a/ext/broadcast_channel/lib.rs b/ext/broadcast_channel/lib.rs index 3df11566f..5b38e70f8 100644 --- a/ext/broadcast_channel/lib.rs +++ b/ext/broadcast_channel/lib.rs @@ -114,13 +114,13 @@ deno_core::extension!(deno_broadcast_channel, op_broadcast_recv, ], esm = [ "01_broadcast_channel.js" ], - config = { + options = { bc: BC, unstable: bool, }, - state = |state, bc, unstable| { - state.put(bc); - state.put(Unstable(unstable)); + state = |state, options| { + state.put(options.bc); + state.put(Unstable(options.unstable)); }, ); -- cgit v1.2.3