### 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.