Skip to content

Commit

Permalink
src: replace assert with CHECK_LE in node_api.cc
Browse files Browse the repository at this point in the history
PR-URL: #14514
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Jason Ginchereau <jasongin@microsoft.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Tobias Nie�en <tniessen@tnie.de>
  • Loading branch information
bnoordhuis committed Aug 7, 2017
1 parent 41a0dfc commit f3afe29
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/node_api.cc
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
#include <node_object_wrap.h>
#include <string.h>
#include <algorithm>
#include <cassert>
#include <cmath>
#include <vector>
#include "uv.h"
Expand Down Expand Up @@ -852,7 +851,7 @@ napi_status napi_get_last_error_info(napi_env env,
static_assert(
node::arraysize(error_messages) == napi_escape_called_twice + 1,
"Count of error messages must match count of error values");
assert(env->last_error.error_code <= napi_escape_called_twice);
CHECK_LE(env->last_error.error_code, napi_escape_called_twice);

// Wait until someone requests the last error information to fetch the error
// message string
Expand Down

0 comments on commit f3afe29

Please sign in to comment.