Skip to content
Permalink
Browse files

n-api: add error message for date expected

PR-URL: #28303
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
  • Loading branch information...
gabrielschulhof committed Jun 19, 2019
1 parent 7696663 commit 7fad0afd2c02496185a5aa5fab750b9d5cbab3bc
Showing with 2 additions and 1 deletion.
  1. +2 −1 src/js_native_api_v8.cc
@@ -676,6 +676,7 @@ const char* error_messages[] = {nullptr,
"Thread-safe function queue is full",
"Thread-safe function handle is closing",
"A bigint was expected",
"A date was expected",
};

napi_status napi_get_last_error_info(napi_env env,
@@ -688,7 +689,7 @@ napi_status napi_get_last_error_info(napi_env env,
// We don't have a napi_status_last as this would result in an ABI
// change each time a message was added.
static_assert(
NAPI_ARRAYSIZE(error_messages) == napi_bigint_expected + 1,
NAPI_ARRAYSIZE(error_messages) == napi_date_expected + 1,
"Count of error messages must match count of error values");
CHECK_LE(env->last_error.error_code, napi_callback_scope_mismatch);

0 comments on commit 7fad0af

Please sign in to comment.
You can’t perform that action at this time.