Merge pull request #348 from giovannicalo/master

Added support for Node 13
zeromq · Nov 16, 2019 · 2fab8a9 · 2fab8a9
2 parents cbc9b7b + e2ff5e4
commit 2fab8a9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -4,6 +4,7 @@ node_js:
 - "8"
 - "10"
 - "11"
+- "13"
 sudo: required
 
 matrix:

diff --git a/binding.cc b/binding.cc
@@ -1297,8 +1297,8 @@ namespace zmq {
         }
       }
 
-      Local<Object> buf = batch->Get(i).As<Object>();
-      Local<Number> flagsObj = batch->Get(i + 1).As<Number>();
+      Local<Object> buf = batch->Get(Nan::GetCurrentContext(), i).ToLocalChecked().As<Object>();
+      Local<Number> flagsObj = batch->Get(Nan::GetCurrentContext(), i + 1).ToLocalChecked().As<Number>();
 
       int flags = Nan::To<int>(flagsObj).FromJust();