Affected URL(s)
https://nodejs.org/api/n-api.html#usage
Description of the problem
Shouldn't return NULL be moved outside of the if block in the NAPI_CALL macro? Like this:
if (!is_pending) { \
... \
napi_throw_error((env), NULL, message); \
/* return NULL; */ \
} \
return NULL; \
Otherwise the user's function will continue and may result in more napi calls.
Affected URL(s)
https://nodejs.org/api/n-api.html#usage
Description of the problem
Shouldn't
return NULLbe moved outside of the if block in the NAPI_CALL macro? Like this:Otherwise the user's function will continue and may result in more napi calls.