Remove unnecessary Send bound on closures wrapped in JsFunction

neon-bindings · Nov 11, 2021 · be6a7ed · be6a7ed
1 parent 45e123a
commit be6a7ed
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/crates/neon-runtime/src/napi/fun.rs b/crates/neon-runtime/src/napi/fun.rs
@@ -9,7 +9,7 @@ use crate::raw::{Env, Local};
 
 pub unsafe fn new<F>(env: Env, name: &str, callback: F) -> Result<Local, napi::Status>
 where
-    F: Fn(Env, napi::CallbackInfo) -> Local + Send + 'static,
+    F: Fn(Env, napi::CallbackInfo) -> Local + 'static,
 {
     let mut out = MaybeUninit::uninit();
     let data = Box::into_raw(Box::new(callback));
@@ -59,7 +59,7 @@ where
 
 unsafe extern "C" fn call_boxed<F>(env: Env, info: napi::CallbackInfo) -> Local
 where
-    F: Fn(Env, napi::CallbackInfo) -> Local + Send + 'static,
+    F: Fn(Env, napi::CallbackInfo) -> Local + 'static,
 {
     let mut data = MaybeUninit::uninit();
     let status = napi::get_cb_info(

diff --git a/src/context/mod.rs b/src/context/mod.rs
@@ -721,7 +721,7 @@ impl<'a> ModuleContext<'a> {
     /// Convenience method for exporting a Neon function from a module.
     pub fn export_function<F, V>(&mut self, key: &str, f: F) -> NeonResult<()>
     where
-        F: Fn(FunctionContext) -> JsResult<V> + Send + 'static,
+        F: Fn(FunctionContext) -> JsResult<V> + 'static,
         V: Value,
     {
         let value = JsFunction::new(self, f)?.upcast::<JsValue>();

diff --git a/src/types/mod.rs b/src/types/mod.rs
@@ -746,7 +746,7 @@ impl JsFunction {
     pub fn new<'a, C, F, V>(cx: &mut C, f: F) -> JsResult<'a, JsFunction>
     where
         C: Context<'a>,
-        F: Fn(FunctionContext) -> JsResult<V> + Send + 'static,
+        F: Fn(FunctionContext) -> JsResult<V> + 'static,
         V: Value,
     {
         Self::new_internal(cx, f)
@@ -756,7 +756,7 @@ impl JsFunction {
     fn new_internal<'a, C, F, V>(cx: &mut C, f: F) -> JsResult<'a, JsFunction>
     where
         C: Context<'a>,
-        F: Fn(FunctionContext) -> JsResult<V> + Send + 'static,
+        F: Fn(FunctionContext) -> JsResult<V> + 'static,
         V: Value,
     {
         use std::any;