Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 78a9aaa

Browse filesBrowse files
netagibfahn
authored andcommitted
src: add napi_handle_scope_mismatch to msg list
PR-URL: #17161 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
1 parent 861082a commit 78a9aaa
Copy full SHA for 78a9aaa

File tree

Expand file treeCollapse file tree

1 file changed

+4
-3
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+4
-3
lines changed
Open diff view settings
Collapse file

‎src/node_api.cc‎

Copy file name to clipboardExpand all lines: src/node_api.cc
+4-3Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -898,7 +898,8 @@ const char* error_messages[] = {nullptr,
898898
"Unknown failure",
899899
"An exception is pending",
900900
"The async work item was cancelled",
901-
"napi_escape_handle already called on scope"};
901+
"napi_escape_handle already called on scope",
902+
"Invalid handle scope usage"};
902903

903904
static inline napi_status napi_clear_last_error(napi_env env) {
904905
env->last_error.error_code = napi_ok;
@@ -929,9 +930,9 @@ napi_status napi_get_last_error_info(napi_env env,
929930
// We don't have a napi_status_last as this would result in an ABI
930931
// change each time a message was added.
931932
static_assert(
932-
node::arraysize(error_messages) == napi_escape_called_twice + 1,
933+
node::arraysize(error_messages) == napi_handle_scope_mismatch + 1,
933934
"Count of error messages must match count of error values");
934-
CHECK_LE(env->last_error.error_code, napi_escape_called_twice);
935+
CHECK_LE(env->last_error.error_code, napi_handle_scope_mismatch);
935936

936937
// Wait until someone requests the last error information to fetch the error
937938
// message string

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.