Skip to content

Commit 9832b8e

Browse files
netaMylesBorins
neta
authored andcommittedDec 12, 2017
src: add napi_handle_scope_mismatch to msg list
PR-URL: #17161 Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Alexey Orlenko <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]>
1 parent 72e480d commit 9832b8e

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed
 

‎src/node_api.cc

+4-3
Original file line numberDiff line numberDiff line change
@@ -901,7 +901,8 @@ const char* error_messages[] = {nullptr,
901901
"Unknown failure",
902902
"An exception is pending",
903903
"The async work item was cancelled",
904-
"napi_escape_handle already called on scope"};
904+
"napi_escape_handle already called on scope",
905+
"Invalid handle scope usage"};
905906

906907
static inline napi_status napi_clear_last_error(napi_env env) {
907908
env->last_error.error_code = napi_ok;
@@ -932,9 +933,9 @@ napi_status napi_get_last_error_info(napi_env env,
932933
// We don't have a napi_status_last as this would result in an ABI
933934
// change each time a message was added.
934935
static_assert(
935-
node::arraysize(error_messages) == napi_escape_called_twice + 1,
936+
node::arraysize(error_messages) == napi_handle_scope_mismatch + 1,
936937
"Count of error messages must match count of error values");
937-
CHECK_LE(env->last_error.error_code, napi_escape_called_twice);
938+
CHECK_LE(env->last_error.error_code, napi_handle_scope_mismatch);
938939

939940
// Wait until someone requests the last error information to fetch the error
940941
// message string

0 commit comments

Comments
 (0)
Please sign in to comment.