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

chore(napi): add status to error messages in AsyncWork (#1880)

Browse source
This commit is contained in:
LongYinan 2023-12-30 11:56:36 +08:00 committed by GitHub
parent f2972c743f
commit 65273a4631
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
crates/napi/src/async_work.rs
View file

@ -127,12 +127,20 @@ unsafe extern "C" fn complete<T: Task>(
{
Ok(v) => {
let status = unsafe { sys::napi_resolve_deferred(env, deferred, v) };
debug_assert!(status == sys::Status::napi_ok, "Resolve promise failed");
debug_assert!(
status == sys::Status::napi_ok,
"Resolve promise failed, status: {:?}",
crate::Status::from(status)
);
}
Err(e) => {
let status =
unsafe { sys::napi_reject_deferred(env, deferred, JsError::from(e).into_value(env)) };
debug_assert!(status == sys::Status::napi_ok, "Reject promise failed");
debug_assert!(
status == sys::Status::napi_ok,
"Reject promise failedm status: {:?}",
crate::Status::from(status)
);
}
};
}
@ -142,7 +150,8 @@ unsafe extern "C" fn complete<T: Task>(
let delete_status = unsafe { sys::napi_delete_async_work(env, napi_async_work) };
debug_assert!(
delete_status == sys::Status::napi_ok,
"Delete async work failed"
"Delete async work failed, status {:?}",
crate::Status::from(delete_status)
);
work.status.store(1, Ordering::Relaxed);
}