Make napi::Status #[must_use]

crates/neon/src/sys/async_work.rs

@@ -74,7 +74,7 @@ pub unsafe fn schedule<I, O, D>(
         napi::Status::Ok => {}
         status => {
             // If queueing failed, delete the work to prevent a leak
-            napi::delete_async_work(env, *work);
+            let _ =napi::delete_async_work(env, *work);
             assert_eq!(status, napi::Status::Ok);
         }
     }
@@ -150,7 +150,7 @@ unsafe extern "C" fn call_complete<I, O, D>(env: Env, status: napi::Status, data
         ..
     } = *Box::<Data<I, O, D>>::from_raw(data.cast());
 
-    napi::delete_async_work(env, work);
+    debug_assert_eq!(napi::delete_async_work(env, work), napi::Status::Ok);
 
     BOUNDARY.catch_failure(env, None, move |env| {
         // `unwrap` is okay because `call_complete` should be called exactly once

crates/neon/src/sys/bindings/types.rs

@@ -69,6 +69,7 @@ pub type ThreadsafeFunctionCallJs = Option<
 >;
 
 #[allow(dead_code)]
+#[must_use]
 #[repr(u32)]
 #[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)]
 pub enum Status {

crates/neon/src/sys/object.rs

@@ -7,7 +7,7 @@ use super::{
 
 /// Mutates the `out` argument to refer to a `napi_value` containing a newly created JavaScript Object.
 pub unsafe fn new(out: &mut Local, env: Env) {
-    napi::create_object(env, out as *mut _);
+    assert_eq!(napi::create_object(env, out as *mut _), napi::Status::Ok);
 }
 
 #[cfg(feature = "napi-8")]

crates/neon/src/sys/primitive.rs

@@ -5,17 +5,17 @@ use super::{
 
 /// Mutates the `out` argument provided to refer to the global `undefined` object.
 pub unsafe fn undefined(out: &mut Local, env: Env) {
-    napi::get_undefined(env, out as *mut Local);
+    assert_eq!(napi::get_undefined(env, out as *mut Local), napi::Status::Ok);
 }
 
 /// Mutates the `out` argument provided to refer to the global `null` object.
 pub unsafe fn null(out: &mut Local, env: Env) {
-    napi::get_null(env, out as *mut Local);
+    assert_eq!(napi::get_null(env, out as *mut Local), napi::Status::Ok);
 }
 
 /// Mutates the `out` argument provided to refer to one of the global `true` or `false` objects.
 pub unsafe fn boolean(out: &mut Local, env: Env, b: bool) {
-    napi::get_boolean(env, b, out as *mut Local);
+    assert_eq!(napi::get_boolean(env, b, out as *mut Local), napi::Status::Ok);
 }
 
 /// Get the boolean value out of a `Local` object. If the `Local` object does not contain a
@@ -32,7 +32,7 @@ pub unsafe fn boolean_value(env: Env, p: Local) -> bool {
 /// Mutates the `out` argument provided to refer to a newly created `Local` containing a
 /// JavaScript number.
 pub unsafe fn number(out: &mut Local, env: Env, v: f64) {
-    napi::create_double(env, v, out as *mut Local);
+    assert_eq!(napi::create_double(env, v, out as *mut Local), napi::Status::Ok);
 }
 
 /// Gets the underlying value of an `Local` object containing a JavaScript number. Panics if

crates/neon/src/sys/tsfn.rs

@@ -187,7 +187,7 @@ impl<T> Drop for ThreadsafeFunction<T> {
         }
 
         unsafe {
-            napi::release_threadsafe_function(
+            let _ = napi::release_threadsafe_function(
                 self.tsfn.0,
                 napi::ThreadsafeFunctionReleaseMode::Release,
             );