Fix linter error

nodejs · Sep 18, 2017 · 8e7bf47 · 8e7bf47
1 parent b0175ae
commit 8e7bf47
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 8 deletions.
diff --git a/src/node_api.cc b/src/node_api.cc
@@ -939,12 +939,14 @@ NAPI_NO_RETURN void napi_fatal_error(const char* location,
   char* location_string = const_cast<char*>(location);
   char* message_string = const_cast<char*>(message);
   if (location_len != -1) {
-    location_string = (char*) malloc(location_len * sizeof(char) + 1);
+    location_string = reinterpret_cast<char*>(
+        malloc(location_len * sizeof(char) + 1));
     strncpy(location_string, location, location_len);
     location_string[location_len] = '\0';
   }
   if (message_len != -1) {
-    message_string = (char*) malloc(message_len * sizeof(char) + 1);
+    message_string = reinterpret_cast<char*>(
+        malloc(message_len * sizeof(char) + 1));
     strncpy(message_string, message, message_len);
     message_string[message_len] = '\0';
   }

diff --git a/test/addons-napi/6_object_wrap/myobject.cc b/test/addons-napi/6_object_wrap/myobject.cc
@@ -22,8 +22,8 @@ void MyObject::Init(napi_env env, napi_value exports) {
   };
 
   napi_value cons;
-  NAPI_CALL_RETURN_VOID(env,
-      napi_define_class(env, "MyObject", -1, New, nullptr, 3, properties, &cons));
+  NAPI_CALL_RETURN_VOID(env, napi_define_class(
+      env, "MyObject", -1, New, nullptr, 3, properties, &cons));
 
   NAPI_CALL_RETURN_VOID(env, napi_create_reference(env, cons, 1, &constructor));
 

diff --git a/test/addons-napi/7_factory_wrap/myobject.cc b/test/addons-napi/7_factory_wrap/myobject.cc
@@ -21,8 +21,8 @@ napi_status MyObject::Init(napi_env env) {
   };
 
   napi_value cons;
-  status =
-      napi_define_class(env, "MyObject", -1, New, nullptr, 1, properties, &cons);
+  status = napi_define_class(
+      env, "MyObject", -1, New, nullptr, 1, properties, &cons);
   if (status != napi_ok) return status;
 
   status = napi_create_reference(env, cons, 1, &constructor);

diff --git a/test/addons-napi/8_passing_wrapped/myobject.cc b/test/addons-napi/8_passing_wrapped/myobject.cc
@@ -17,7 +17,8 @@ napi_status MyObject::Init(napi_env env) {
   napi_status status;
 
   napi_value cons;
-  status = napi_define_class(env, "MyObject", -1, New, nullptr, 0, nullptr, &cons);
+  status = napi_define_class(
+      env, "MyObject", -1, New, nullptr, 0, nullptr, &cons);
   if (status != napi_ok) return status;
 
   status = napi_create_reference(env, cons, 1, &constructor);

diff --git a/test/addons-napi/test_constructor/test2.js b/test/addons-napi/test_constructor/test2.js
@@ -5,4 +5,4 @@ const assert = require('assert');
 // Testing api calls for a constructor that defines properties
 const TestConstructor =
     require(`./build/${common.buildType}/test_constructor_name`);
-assert.equal(TestConstructor.name, 'MyObject');
+assert.strictEqual(TestConstructor.name, 'MyObject');