naskya/napi-rs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(napi): move sys call out from debug_assert! macro

Browse source
This commit is contained in:
LongYinan 2020-12-09 21:57:48 +08:00
parent 78245b0e8f
commit 1c0634f6f2
No known key found for this signature in database
GPG key ID: C3666B7FC82ADAD7
1 changed files with 13 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
napi/src/js_values/error.rs
View file

@ -25,19 +25,18 @@ macro_rules! impl_object_methods {
let mut error_code = ptr::null_mut(); let mut error_code = ptr::null_mut();
let mut reason_string = ptr::null_mut(); let mut reason_string = ptr::null_mut();
let mut js_error = ptr::null_mut(); let mut js_error = ptr::null_mut();
debug_assert!( let create_code_status = sys::napi_create_string_utf8(
sys::napi_create_string_utf8( env,
env, error_code_string.as_ptr(),
error_code_string.as_ptr(), status_len,
status_len, &mut error_code,
&mut error_code
) == sys::Status::napi_ok
); );
debug_assert!( debug_assert!(create_code_status == sys::Status::napi_ok);
sys::napi_create_string_utf8(env, reason.as_ptr(), reason_len, &mut reason_string) let create_reason_status =
== sys::Status::napi_ok sys::napi_create_string_utf8(env, reason.as_ptr(), reason_len, &mut reason_string);
); debug_assert!(create_reason_status == sys::Status::napi_ok);
debug_assert!($kind(env, error_code, reason_string, &mut js_error) == sys::Status::napi_ok); let create_error_status = $kind(env, error_code, reason_string, &mut js_error);
debug_assert!(create_error_status == sys::Status::napi_ok);
js_error js_error
} }
@ -47,7 +46,8 @@ macro_rules! impl_object_methods {
/// This function is safety if env is not null ptr. /// This function is safety if env is not null ptr.
pub unsafe fn throw_into(self, env: sys::napi_env) { pub unsafe fn throw_into(self, env: sys::napi_env) {
let js_error = self.into_value(env); let js_error = self.into_value(env);
debug_assert!(sys::napi_throw(env, js_error) == sys::Status::napi_ok); let throw_status = sys::napi_throw(env, js_error);
debug_assert!(throw_status == sys::Status::napi_ok);
} }
#[inline(always)] #[inline(always)]