src: replace assert with CHECK_LE in node_api.cc

bnoordhuis · MylesBorins · commit 0ef0b342e9d8 · 2018-04-16T17:46:03.000-04:00
Backport-PR-URL: #19447 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>
diff --git a/src/node_api.cc b/src/node_api.cc
@@ -13,7 +13,6 @@
 
 #include <string.h>
 #include <algorithm>
-#include <cassert>
 #include <cmath>
 #include <vector>
 #include "uv.h"
@@ -855,7 +854,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
