Merge pull request #10 from Nagyjano/nodeJS12

Node JS12 🎉

Author: MatthD

binding.gyp

@@ -29,7 +29,7 @@
       },
       'sources': [
         'libxml.cpp',
-        # 'libxml-syntax-error.cpp',
+        'libxml-syntax-error.cpp',
         'vendor/libxml/buf.c',
         'vendor/libxml/catalog.c',
         'vendor/libxml/chvalid.c',

libxml-syntax-error.cpp

@@ -1,5 +1,5 @@
 /**
-* This is the Errros Class script, for wellformed & validation DTD
+* This is the Errors Class script, for wellformed & validation DTD
 */
 #include <cstring>
 #include <iostream>
@@ -12,13 +12,13 @@ void setStringField(Napi::Object obj, const char *name, const char *value, Napi:
   {
     return;
   }
-  (obj).Set(Napi::String::New(env, name), Napi::String::New(env, value, strlen(value)));
+  obj.Set(name, Napi::String::New(env, value));
 }
 
 void setNumericField(Napi::Object obj, const char *name, const int value, Napi::Env env)
 {
   Napi::HandleScope scope(env);
-  (obj).Set(Napi::String::New(env, name), Napi::Number::New(env, value));
+  obj.Set(name, Napi::Number::New(env, value));
 }
 
 Napi::Value
@@ -28,7 +28,7 @@ XmlSyntaxError::BuildSyntaxError(xmlError *error, Napi::Env env)
 
   auto err = Napi::TypeError::New(env,
                                   Napi::String::New(env, error->message));
-  Napi::Object out = Napi::Object();
+  Napi::Object out = Napi::Object::New(env);
 
   setStringField(out, "message", error->message, env);
   setNumericField(out, "level", error->level, env);
@@ -43,7 +43,8 @@ XmlSyntaxError::BuildSyntaxError(xmlError *error, Napi::Env env)
   return out;
 }
 
-int XmlSyntaxError::maxError{100};
+uint32_t XmlSyntaxError::maxError{100};
+Napi::Env* XmlSyntaxError::env = nullptr;
 
 void XmlSyntaxError::ChangeMaxNumberOfError(int max)
 {
@@ -53,10 +54,20 @@ void XmlSyntaxError::ChangeMaxNumberOfError(int max)
 void XmlSyntaxError::PushToArray(void *errs, xmlError *error)
 {
   Napi::Array errors = *reinterpret_cast<Napi::Array *>(errs);
-  if (errors.Length >= maxError)
+  if (errors.Length() >= maxError)
   {
     return;
   }
-  Napi::Value castedError = {XmlSyntaxError::BuildSyntaxError(error, XmlSyntaxError::env)};
-  errors.Set(errors.Length, castedError);
+  Napi::Value castedError = {XmlSyntaxError::BuildSyntaxError(error, *XmlSyntaxError::env)};
+  errors.Set(errors.Length(), castedError);
+}
+
+void XmlSyntaxError::PushToArray(Napi::Array& errors, const char* errorMessage)
+{
+  if (errors.Length() >= maxError)
+  {
+    return;
+  }
+  Napi::String messageToPush = Napi::String::New(*XmlSyntaxError::env, errorMessage);
+  errors.Set(errors.Length(), messageToPush);
 }

libxml-syntax-error.h

@@ -7,16 +7,17 @@
 
 class XmlSyntaxError
 {
-  static int maxError;
+  static uint32_t maxError;
 
 public:
-  static Napi::Env env;
+  static Napi::Env* env;
   static void ChangeMaxNumberOfError(int max);
 
-  // push xmlError onto v8::Array
+  // push xmlError onto Napi::Array
   static void PushToArray(void *errs, xmlError *error);
+  static void PushToArray(Napi::Array& errs, const char* errorMessage);
 
-  // create a v8 object for the syntax eror
+  // create a Napi object for the syntax eror
   static Napi::Value BuildSyntaxError(xmlError *error, Napi::Env env);
 };