From 7fad0afd2c02496185a5aa5fab750b9d5cbab3bc Mon Sep 17 00:00:00 2001 From: Gabriel Schulhof Date: Wed, 19 Jun 2019 13:11:39 -0700 Subject: [PATCH] n-api: add error message for date expected PR-URL: https://github.com/nodejs/node/pull/28303 Reviewed-By: Anna Henningsen Reviewed-By: Colin Ihrig Reviewed-By: Michael Dawson --- src/js_native_api_v8.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/js_native_api_v8.cc b/src/js_native_api_v8.cc index 30f34b474c2..2f5a4bf83cd 100644 --- a/src/js_native_api_v8.cc +++ b/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);