From ecdce9a844052909f3cf9b29ee119fb5dac892f2 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 19:32:13 -0400
Subject: [PATCH 1/6] WIP: Migrating to NAN

---
 .../default/virtualbox/action_provision       |   1 +
 .../default/virtualbox/action_set_name        |   1 +
 .vagrant/machines/default/virtualbox/id       |   1 +
 .../machines/default/virtualbox/index_uuid    |   1 +
 .../machines/default/virtualbox/private_key   |  27 +
 .../default/virtualbox/synced_folders         |   1 +
 Vagrantfile                                   |  21 +
 binding.gyp                                   |  21 +-
 package.json                                  |  37 +-
 src/bindings.cc                               | 628 +++++++++---------
 src/bindings.h                                |  14 +-
 src/node_inotify.cc                           |  18 +-
 src/node_inotify.h                            |   3 +-
 13 files changed, 417 insertions(+), 357 deletions(-)
 create mode 100644 .vagrant/machines/default/virtualbox/action_provision
 create mode 100644 .vagrant/machines/default/virtualbox/action_set_name
 create mode 100644 .vagrant/machines/default/virtualbox/id
 create mode 100644 .vagrant/machines/default/virtualbox/index_uuid
 create mode 100644 .vagrant/machines/default/virtualbox/private_key
 create mode 100644 .vagrant/machines/default/virtualbox/synced_folders
 create mode 100644 Vagrantfile

diff --git a/.vagrant/machines/default/virtualbox/action_provision b/.vagrant/machines/default/virtualbox/action_provision
new file mode 100644
index 0000000..d44cf7b
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/action_provision
@@ -0,0 +1 @@
+1.5:b05f2c49-c173-406e-9614-285b9d5c34d7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/action_set_name b/.vagrant/machines/default/virtualbox/action_set_name
new file mode 100644
index 0000000..43fe352
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/action_set_name
@@ -0,0 +1 @@
+1430771691
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/id b/.vagrant/machines/default/virtualbox/id
new file mode 100644
index 0000000..a2cc72b
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/id
@@ -0,0 +1 @@
+b05f2c49-c173-406e-9614-285b9d5c34d7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/index_uuid b/.vagrant/machines/default/virtualbox/index_uuid
new file mode 100644
index 0000000..5082f85
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/index_uuid
@@ -0,0 +1 @@
+a6fa3259af58456ebf88e203d70fa8c7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/private_key b/.vagrant/machines/default/virtualbox/private_key
new file mode 100644
index 0000000..4db1bb4
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/private_key
@@ -0,0 +1,27 @@
+-----BEGIN RSA PRIVATE KEY-----
+MIIEpAIBAAKCAQEA1xngHgMY+fm5lSLusCjiAUNzQv82UpL4ymHCAq0yoTmWug3T
+qqJRPB37k+c5rbpL3CaJ5RgLsAAGFdu+60ooMCXA43FLbK4Jt8SgzXSKA0p06opu
+XxTbgsnzNErnMmTn5SaYqE/tJP08xFAyXywOj2/VnSV3fhUZrVB78iUsFoM0V/Jf
+KRkBvj+h07glD/HA+CrF9TLZ0aDRzkRdVI7qJ/mMAg9Lqdk0JjI1+ldn95Q3AXHc
+77SBjTkifjm49ZZkQOTi7ODpka87X27jwVsKmJ89MCBhu3AiMX0zj3copvmuiY2F
+NHqMzWYpQFfkVpjeQ6L2fu92DJfiyeiIDvVMsQIDAQABAoIBAQC1/a2SlvpV3cT3
++qNKpnx5pzDpOXcjrDgVO1rj2thgcpIlfrdcwpLKs298twElDN9MN8sp6wtQcOS1
+L8Q73PVdr0Og6+Vt4V5VfmazDmwK1r9PBarWhZYn9WfxEYLFL3zIc7xrg5xYVG5K
+hrPntCXplCx/FgkgwweDjPohaPQBYpmwCZL/qWuCs00FIxjRYwB+Aweg6ZXqEv7g
+o4DbJ/LIwQ2y0gwmXexWEX1TQvmtlYqP3fnTQVANAGU7N6HtmtYf7B+g8/OFDME2
+LgBoV4DpIzG/oBE0pxzWoXTi9NRDRRyjNHxSaM/RjHc68IkN/iTJQzp1BcF/kZlO
+XQmGk/qFAoGBAPW3PC30VpZNSohaCKdFMLX/4ZFyI+nup+mueQ/wZa5uhdOCAIdj
+eu/vZDtIt7o7UYq8/z98fENPhMdskShMXk6RYQb9WzC8DUNFvQxozx9DgkyHc/Tm
+lrRwdHVRJ7BxOlBKBe7TBDTHMyCHUpBlFjmaLebakLZhzT8ec+aKJd5HAoGBAOAa
+nSoAsXhtirSTmXEFAXnq3fGB/tfvmYzpIilaiSv8p92xto/TUzlsrfNI8cRA+uv0
+ezaKEDHzE1sOkgZhZ60vJ/ehvVDlLlWM6XUuRAwqIl65vQAbFYYo8JJOuFTIzBjl
+nHePH1MFwmqIl1FIGmel+LK7N8s8SWQcE9uIBqFHAoGAApz9RdvYY+bUiRi0CNti
+hjz/cCWPnjhMjt6tKO88lZtVTweN6+Kx9wVLwpmmYpOo7VnPOSSgybJ5zgkojg4p
+3poLNyDQrHcDiRifxxcCROnZ4+h44jzlk2uywWColGbP4PS4lW6nF8yajZkC1shM
+cTZjkJX4fggSP76aKvSDLJcCgYASDhLbb3Lc4PZr1fhmwqB70hBPhMmjXApi7+Vw
+r3qdsJ1Vnxu2OrBZ0AezsgzIYvPUTr4ehsEHRS4qB+J74R0ZNtNhgAZgnhY9jjzA
+1BlnYX2jZBN2DriNNn8rSmxPaF4JI3aXq1A6wBwJuTd3tPdrolOHEs5L8W94QUrY
+RP1DwQKBgQCJxRj9uu21ENF48Z4v87sibdLHoIjstp+g0980WW9MJjCeU3VVjGWy
+oqNat0CQyU6sEOxj/JdPd1iHvSBQxubYSARV9Gc7I2emEYHqwAfDiHqWV3x+ZyJh
+dZoyIYJlBx1w8/TsXWFJ78mDnVvE64nUJHPnlxsKQc4mxhmBcETjfg==
+-----END RSA PRIVATE KEY-----
diff --git a/.vagrant/machines/default/virtualbox/synced_folders b/.vagrant/machines/default/virtualbox/synced_folders
new file mode 100644
index 0000000..ece4d77
--- /dev/null
+++ b/.vagrant/machines/default/virtualbox/synced_folders
@@ -0,0 +1 @@
+{"virtualbox":{"/vagrant":{"guestpath":"/vagrant","hostpath":"/Users/c4milo/Projects/node-inotify","disabled":false}}}
\ No newline at end of file
diff --git a/Vagrantfile b/Vagrantfile
new file mode 100644
index 0000000..3d8e157
--- /dev/null
+++ b/Vagrantfile
@@ -0,0 +1,21 @@
+$script = <<SCRIPT
+sudo apt-get update -y
+apt-get install -y build-essential
+curl -sL https://deb.nodesource.com/setup_0.12 | sudo bash -
+sudo apt-get install -y nodejs
+SCRIPT
+
+Vagrant.configure("2") do |config|
+	config.vm.box = "ubuntu/trusty64"
+	config.vm.box_check_update = false
+
+	config.vm.hostname = "inotify.local"
+
+	config.vm.provider "virtualbox" do |vb|
+		vb.gui = false
+		vb.memory = 1024
+		vb.cpus = 1
+	end
+
+	config.vm.provision "shell", inline: $script
+end
diff --git a/binding.gyp b/binding.gyp
index 499c4a9..392da49 100644
--- a/binding.gyp
+++ b/binding.gyp
@@ -1,11 +1,14 @@
 {
-  'targets': [
-    {
-      'target_name': 'inotify',
-      'sources': [
-        'src/bindings.cc',
-        'src/node_inotify.cc'
-      ],
-    }
-  ]
+	'targets': [
+		{
+			'target_name': 'inotify',
+			'sources': [
+				'src/bindings.cc',
+				'src/node_inotify.cc'
+			],
+			"include_dirs" : [
+				"<!(node -e \"require('nan')\")"
+			]
+		}
+	]
 }
diff --git a/package.json b/package.json
index 1c95e8d..7f2090a 100644
--- a/package.json
+++ b/package.json
@@ -1,14 +1,25 @@
-{   "name" : "inotify",
-    "version": "1.2.1",
-    "author": "Camilo Aguilar <camilo@cloudescape.com>",
-    "email": "camilo@cloudescape.com",
-    "keywords": ["inotify", "watch", "monitor", "watch files", "watch directories"],
-    "description" : "inotify bindings for v8 javascript engine",
-    "repository": {
-            "type": "git",
-            "url": "http://github.com/c4milo/node-inotify"
-     },
-    "engines": {"node": ">=0.8"},
-    "main" : "inotify"
+{
+  "name": "inotify",
+  "version": "1.3.0",
+  "author": "Camilo Aguilar <camilo@cloudescape.com>",
+  "email": "camilo@cloudescape.com",
+  "keywords": [
+    "inotify",
+    "watch",
+    "monitor",
+    "watch files",
+    "watch directories"
+  ],
+  "description": "inotify bindings for v8 javascript engine",
+  "repository": {
+    "type": "git",
+    "url": "http://github.com/c4milo/node-inotify"
+  },
+  "engines": {
+    "node": ">=0.8"
+  },
+  "main": "inotify",
+  "dependencies": {
+    "nan": "^1.8.4"
+  }
 }
-
diff --git a/src/bindings.cc b/src/bindings.cc
index 61c9e83..611051d 100644
--- a/src/bindings.cc
+++ b/src/bindings.cc
@@ -1,4 +1,4 @@
-// Copyright 2010, Camilo Aguilar. Cloudescape, LLC.
+// Copyright 2015, Camilo Aguilar
 #include "bindings.h"
 
 /* size of the event structure, not counting name */
@@ -8,325 +8,317 @@
 #define BUF_LEN        (1024 * (EVENT_SIZE + 16))
 
 namespace NodeInotify {
-    static Persistent<String> path_sym;
-    static Persistent<String> watch_for_sym;
-    static Persistent<String> callback_sym;
-    static Persistent<String> persistent_sym;
-    static Persistent<String> watch_sym;
-    static Persistent<String> mask_sym;
-    static Persistent<String> cookie_sym;
-    static Persistent<String> name_sym;
-
-    void Inotify::Initialize(Handle<Object> target) {
-        Local<FunctionTemplate> t = FunctionTemplate::New(Inotify::New);
-
-        t->InstanceTemplate()->SetInternalFieldCount(1);
-
-        NODE_SET_PROTOTYPE_METHOD(t, "addWatch",
-                                      Inotify::AddWatch);
-        NODE_SET_PROTOTYPE_METHOD(t, "removeWatch",
-                                      Inotify::RemoveWatch);
-        NODE_SET_PROTOTYPE_METHOD(t, "close",
-                                      Inotify::Close);
-
-        //Constants initialization
-        NODE_DEFINE_CONSTANT(t, IN_ACCESS); //File was accessed (read)
-        NODE_DEFINE_CONSTANT(t, IN_ATTRIB); //Metadata changed, e.g., permissions, timestamps,
-                                                      //extended attributes, link count (since Linux 2.6.25),
-                                                      //UID, GID, etc.
-        NODE_DEFINE_CONSTANT(t, IN_CLOSE_WRITE); //File opened for writing was closed
-        NODE_DEFINE_CONSTANT(t, IN_CLOSE_NOWRITE); //File not opened for writing was closed
-        NODE_DEFINE_CONSTANT(t, IN_CREATE); //File/directory created in watched directory
-        NODE_DEFINE_CONSTANT(t, IN_DELETE); //File/directory deleted from watched directory
-        NODE_DEFINE_CONSTANT(t, IN_DELETE_SELF); //Watched file/directory was itself deleted
-        NODE_DEFINE_CONSTANT(t, IN_MODIFY); //File was modified
-        NODE_DEFINE_CONSTANT(t, IN_MOVE_SELF); //Watched file/directory was itself moved
-        NODE_DEFINE_CONSTANT(t, IN_MOVED_FROM); //File moved out of watched directory
-        NODE_DEFINE_CONSTANT(t, IN_MOVED_TO); //File moved into watched directory
-        NODE_DEFINE_CONSTANT(t, IN_OPEN); //File was opened
-        NODE_DEFINE_CONSTANT(t, IN_IGNORED); // Watch was removed explicitly (inotify.watch.rm) or
-                                                       //automatically (file was deleted, or file system was
-                                                        //unmounted)
-        NODE_DEFINE_CONSTANT(t, IN_ISDIR); //Subject of this event is a directory
-        NODE_DEFINE_CONSTANT(t, IN_Q_OVERFLOW); //Event queue overflowed (wd is -1 for this event)
-        NODE_DEFINE_CONSTANT(t, IN_UNMOUNT); //File system containing watched object was unmounted
-        NODE_DEFINE_CONSTANT(t, IN_ALL_EVENTS);
-
-        NODE_DEFINE_CONSTANT(t, IN_ONLYDIR); // Only watch the path if it is a directory.
-        NODE_DEFINE_CONSTANT(t, IN_DONT_FOLLOW); // Do not follow a sym link
-        NODE_DEFINE_CONSTANT(t, IN_ONESHOT); // Only send event once
-        NODE_DEFINE_CONSTANT(t, IN_MASK_ADD); //Add (OR) events to watch mask for this pathname if it
-                                                        //already exists (instead of replacing mask).
-
-        NODE_DEFINE_CONSTANT(t, IN_CLOSE); // (IN_CLOSE_WRITE | IN_CLOSE_NOWRITE)  Close
-        NODE_DEFINE_CONSTANT(t, IN_MOVE);  //  (IN_MOVED_FROM | IN_MOVED_TO)  Moves
-
-        path_sym        = NODE_PSYMBOL("path");
-        watch_for_sym   = NODE_PSYMBOL("watch_for");
-        callback_sym    = NODE_PSYMBOL("callback");
-        persistent_sym  = NODE_PSYMBOL("persistent");
-
-        watch_sym       = NODE_PSYMBOL("watch");
-        mask_sym        = NODE_PSYMBOL("mask");
-        cookie_sym      = NODE_PSYMBOL("cookie");
-        name_sym        = NODE_PSYMBOL("name");
-
-        Local<ObjectTemplate> object_tmpl = t->InstanceTemplate();
-        object_tmpl->SetAccessor(persistent_sym, Inotify::GetPersistent);
-
-        t->SetClassName(String::NewSymbol("Inotify"));
-        target->Set(String::NewSymbol("Inotify"), t->GetFunction());
-    }
-
-    Inotify::Inotify() : ObjectWrap() {
-        //ev_init(&read_watcher, Inotify::Callback);
-        read_watcher = new uv_poll_t;
-        read_watcher->data = this;  //preserving my reference to use it inside Inotify::Callback
-        //uv_poll_init(uv_default_loop(), &read_watcher, Inotify::Callback);
-        persistent = true;
-        poll_stopped = 0;
-    }
-
-    Inotify::Inotify(bool nonpersistent) : ObjectWrap() {
-        read_watcher = new uv_poll_t;
-        read_watcher->data = this; //preserving my reference so that we can use it inside Inotify::Callback
-        //ev_init(&read_watcher, Inotify::Callback);
-        persistent = nonpersistent;
-        poll_stopped = 0;
-    }
-
-    Inotify::~Inotify() {
-        if(!persistent) {
-            //ev_ref(EV_DEFAULT_UC);
-            uv_ref((uv_handle_t *) read_watcher);
-        }
-        //ev_io_stop(EV_DEFAULT_UC_ &read_watcher);
-
-        // if Inotify::Close() was already called we do not need to
-        // stop polling again thus it causes fail of assertion test
-        StopPolling();
-
-        //assert(!uv_is_pending(&read_watcher));
-    }
-
-    Handle<Value> Inotify::New(const Arguments& args) {
-        HandleScope scope;
-
-        Inotify *inotify = NULL;
-        if(args.Length() == 1 && args[0]->IsBoolean()) {
-            inotify = new Inotify(args[0]->IsTrue());
-        } else {
-            inotify = new Inotify();
-        }
-
- 	inotify->fd = inotify_init();
-
-        if(inotify->fd == -1) {
-            ThrowException(String::New(strerror(errno)));
-            return Null();
-        }
-	
-	int flags = fcntl(inotify->fd, F_GETFL);
-	if(flags == -1) {
-	    flags = 0;
+	static Persistent<String> path_sym;
+	static Persistent<String> watch_for_sym;
+	static Persistent<String> callback_sym;
+	static Persistent<String> persistent_sym;
+	static Persistent<String> watch_sym;
+	static Persistent<String> mask_sym;
+	static Persistent<String> cookie_sym;
+	static Persistent<String> name_sym;
+
+	void Inotify::Initialize(Handle<Object> exports) {
+		Local<FunctionTemplate> t = NanNew<FunctionTemplate>(New);
+		t->SetClassName(NanNew<String>("Inotify"));
+		t->InstanceTemplate()->SetInternalFieldCount(1);
+
+		NODE_SET_PROTOTYPE_METHOD(t, "addWatch", Inotify::AddWatch);
+		NODE_SET_PROTOTYPE_METHOD(t, "removeWatch", Inotify::RemoveWatch);
+		NODE_SET_PROTOTYPE_METHOD(t, "close", Inotify::Close);
+
+		//Constants initialization
+		NODE_DEFINE_CONSTANT(exports, IN_ACCESS); //File was accessed (read)
+		NODE_DEFINE_CONSTANT(exports, IN_ATTRIB); //Metadata changed, e.g., permissions, timestamps,
+													  //extended attributes, link count (since Linux 2.6.25),
+													  //UID, GID, etc.
+		NODE_DEFINE_CONSTANT(exports, IN_CLOSE_WRITE); //File opened for writing was closed
+		NODE_DEFINE_CONSTANT(exports, IN_CLOSE_NOWRITE); //File not opened for writing was closed
+		NODE_DEFINE_CONSTANT(exports, IN_CREATE); //File/directory created in watched directory
+		NODE_DEFINE_CONSTANT(exports, IN_DELETE); //File/directory deleted from watched directory
+		NODE_DEFINE_CONSTANT(exports, IN_DELETE_SELF); //Watched file/directory was itself deleted
+		NODE_DEFINE_CONSTANT(exports, IN_MODIFY); //File was modified
+		NODE_DEFINE_CONSTANT(exports, IN_MOVE_SELF); //Watched file/directory was itself moved
+		NODE_DEFINE_CONSTANT(exports, IN_MOVED_FROM); //File moved out of watched directory
+		NODE_DEFINE_CONSTANT(exports, IN_MOVED_TO); //File moved into watched directory
+		NODE_DEFINE_CONSTANT(exports, IN_OPEN); //File was opened
+		NODE_DEFINE_CONSTANT(exports, IN_IGNORED);// Watch was removed explicitly (inotify.watch.rm) or
+											// automatically (file was deleted, or file system was
+											// unmounted)
+		NODE_DEFINE_CONSTANT(exports, IN_ISDIR); //Subject of this event is a directory
+		NODE_DEFINE_CONSTANT(exports, IN_Q_OVERFLOW); //Event queue overflowed (wd is -1 for this event)
+		NODE_DEFINE_CONSTANT(exports, IN_UNMOUNT); //File system containing watched object was unmounted
+		NODE_DEFINE_CONSTANT(exports, IN_ALL_EVENTS);
+
+		NODE_DEFINE_CONSTANT(exports, IN_ONLYDIR); // Only watch the path if it is a directory.
+		NODE_DEFINE_CONSTANT(exports, IN_DONT_FOLLOW); // Do not follow a sym link
+		NODE_DEFINE_CONSTANT(exports, IN_ONESHOT); // Only send event once
+		NODE_DEFINE_CONSTANT(exports, IN_MASK_ADD); //Add (OR) events to watch mask for this pathname if it
+											// already exists (instead of replacing mask).
+
+		NODE_DEFINE_CONSTANT(exports, IN_CLOSE); // (IN_CLOSE_WRITE | IN_CLOSE_NOWRITE)  Close
+		NODE_DEFINE_CONSTANT(exports, IN_MOVE);  //  (IN_MOVED_FROM | IN_MOVED_TO)  Moves
+
+		NanAssignPersistent(path_sym, NanNew<String>("path"));
+		NanAssignPersistent(watch_for_sym, NanNew<String>("watch_for"));
+		NanAssignPersistent(callback_sym, NanNew<String>("callback"));
+		NanAssignPersistent(persistent_sym, NanNew<String>("persistent"));
+		NanAssignPersistent(watch_sym, NanNew<String>("watch"));
+		NanAssignPersistent(mask_sym, NanNew<String>("mask"));
+		NanAssignPersistent(cookie_sym, NanNew<String>("cookie"));
+		NanAssignPersistent(name_sym, NanNew<String>("name"));
+
+		Local<ObjectTemplate> object_tmpl = t->InstanceTemplate();
+		object_tmpl->SetAccessor(persistent_sym, Inotify::GetPersistent);
+
+		exports->Set(NanNew<String>("Inotify"), t->GetFunction());
 	}
 
-	fcntl(inotify->fd, F_SETFL, flags | O_NONBLOCK);
-
-        //ev_io_set(&inotify->read_watcher, inotify->fd, EV_READ);
-        //ev_io_start(EV_DEFAULT_UC_ &inotify->read_watcher);
-        uv_poll_init(uv_default_loop(), inotify->read_watcher, inotify->fd);
-        uv_poll_start(inotify->read_watcher, UV_READABLE, Inotify::Callback); 
-
-	
-        Local<Object> obj = args.This();
-        inotify->Wrap(obj);
-
-        if(!inotify->persistent) {
-            //ev_unref(EV_DEFAULT_UC);
-            uv_unref((uv_handle_t *) inotify->read_watcher);
-        }
-        /**
- 	 * Increment object references to avoid be GCed while
-   	 * we are waiting for events in the inotify file descriptor.
-         * Also, the object is not weak anymore. 
-	 */
-        inotify->Ref();
-
-        return scope.Close(obj);
-    }
-
-    Handle<Value> Inotify::AddWatch(const Arguments& args) {
-        HandleScope scope;
-        uint32_t mask = 0;
-        int watch_descriptor = 0;
-
-        if(args.Length() < 1 || !args[0]->IsObject()) {
-            return ThrowException(Exception::TypeError(
-            String::New("You must specify an object as first argument")));
-        }
-
-        Local<Object> args_ = args[0]->ToObject();
-
-        if(!args_->Has(path_sym)) {
-            return ThrowException(Exception::TypeError(
-            String::New("You must specify a path to watch for events")));
-        }
-
-        if(!args_->Has(callback_sym) ||
-            !args_->Get(callback_sym)->IsFunction()) {
-            return ThrowException(Exception::TypeError(
-            String::New("You must specify a callback function")));
-        }
-
-        if(!args_->Has(watch_for_sym)) {
-            mask |= IN_ALL_EVENTS;
-        } else {
-            if(!args_->Get(watch_for_sym)->IsInt32()) {
-                return ThrowException(Exception::TypeError(
-                String::New("You must specify OR'ed set of events")));
-            }
-            mask |= args_->Get(watch_for_sym)->Int32Value();
-            if(mask == 0) {
-                return ThrowException(Exception::TypeError(
-                String::New("You must specify OR'ed set of events")));
-            }
-       }
-
-        String::Utf8Value path(args_->Get(path_sym));
-
-        Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
-
-        /* add watch */
-        watch_descriptor = inotify_add_watch(inotify->fd, (const char *) *path, mask);
-
-        Local<Integer> descriptor = Integer::New(watch_descriptor);
-
-        //Local<Function> callback = Local<Function>::Cast(args_->Get(callback_sym));
-        inotify->handle_->Set(descriptor, args_->Get(callback_sym));
-
-        return scope.Close(descriptor);
-    }
-
-    Handle<Value> Inotify::RemoveWatch(const Arguments& args) {
-        HandleScope scope;
-        uint32_t watch = 0;
-        int ret = -1;
-
-        if(args.Length() == 0 || !args[0]->IsInt32()) {
-            return ThrowException(Exception::TypeError(
-            String::New("You must specify a valid watcher descriptor as argument")));
-        }
-        watch = args[0]->Int32Value();
-
-        Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
-
-        ret = inotify_rm_watch(inotify->fd, watch);
-        if(ret == -1) {
-            ThrowException(String::New(strerror(errno)));
-            return False();
-        }
-
-        return True();
-    }
-
-    void Inotify::on_handle_close(uv_handle_t* handle) {
-        assert(!uv_is_active(handle));
-        delete handle;
-    }
-    
-    Handle<Value> Inotify::Close(const Arguments& args) {
-        HandleScope scope;
-        int ret = -1;
-
-        Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
-        ret = close(inotify->fd);
-
-        if(ret == -1) {
-            ThrowException(String::New(strerror(errno)));
-            return False();
-        }
-
-        if(!inotify->persistent) {
-            //ev_ref(EV_DEFAULT_UC);
-            uv_ref((uv_handle_t *) inotify->read_watcher);
-        }
-
-        //ev_io_stop(EV_DEFAULT_UC_ &inotify->read_watcher);
-        inotify->StopPolling();
-	
-        /*Eliminating reference created inside of Inotify::New.
-        The object is also weak again.
-        Now v8 can do its stuff and GC the object.
-        */
-        inotify->Unref();
-
-        return True();
-    }
-
-    void Inotify::Callback(uv_poll_t *watcher, int status, int revents) {
-        HandleScope scope;
-
-        Inotify *inotify = static_cast<Inotify*>(watcher->data);
-        assert(watcher == inotify->read_watcher);
-
-        char buffer[BUF_LEN];
-
-        //int length = read(inotify->fd, buffer, BUF_LEN);
-
-        Local<Value> argv[1];
-        TryCatch try_catch;
-
-        int i = 0;
-	int sz = 0;
-        while ((sz = read(inotify->fd, buffer, BUF_LEN)) > 0) {
-	  struct inotify_event *event;
-	  for (i = 0; i <= (sz-EVENT_SIZE); i += (EVENT_SIZE + event->len)) {
-            event = (struct inotify_event *) &buffer[i];
-
-            Local<Object> obj = Object::New();
-            obj->Set(watch_sym, Integer::New(event->wd));
-            obj->Set(mask_sym, Integer::New(event->mask));
-            obj->Set(cookie_sym, Integer::New(event->cookie));
-
-            if(event->len) {
-                obj->Set(name_sym, String::New(event->name));
-            }
-            argv[0] = obj;
-
-            inotify->Ref();
-            Local<Value> callback_ = inotify->handle_->Get(Integer::New(event->wd));
-            Local<Function> callback = Local<Function>::Cast(callback_);
-
-            callback->Call(inotify->handle_, 1, argv);
-            inotify->Unref();
-
-            if(event->mask & IN_IGNORED) {
-                //deleting callback because the watch was removed
-                Local<Value> wd = Integer::New(event->wd);
-                inotify->handle_->Delete(wd->ToString());
-            }
-
-            if (try_catch.HasCaught()) {
-                FatalException(try_catch);
-            }
-	  }
+	Inotify::Inotify() : ObjectWrap() {
+		//ev_init(&read_watcher, Inotify::Callback);
+		read_watcher = new uv_poll_t;
+		read_watcher->data = this;  //preserving my reference to use it inside Inotify::Callback
+		//uv_poll_init(uv_default_loop(), &read_watcher, Inotify::Callback);
+		persistent = true;
+		poll_stopped = 0;
+	}
+
+	Inotify::Inotify(bool nonpersistent) : ObjectWrap() {
+		read_watcher = new uv_poll_t;
+		read_watcher->data = this; //preserving my reference so that we can use it inside Inotify::Callback
+		//ev_init(&read_watcher, Inotify::Callback);
+		persistent = nonpersistent;
+		poll_stopped = 0;
+	}
+
+	Inotify::~Inotify() {
+		if(!persistent) {
+			//ev_ref(EV_DEFAULT_UC);
+			uv_ref((uv_handle_t *) read_watcher);
+		}
+		//ev_io_stop(EV_DEFAULT_UC_ &read_watcher);
+
+		// if Inotify::Close() was already called we do not need to
+		// stop polling again thus it causes fail of assertion test
+		StopPolling();
+
+		//assert(!uv_is_pending(&read_watcher));
+	}
+
+	NAN_METHOD(Inotify::New) {
+		NanEscapableScope();
+
+		Inotify *inotify = NULL;
+		if(args.Length() == 1 && args[0]->IsBoolean()) {
+			inotify = new Inotify(args[0]->IsTrue());
+		} else {
+			inotify = new Inotify();
+		}
+
+		inotify->fd = inotify_init();
+
+		if(inotify->fd == -1) {
+			NanThrowError(NanNew<String>(strerror(errno)));
+			NanReturnValue(NanNull());
+		}
+
+		int flags = fcntl(inotify->fd, F_GETFL);
+		if(flags == -1) {
+			flags = 0;
+		}
+
+		fcntl(inotify->fd, F_SETFL, flags | O_NONBLOCK);
+
+		//ev_io_set(&inotify->read_watcher, inotify->fd, EV_READ);
+		//ev_io_start(EV_DEFAULT_UC_ &inotify->read_watcher);
+		uv_poll_init(uv_default_loop(), inotify->read_watcher, inotify->fd);
+		uv_poll_start(inotify->read_watcher, UV_READABLE, Inotify::Callback);
+
+
+		Local<Object> obj = args.This();
+		inotify->Wrap(obj);
+
+		if(!inotify->persistent) {
+			//ev_unref(EV_DEFAULT_UC);
+			uv_unref((uv_handle_t *) inotify->read_watcher);
+		}
+		/**
+		* Increment object references to avoid be GCed while
+		* we are waiting for events in the inotify file descriptor.
+		* Also, the object is not weak anymore.
+		*/
+		inotify->Ref();
+
+		return NanEscapeScope(obj);
+	}
+
+	NAN_METHOD(Inotify::AddWatch) {
+		NanEscapableScope();
+		uint32_t mask = 0;
+		int watch_descriptor = 0;
+
+		if(args.Length() < 1 || !args[0]->IsObject()) {
+			return NanThrowTypeError("You must specify an object as first argument");
+		}
+
+		Local<Object> args_ = args[0]->ToObject();
+
+		if(!args_->Has(path_sym)) {
+			return NanThrowTypeError("You must specify a path to watch for events");
+		}
+
+		if(!args_->Has(callback_sym) ||
+			!args_->Get(callback_sym)->IsFunction()) {
+			return NanThrowTypeError("You must specify a callback function");
+		}
+
+		if(!args_->Has(watch_for_sym)) {
+			mask |= IN_ALL_EVENTS;
+		} else {
+			if(!args_->Get(watch_for_sym)->IsInt32()) {
+				return NanThrowTypeError("You must specify OR'ed set of events");
+			}
+			mask |= args_->Get(watch_for_sym)->Int32Value();
+			if(mask == 0) {
+				return NanThrowTypeError("You must specify OR'ed set of events");
+			}
+	   }
+
+		String::Utf8Value path(args_->Get(path_sym));
+
+		Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
+
+		/* add watch */
+		watch_descriptor = inotify_add_watch(inotify->fd, (const char *) *path, mask);
+
+		Local<Integer> descriptor = NanNew<Integer>(watch_descriptor);
+
+		//Local<Function> callback = Local<Function>::Cast(args_->Get(callback_sym));
+		inotify->handle_->Set(descriptor, args_->Get(callback_sym));
+
+		return NanEscapeScope(descriptor);
+	}
+
+	NAN_METHOD(Inotify::RemoveWatch) {
+		NanScope();
+		uint32_t watch = 0;
+		int ret = -1;
+
+		if(args.Length() == 0 || !args[0]->IsInt32()) {
+			return NanThrowTypeError("You must specify a valid watcher descriptor as argument");
+		}
+		watch = args[0]->Int32Value();
+
+		Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
+
+		ret = inotify_rm_watch(inotify->fd, watch);
+		if(ret == -1) {
+			NanThrowError(NanNew<String>(strerror(errno)));
+			NanReturnValue(NanFalse());
+		}
+
+		NanReturnValue(NanTrue());
+	}
+
+	void Inotify::on_handle_close(uv_handle_t* handle) {
+		assert(!uv_is_active(handle));
+		delete handle;
+	}
+
+	NAN_METHOD(Inotify::Close) {
+		NanScope();
+		int ret = -1;
+
+		Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
+		ret = close(inotify->fd);
+
+		if (ret == -1) {
+			NanThrowError(NanNew<String>(strerror(errno)));
+			NanReturnValue(NanFalse());
+		}
+
+		if (!inotify->persistent) {
+			//ev_ref(EV_DEFAULT_UC);
+			uv_ref((uv_handle_t *) inotify->read_watcher);
+		}
+
+		//ev_io_stop(EV_DEFAULT_UC_ &inotify->read_watcher);
+		inotify->StopPolling();
+
+		/*Eliminating reference created inside of Inotify::New.
+		The object is also weak again.
+		Now v8 can do its stuff and GC the object.
+		*/
+		inotify->Unref();
+
+		NanReturnValue(NanTrue());
+	}
+
+	void Inotify::Callback(uv_poll_t *watcher, int status, int revents) {
+		NanScope();
+
+		Inotify *inotify = static_cast<Inotify*>(watcher->data);
+		assert(watcher == inotify->read_watcher);
+
+		char buffer[BUF_LEN];
+
+		//int length = read(inotify->fd, buffer, BUF_LEN);
+
+		Local<Value> argv[1];
+		TryCatch try_catch;
+
+		int i = 0;
+		int sz = 0;
+		while ((sz = read(inotify->fd, buffer, BUF_LEN)) > 0) {
+			struct inotify_event *event;
+			for (i = 0; i <= (sz-EVENT_SIZE); i += (EVENT_SIZE + event->len)) {
+				event = (struct inotify_event *) &buffer[i];
+
+				Local<Object> obj = NanNew<Object>();
+				obj->Set(watch_sym, NanNew<Integer>(event->wd));
+				obj->Set(mask_sym, NanNew<Integer>(event->mask));
+				obj->Set(cookie_sym, NanNew<Integer>(event->cookie));
+
+				if(event->len) {
+					obj->Set(name_sym, NanNew<String>(event->name));
+				}
+				argv[0] = obj;
+
+				inotify->Ref();
+				Local<Value> callback_ = inotify->handle_->Get(NanNew<Integer>(event->wd));
+				Local<Function> callback = Local<Function>::Cast(callback_);
+
+				callback->Call(inotify->handle_, 1, argv);
+				inotify->Unref();
+
+				if(event->mask & IN_IGNORED) {
+					//deleting callback because the watch was removed
+					Local<Value> wd = NanNew<Integer>(event->wd);
+					inotify->handle_->Delete(wd->ToString());
+				}
+
+				if (try_catch.HasCaught()) {
+					FatalException(try_catch);
+				}
+			} // for
+		} // while
+	}
+
+	NAN_GETTER(Inotify::GetPersistent) {
+		Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
+
+		if (inotify->persistent) {
+			NanReturnValue(NanTrue());
+		}
+
+		NanReturnValue(NanFalse());
+	 }
+
+	void Inotify::StopPolling() {
+		if (!poll_stopped) {
+			uv_poll_stop(read_watcher);
+			uv_close((uv_handle_t *) read_watcher, Inotify::on_handle_close);
+			poll_stopped = 1;
+		}
 	}
-    }
-
-    Handle<Value> Inotify::GetPersistent(Local<String> property,
-                                        const AccessorInfo& info) {
-        Inotify *inotify = ObjectWrap::Unwrap<Inotify>(info.This());
-
-        return inotify->persistent ? True() : False();
-     }
-
-    void Inotify::StopPolling() {
-        if (!poll_stopped) {
-            uv_poll_stop(read_watcher);
-            uv_close((uv_handle_t *) read_watcher, Inotify::on_handle_close);
-            poll_stopped = 1;
-        }
-    }
 
 }//namespace NodeInotify
diff --git a/src/bindings.h b/src/bindings.h
index 646fafb..5d07209 100644
--- a/src/bindings.h
+++ b/src/bindings.h
@@ -1,4 +1,4 @@
-// Copyright 2010, Camilo Aguilar. Cloudescape, LLC.
+// Copyright 2015, Camilo Aguilar.
 #ifndef SRC_BINDINGS_H_
 #define SRC_BINDINGS_H_
 #include "node_inotify.h"
@@ -13,12 +13,12 @@ namespace NodeInotify {
             Inotify(bool nonpersistent);
             virtual ~Inotify();
         protected:
-            static Handle<Value> New(const Arguments& args);
-            static Handle<Value> AddWatch(const Arguments& args);
-            static Handle<Value> RemoveWatch(const Arguments& args);
-            static Handle<Value> Close(const Arguments& args);
-            static Handle<Value> GetPersistent(Local<String> property,
-                                             const AccessorInfo& info);
+            static NAN_METHOD(New);
+            static NAN_METHOD(AddWatch);
+            static NAN_METHOD(RemoveWatch);
+            static NAN_METHOD(Close);
+            static NAN_GETTER(GetPersistent);
+
         private:
             int fd;
             uv_poll_t* read_watcher;
diff --git a/src/node_inotify.cc b/src/node_inotify.cc
index a41a4e4..0699375 100644
--- a/src/node_inotify.cc
+++ b/src/node_inotify.cc
@@ -1,25 +1,25 @@
-// Copyright 2010, Camilo Aguilar. Cloudescape, LLC.
+// Copyright 2015, Camilo Aguilar.
 #include "bindings.h"
 
 namespace NodeInotify {
-    void InitializeInotify(Handle<Object> target) {
-        HandleScope scope;
+    void InitializeInotify(Handle<Object> exports) {
+        NanScope();
 
-        Inotify::Initialize(target);
+        Inotify::Initialize(exports);
 
-        target->Set(String::NewSymbol("version"),
+        exports->Set(NanNew<String>("version"),
                     String::New(NODE_INOTIFY_VERSION));
 
         Handle<ObjectTemplate> global = ObjectTemplate::New();
         Handle<Context> context = Context::New(NULL, global);
         Context::Scope context_scope(context);
 
-        context->Global()->Set(String::NewSymbol("Inotify"), target);
+        context->Global()->Set(NanNew<String>("Inotify"), exports);
     }
 
-    extern "C" void init (Handle<Object> target) {
-        HandleScope scope;
-        InitializeInotify(target);
+    extern "C" void init (Handle<Object> exports) {
+        NanScope();
+        InitializeInotify(exports);
     }
 } //namespace NodeInotify
 
diff --git a/src/node_inotify.h b/src/node_inotify.h
index 126ecba..ad27770 100644
--- a/src/node_inotify.h
+++ b/src/node_inotify.h
@@ -10,7 +10,8 @@
 #include <string.h>
 #include <unistd.h>
 #include <fcntl.h>
-#define NODE_INOTIFY_VERSION "1.2.1"
+#include "nan.h"
+#define NODE_INOTIFY_VERSION "1.3.0"
 
 using namespace v8;
 using namespace node;

From 632d2c817ef984d3d8d98908f85e1723de660e25 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 19:53:05 -0400
Subject: [PATCH 2/6] No need to make symbols persistent handles anymore.

---
 src/bindings.cc | 40 ++++++++++++----------------------------
 1 file changed, 12 insertions(+), 28 deletions(-)

diff --git a/src/bindings.cc b/src/bindings.cc
index 611051d..3909c93 100644
--- a/src/bindings.cc
+++ b/src/bindings.cc
@@ -8,15 +8,6 @@
 #define BUF_LEN        (1024 * (EVENT_SIZE + 16))
 
 namespace NodeInotify {
-	static Persistent<String> path_sym;
-	static Persistent<String> watch_for_sym;
-	static Persistent<String> callback_sym;
-	static Persistent<String> persistent_sym;
-	static Persistent<String> watch_sym;
-	static Persistent<String> mask_sym;
-	static Persistent<String> cookie_sym;
-	static Persistent<String> name_sym;
-
 	void Inotify::Initialize(Handle<Object> exports) {
 		Local<FunctionTemplate> t = NanNew<FunctionTemplate>(New);
 		t->SetClassName(NanNew<String>("Inotify"));
@@ -58,17 +49,8 @@ namespace NodeInotify {
 		NODE_DEFINE_CONSTANT(exports, IN_CLOSE); // (IN_CLOSE_WRITE | IN_CLOSE_NOWRITE)  Close
 		NODE_DEFINE_CONSTANT(exports, IN_MOVE);  //  (IN_MOVED_FROM | IN_MOVED_TO)  Moves
 
-		NanAssignPersistent(path_sym, NanNew<String>("path"));
-		NanAssignPersistent(watch_for_sym, NanNew<String>("watch_for"));
-		NanAssignPersistent(callback_sym, NanNew<String>("callback"));
-		NanAssignPersistent(persistent_sym, NanNew<String>("persistent"));
-		NanAssignPersistent(watch_sym, NanNew<String>("watch"));
-		NanAssignPersistent(mask_sym, NanNew<String>("mask"));
-		NanAssignPersistent(cookie_sym, NanNew<String>("cookie"));
-		NanAssignPersistent(name_sym, NanNew<String>("name"));
-
 		Local<ObjectTemplate> object_tmpl = t->InstanceTemplate();
-		object_tmpl->SetAccessor(persistent_sym, Inotify::GetPersistent);
+		object_tmpl->SetAccessor(NanNew<String>("persistent"), Inotify::GetPersistent);
 
 		exports->Set(NanNew<String>("Inotify"), t->GetFunction());
 	}
@@ -162,16 +144,19 @@ namespace NodeInotify {
 
 		Local<Object> args_ = args[0]->ToObject();
 
-		if(!args_->Has(path_sym)) {
+		Local<String> path_sym = NanNew<String>("path");
+		if (!args_->Has(path_sym)) {
 			return NanThrowTypeError("You must specify a path to watch for events");
 		}
 
-		if(!args_->Has(callback_sym) ||
+		Local<String> callback_sym = NanNew<String>("callback");
+		if (!args_->Has(callback_sym) ||
 			!args_->Get(callback_sym)->IsFunction()) {
 			return NanThrowTypeError("You must specify a callback function");
 		}
 
-		if(!args_->Has(watch_for_sym)) {
+		Local<String> watch_for_sym = NanNew<String>("watch_for");
+		if (!args_->Has(watch_for_sym)) {
 			mask |= IN_ALL_EVENTS;
 		} else {
 			if(!args_->Get(watch_for_sym)->IsInt32()) {
@@ -266,20 +251,19 @@ namespace NodeInotify {
 		Local<Value> argv[1];
 		TryCatch try_catch;
 
-		int i = 0;
 		int sz = 0;
 		while ((sz = read(inotify->fd, buffer, BUF_LEN)) > 0) {
 			struct inotify_event *event;
-			for (i = 0; i <= (sz-EVENT_SIZE); i += (EVENT_SIZE + event->len)) {
+			for (unsigned int i = 0; i <= (sz-EVENT_SIZE); i += (EVENT_SIZE + event->len)) {
 				event = (struct inotify_event *) &buffer[i];
 
 				Local<Object> obj = NanNew<Object>();
-				obj->Set(watch_sym, NanNew<Integer>(event->wd));
-				obj->Set(mask_sym, NanNew<Integer>(event->mask));
-				obj->Set(cookie_sym, NanNew<Integer>(event->cookie));
+				obj->Set(NanNew<String>("watch"), NanNew<Integer>(event->wd));
+				obj->Set(NanNew<String>("mask"), NanNew<Integer>(event->mask));
+				obj->Set(NanNew<String>("cookie"), NanNew<Integer>(event->cookie));
 
 				if(event->len) {
-					obj->Set(name_sym, NanNew<String>(event->name));
+					obj->Set(NanNew<String>("name"), NanNew<String>(event->name));
 				}
 				argv[0] = obj;
 

From 7e08d3f5afb187d1b08405a383cc02109d51e119 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 20:25:20 -0400
Subject: [PATCH 3/6] Finishes porting to NAN.

---
 src/bindings.cc     | 81 +++++++++++++++++++++++----------------------
 src/node_inotify.cc |  4 +--
 2 files changed, 43 insertions(+), 42 deletions(-)

diff --git a/src/bindings.cc b/src/bindings.cc
index 3909c93..98c6299 100644
--- a/src/bindings.cc
+++ b/src/bindings.cc
@@ -17,42 +17,43 @@ namespace NodeInotify {
 		NODE_SET_PROTOTYPE_METHOD(t, "removeWatch", Inotify::RemoveWatch);
 		NODE_SET_PROTOTYPE_METHOD(t, "close", Inotify::Close);
 
-		//Constants initialization
-		NODE_DEFINE_CONSTANT(exports, IN_ACCESS); //File was accessed (read)
-		NODE_DEFINE_CONSTANT(exports, IN_ATTRIB); //Metadata changed, e.g., permissions, timestamps,
+		Local<ObjectTemplate> object_tmpl = t->InstanceTemplate();
+		object_tmpl->SetAccessor(NanNew<String>("persistent"), Inotify::GetPersistent);
+
+		Local<Function> fn = t->GetFunction();
+		exports->Set(NanNew<String>("Inotify"), fn);
+
+		// Constants initialization
+		NODE_DEFINE_CONSTANT(fn, IN_ACCESS); //File was accessed (read)
+		NODE_DEFINE_CONSTANT(fn, IN_ATTRIB); //Metadata changed, e.g., permissions, timestamps,
 													  //extended attributes, link count (since Linux 2.6.25),
 													  //UID, GID, etc.
-		NODE_DEFINE_CONSTANT(exports, IN_CLOSE_WRITE); //File opened for writing was closed
-		NODE_DEFINE_CONSTANT(exports, IN_CLOSE_NOWRITE); //File not opened for writing was closed
-		NODE_DEFINE_CONSTANT(exports, IN_CREATE); //File/directory created in watched directory
-		NODE_DEFINE_CONSTANT(exports, IN_DELETE); //File/directory deleted from watched directory
-		NODE_DEFINE_CONSTANT(exports, IN_DELETE_SELF); //Watched file/directory was itself deleted
-		NODE_DEFINE_CONSTANT(exports, IN_MODIFY); //File was modified
-		NODE_DEFINE_CONSTANT(exports, IN_MOVE_SELF); //Watched file/directory was itself moved
-		NODE_DEFINE_CONSTANT(exports, IN_MOVED_FROM); //File moved out of watched directory
-		NODE_DEFINE_CONSTANT(exports, IN_MOVED_TO); //File moved into watched directory
-		NODE_DEFINE_CONSTANT(exports, IN_OPEN); //File was opened
-		NODE_DEFINE_CONSTANT(exports, IN_IGNORED);// Watch was removed explicitly (inotify.watch.rm) or
+		NODE_DEFINE_CONSTANT(fn, IN_CLOSE_WRITE); //File opened for writing was closed
+		NODE_DEFINE_CONSTANT(fn, IN_CLOSE_NOWRITE); //File not opened for writing was closed
+		NODE_DEFINE_CONSTANT(fn, IN_CREATE); //File/directory created in watched directory
+		NODE_DEFINE_CONSTANT(fn, IN_DELETE); //File/directory deleted from watched directory
+		NODE_DEFINE_CONSTANT(fn, IN_DELETE_SELF); //Watched file/directory was itself deleted
+		NODE_DEFINE_CONSTANT(fn, IN_MODIFY); //File was modified
+		NODE_DEFINE_CONSTANT(fn, IN_MOVE_SELF); //Watched file/directory was itself moved
+		NODE_DEFINE_CONSTANT(fn, IN_MOVED_FROM); //File moved out of watched directory
+		NODE_DEFINE_CONSTANT(fn, IN_MOVED_TO); //File moved into watched directory
+		NODE_DEFINE_CONSTANT(fn, IN_OPEN); //File was opened
+		NODE_DEFINE_CONSTANT(fn, IN_IGNORED);// Watch was removed explicitly (inotify.watch.rm) or
 											// automatically (file was deleted, or file system was
 											// unmounted)
-		NODE_DEFINE_CONSTANT(exports, IN_ISDIR); //Subject of this event is a directory
-		NODE_DEFINE_CONSTANT(exports, IN_Q_OVERFLOW); //Event queue overflowed (wd is -1 for this event)
-		NODE_DEFINE_CONSTANT(exports, IN_UNMOUNT); //File system containing watched object was unmounted
-		NODE_DEFINE_CONSTANT(exports, IN_ALL_EVENTS);
-
-		NODE_DEFINE_CONSTANT(exports, IN_ONLYDIR); // Only watch the path if it is a directory.
-		NODE_DEFINE_CONSTANT(exports, IN_DONT_FOLLOW); // Do not follow a sym link
-		NODE_DEFINE_CONSTANT(exports, IN_ONESHOT); // Only send event once
-		NODE_DEFINE_CONSTANT(exports, IN_MASK_ADD); //Add (OR) events to watch mask for this pathname if it
+		NODE_DEFINE_CONSTANT(fn, IN_ISDIR); //Subject of this event is a directory
+		NODE_DEFINE_CONSTANT(fn, IN_Q_OVERFLOW); //Event queue overflowed (wd is -1 for this event)
+		NODE_DEFINE_CONSTANT(fn, IN_UNMOUNT); //File system containing watched object was unmounted
+		NODE_DEFINE_CONSTANT(fn, IN_ALL_EVENTS);
+
+		NODE_DEFINE_CONSTANT(fn, IN_ONLYDIR); // Only watch the path if it is a directory.
+		NODE_DEFINE_CONSTANT(fn, IN_DONT_FOLLOW); // Do not follow a sym link
+		NODE_DEFINE_CONSTANT(fn, IN_ONESHOT); // Only send event once
+		NODE_DEFINE_CONSTANT(fn, IN_MASK_ADD); //Add (OR) events to watch mask for this pathname if it
 											// already exists (instead of replacing mask).
 
-		NODE_DEFINE_CONSTANT(exports, IN_CLOSE); // (IN_CLOSE_WRITE | IN_CLOSE_NOWRITE)  Close
-		NODE_DEFINE_CONSTANT(exports, IN_MOVE);  //  (IN_MOVED_FROM | IN_MOVED_TO)  Moves
-
-		Local<ObjectTemplate> object_tmpl = t->InstanceTemplate();
-		object_tmpl->SetAccessor(NanNew<String>("persistent"), Inotify::GetPersistent);
-
-		exports->Set(NanNew<String>("Inotify"), t->GetFunction());
+		NODE_DEFINE_CONSTANT(fn, IN_CLOSE); // (IN_CLOSE_WRITE | IN_CLOSE_NOWRITE)  Close
+		NODE_DEFINE_CONSTANT(fn, IN_MOVE);  //  (IN_MOVED_FROM | IN_MOVED_TO)  Moves
 	}
 
 	Inotify::Inotify() : ObjectWrap() {
@@ -87,7 +88,7 @@ namespace NodeInotify {
 	}
 
 	NAN_METHOD(Inotify::New) {
-		NanEscapableScope();
+		NanScope();
 
 		Inotify *inotify = NULL;
 		if(args.Length() == 1 && args[0]->IsBoolean()) {
@@ -130,11 +131,11 @@ namespace NodeInotify {
 		*/
 		inotify->Ref();
 
-		return NanEscapeScope(obj);
+		NanReturnValue(obj);
 	}
 
 	NAN_METHOD(Inotify::AddWatch) {
-		NanEscapableScope();
+		NanScope();
 		uint32_t mask = 0;
 		int watch_descriptor = 0;
 
@@ -159,11 +160,11 @@ namespace NodeInotify {
 		if (!args_->Has(watch_for_sym)) {
 			mask |= IN_ALL_EVENTS;
 		} else {
-			if(!args_->Get(watch_for_sym)->IsInt32()) {
+			if (!args_->Get(watch_for_sym)->IsInt32()) {
 				return NanThrowTypeError("You must specify OR'ed set of events");
 			}
 			mask |= args_->Get(watch_for_sym)->Int32Value();
-			if(mask == 0) {
+			if (mask == 0) {
 				return NanThrowTypeError("You must specify OR'ed set of events");
 			}
 	   }
@@ -178,9 +179,9 @@ namespace NodeInotify {
 		Local<Integer> descriptor = NanNew<Integer>(watch_descriptor);
 
 		//Local<Function> callback = Local<Function>::Cast(args_->Get(callback_sym));
-		inotify->handle_->Set(descriptor, args_->Get(callback_sym));
+		inotify->handle()->Set(descriptor, args_->Get(callback_sym));
 
-		return NanEscapeScope(descriptor);
+		NanReturnValue(descriptor);
 	}
 
 	NAN_METHOD(Inotify::RemoveWatch) {
@@ -268,16 +269,16 @@ namespace NodeInotify {
 				argv[0] = obj;
 
 				inotify->Ref();
-				Local<Value> callback_ = inotify->handle_->Get(NanNew<Integer>(event->wd));
+				Local<Value> callback_ = inotify->handle()->Get(NanNew<Integer>(event->wd));
 				Local<Function> callback = Local<Function>::Cast(callback_);
 
-				callback->Call(inotify->handle_, 1, argv);
+				callback->Call(inotify->handle(), 1, argv);
 				inotify->Unref();
 
 				if(event->mask & IN_IGNORED) {
 					//deleting callback because the watch was removed
 					Local<Value> wd = NanNew<Integer>(event->wd);
-					inotify->handle_->Delete(wd->ToString());
+					inotify->handle()->Delete(wd->ToString());
 				}
 
 				if (try_catch.HasCaught()) {
diff --git a/src/node_inotify.cc b/src/node_inotify.cc
index 0699375..927210a 100644
--- a/src/node_inotify.cc
+++ b/src/node_inotify.cc
@@ -8,10 +8,10 @@ namespace NodeInotify {
         Inotify::Initialize(exports);
 
         exports->Set(NanNew<String>("version"),
-                    String::New(NODE_INOTIFY_VERSION));
+                    NanNew<String>(NODE_INOTIFY_VERSION));
 
         Handle<ObjectTemplate> global = ObjectTemplate::New();
-        Handle<Context> context = Context::New(NULL, global);
+        Handle<Context> context = NanNew<Context>(reinterpret_cast<ExtensionConfiguration *>(NULL), global);
         Context::Scope context_scope(context);
 
         context->Global()->Set(NanNew<String>("Inotify"), exports);

From 8a9e67ad3f14c0c18f6e9edfc913957f17394dc9 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 22:47:39 -0400
Subject: [PATCH 4/6] Adds support for node 0.10.x and 0.12.x in the same
 version.

---
 .gitignore                                    |  1 +
 .../default/virtualbox/action_provision       |  1 -
 .../default/virtualbox/action_set_name        |  1 -
 .vagrant/machines/default/virtualbox/id       |  1 -
 .../machines/default/virtualbox/index_uuid    |  1 -
 .../machines/default/virtualbox/private_key   | 27 ------------
 .../default/virtualbox/synced_folders         |  1 -
 Vagrantfile                                   |  6 ++-
 src/bindings.cc                               | 19 +++++++--
 src/bindings.h                                | 42 +++++++++----------
 src/node_inotify.h                            |  1 +
 11 files changed, 42 insertions(+), 59 deletions(-)
 delete mode 100644 .vagrant/machines/default/virtualbox/action_provision
 delete mode 100644 .vagrant/machines/default/virtualbox/action_set_name
 delete mode 100644 .vagrant/machines/default/virtualbox/id
 delete mode 100644 .vagrant/machines/default/virtualbox/index_uuid
 delete mode 100644 .vagrant/machines/default/virtualbox/private_key
 delete mode 100644 .vagrant/machines/default/virtualbox/synced_folders

diff --git a/.gitignore b/.gitignore
index 5c0401e..5d58655 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
+.vagrant
 node_modules
 build
 .lock-*
diff --git a/.vagrant/machines/default/virtualbox/action_provision b/.vagrant/machines/default/virtualbox/action_provision
deleted file mode 100644
index d44cf7b..0000000
--- a/.vagrant/machines/default/virtualbox/action_provision
+++ /dev/null
@@ -1 +0,0 @@
-1.5:b05f2c49-c173-406e-9614-285b9d5c34d7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/action_set_name b/.vagrant/machines/default/virtualbox/action_set_name
deleted file mode 100644
index 43fe352..0000000
--- a/.vagrant/machines/default/virtualbox/action_set_name
+++ /dev/null
@@ -1 +0,0 @@
-1430771691
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/id b/.vagrant/machines/default/virtualbox/id
deleted file mode 100644
index a2cc72b..0000000
--- a/.vagrant/machines/default/virtualbox/id
+++ /dev/null
@@ -1 +0,0 @@
-b05f2c49-c173-406e-9614-285b9d5c34d7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/index_uuid b/.vagrant/machines/default/virtualbox/index_uuid
deleted file mode 100644
index 5082f85..0000000
--- a/.vagrant/machines/default/virtualbox/index_uuid
+++ /dev/null
@@ -1 +0,0 @@
-a6fa3259af58456ebf88e203d70fa8c7
\ No newline at end of file
diff --git a/.vagrant/machines/default/virtualbox/private_key b/.vagrant/machines/default/virtualbox/private_key
deleted file mode 100644
index 4db1bb4..0000000
--- a/.vagrant/machines/default/virtualbox/private_key
+++ /dev/null
@@ -1,27 +0,0 @@
------BEGIN RSA PRIVATE KEY-----
-MIIEpAIBAAKCAQEA1xngHgMY+fm5lSLusCjiAUNzQv82UpL4ymHCAq0yoTmWug3T
-qqJRPB37k+c5rbpL3CaJ5RgLsAAGFdu+60ooMCXA43FLbK4Jt8SgzXSKA0p06opu
-XxTbgsnzNErnMmTn5SaYqE/tJP08xFAyXywOj2/VnSV3fhUZrVB78iUsFoM0V/Jf
-KRkBvj+h07glD/HA+CrF9TLZ0aDRzkRdVI7qJ/mMAg9Lqdk0JjI1+ldn95Q3AXHc
-77SBjTkifjm49ZZkQOTi7ODpka87X27jwVsKmJ89MCBhu3AiMX0zj3copvmuiY2F
-NHqMzWYpQFfkVpjeQ6L2fu92DJfiyeiIDvVMsQIDAQABAoIBAQC1/a2SlvpV3cT3
-+qNKpnx5pzDpOXcjrDgVO1rj2thgcpIlfrdcwpLKs298twElDN9MN8sp6wtQcOS1
-L8Q73PVdr0Og6+Vt4V5VfmazDmwK1r9PBarWhZYn9WfxEYLFL3zIc7xrg5xYVG5K
-hrPntCXplCx/FgkgwweDjPohaPQBYpmwCZL/qWuCs00FIxjRYwB+Aweg6ZXqEv7g
-o4DbJ/LIwQ2y0gwmXexWEX1TQvmtlYqP3fnTQVANAGU7N6HtmtYf7B+g8/OFDME2
-LgBoV4DpIzG/oBE0pxzWoXTi9NRDRRyjNHxSaM/RjHc68IkN/iTJQzp1BcF/kZlO
-XQmGk/qFAoGBAPW3PC30VpZNSohaCKdFMLX/4ZFyI+nup+mueQ/wZa5uhdOCAIdj
-eu/vZDtIt7o7UYq8/z98fENPhMdskShMXk6RYQb9WzC8DUNFvQxozx9DgkyHc/Tm
-lrRwdHVRJ7BxOlBKBe7TBDTHMyCHUpBlFjmaLebakLZhzT8ec+aKJd5HAoGBAOAa
-nSoAsXhtirSTmXEFAXnq3fGB/tfvmYzpIilaiSv8p92xto/TUzlsrfNI8cRA+uv0
-ezaKEDHzE1sOkgZhZ60vJ/ehvVDlLlWM6XUuRAwqIl65vQAbFYYo8JJOuFTIzBjl
-nHePH1MFwmqIl1FIGmel+LK7N8s8SWQcE9uIBqFHAoGAApz9RdvYY+bUiRi0CNti
-hjz/cCWPnjhMjt6tKO88lZtVTweN6+Kx9wVLwpmmYpOo7VnPOSSgybJ5zgkojg4p
-3poLNyDQrHcDiRifxxcCROnZ4+h44jzlk2uywWColGbP4PS4lW6nF8yajZkC1shM
-cTZjkJX4fggSP76aKvSDLJcCgYASDhLbb3Lc4PZr1fhmwqB70hBPhMmjXApi7+Vw
-r3qdsJ1Vnxu2OrBZ0AezsgzIYvPUTr4ehsEHRS4qB+J74R0ZNtNhgAZgnhY9jjzA
-1BlnYX2jZBN2DriNNn8rSmxPaF4JI3aXq1A6wBwJuTd3tPdrolOHEs5L8W94QUrY
-RP1DwQKBgQCJxRj9uu21ENF48Z4v87sibdLHoIjstp+g0980WW9MJjCeU3VVjGWy
-oqNat0CQyU6sEOxj/JdPd1iHvSBQxubYSARV9Gc7I2emEYHqwAfDiHqWV3x+ZyJh
-dZoyIYJlBx1w8/TsXWFJ78mDnVvE64nUJHPnlxsKQc4mxhmBcETjfg==
------END RSA PRIVATE KEY-----
diff --git a/.vagrant/machines/default/virtualbox/synced_folders b/.vagrant/machines/default/virtualbox/synced_folders
deleted file mode 100644
index ece4d77..0000000
--- a/.vagrant/machines/default/virtualbox/synced_folders
+++ /dev/null
@@ -1 +0,0 @@
-{"virtualbox":{"/vagrant":{"guestpath":"/vagrant","hostpath":"/Users/c4milo/Projects/node-inotify","disabled":false}}}
\ No newline at end of file
diff --git a/Vagrantfile b/Vagrantfile
index 3d8e157..35f2dfa 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -1,8 +1,10 @@
 $script = <<SCRIPT
-sudo apt-get update -y
-apt-get install -y build-essential
+sudo apt-get install -y build-essential
 curl -sL https://deb.nodesource.com/setup_0.12 | sudo bash -
+# curl -sL https://deb.nodesource.com/setup_0.10 | sudo bash -
+# curl -sL https://deb.nodesource.com/setup_iojs_1.x | sudo bash -
 sudo apt-get install -y nodejs
+sudo npm install -g node-gyp
 SCRIPT
 
 Vagrant.configure("2") do |config|
diff --git a/src/bindings.cc b/src/bindings.cc
index 98c6299..bbde8f6 100644
--- a/src/bindings.cc
+++ b/src/bindings.cc
@@ -179,7 +179,11 @@ namespace NodeInotify {
 		Local<Integer> descriptor = NanNew<Integer>(watch_descriptor);
 
 		//Local<Function> callback = Local<Function>::Cast(args_->Get(callback_sym));
+		#if NODE_VERSION_AT_LEAST(0, 11, 0)
 		inotify->handle()->Set(descriptor, args_->Get(callback_sym));
+		#else
+		inotify->handle_->Set(descriptor, args_->Get(callback_sym));
+		#endif
 
 		NanReturnValue(descriptor);
 	}
@@ -269,16 +273,23 @@ namespace NodeInotify {
 				argv[0] = obj;
 
 				inotify->Ref();
-				Local<Value> callback_ = inotify->handle()->Get(NanNew<Integer>(event->wd));
+
+				#if NODE_VERSION_AT_LEAST(0, 11, 0)
+				Local<Object> handle = inotify->handle();
+				#else
+				Persistent<Object> handle = inotify->handle_;
+				#endif
+
+				Local<Value> callback_ = handle->Get(NanNew<Integer>(event->wd));
 				Local<Function> callback = Local<Function>::Cast(callback_);
 
-				callback->Call(inotify->handle(), 1, argv);
+				callback->Call(handle, 1, argv);
 				inotify->Unref();
 
 				if(event->mask & IN_IGNORED) {
 					//deleting callback because the watch was removed
 					Local<Value> wd = NanNew<Integer>(event->wd);
-					inotify->handle()->Delete(wd->ToString());
+					handle->Delete(wd->ToString());
 				}
 
 				if (try_catch.HasCaught()) {
@@ -289,6 +300,7 @@ namespace NodeInotify {
 	}
 
 	NAN_GETTER(Inotify::GetPersistent) {
+		NanScope();
 		Inotify *inotify = ObjectWrap::Unwrap<Inotify>(args.This());
 
 		if (inotify->persistent) {
@@ -305,5 +317,4 @@ namespace NodeInotify {
 			poll_stopped = 1;
 		}
 	}
-
 }//namespace NodeInotify
diff --git a/src/bindings.h b/src/bindings.h
index 5d07209..f4e4b6c 100644
--- a/src/bindings.h
+++ b/src/bindings.h
@@ -5,29 +5,29 @@
 
 namespace NodeInotify {
 
-    class Inotify : public ObjectWrap {
-        public:
-            static void Initialize(Handle<Object> target);
+	class Inotify : public ObjectWrap {
+		public:
+			static void Initialize(Handle<Object> target);
 
-            Inotify();
-            Inotify(bool nonpersistent);
-            virtual ~Inotify();
-        protected:
-            static NAN_METHOD(New);
-            static NAN_METHOD(AddWatch);
-            static NAN_METHOD(RemoveWatch);
-            static NAN_METHOD(Close);
-            static NAN_GETTER(GetPersistent);
+			Inotify();
+			Inotify(bool nonpersistent);
+			virtual ~Inotify();
+		protected:
+			static NAN_METHOD(New);
+			static NAN_METHOD(AddWatch);
+			static NAN_METHOD(RemoveWatch);
+			static NAN_METHOD(Close);
+			static NAN_GETTER(GetPersistent);
 
-        private:
-            int fd;
-            uv_poll_t* read_watcher;
-            bool persistent;
-            char poll_stopped;
-            void StopPolling();
-            static void Callback(uv_poll_t* watcher, int status, int revents);
-            static void on_handle_close(uv_handle_t* handle);
-    };
+		private:
+			int fd;
+			uv_poll_t* read_watcher;
+			bool persistent;
+			char poll_stopped;
+			void StopPolling();
+			static void Callback(uv_poll_t* watcher, int status, int revents);
+			static void on_handle_close(uv_handle_t* handle);
+	};
 
 }  //namespace NodeInotify
 
diff --git a/src/node_inotify.h b/src/node_inotify.h
index ad27770..477d981 100644
--- a/src/node_inotify.h
+++ b/src/node_inotify.h
@@ -3,6 +3,7 @@
 #define SRC_NODE_INOTIFY_H_
 
 #include <node.h>
+#include <node_version.h>
 #include <node_object_wrap.h>
 #include <sys/inotify.h>
 #include <sys/select.h>

From 1b3efb25d1f54db35ec404e6c7d1cae7146b56b5 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 23:04:12 -0400
Subject: [PATCH 5/6] Leaves 0.12.x by default

---
 Vagrantfile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Vagrantfile b/Vagrantfile
index 35f2dfa..4ea07ef 100644
--- a/Vagrantfile
+++ b/Vagrantfile
@@ -3,7 +3,8 @@ sudo apt-get install -y build-essential
 curl -sL https://deb.nodesource.com/setup_0.12 | sudo bash -
 # curl -sL https://deb.nodesource.com/setup_0.10 | sudo bash -
 # curl -sL https://deb.nodesource.com/setup_iojs_1.x | sudo bash -
-sudo apt-get install -y nodejs
+ sudo apt-get install -y nodejs
+# sudo apt-get install -y iojs
 sudo npm install -g node-gyp
 SCRIPT
 

From 27052815fe7733efaf55d59188e111f81fcb2552 Mon Sep 17 00:00:00 2001
From: Camilo Aguilar <camilo.aguilar@gmail.com>
Date: Mon, 4 May 2015 23:14:50 -0400
Subject: [PATCH 6/6] Bumps year.

---
 LICENSE      | 2 +-
 package.json | 5 +++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/LICENSE b/LICENSE
index 8e65a7c..ab229c4 100644
--- a/LICENSE
+++ b/LICENSE
@@ -1,4 +1,4 @@
-# Copyright 2010 Camilo Aguilar. All rights reserved.
+# Copyright 2015 Camilo Aguilar. All rights reserved.
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to
 deal in the Software without restriction, including without limitation the
diff --git a/package.json b/package.json
index 7f2090a..6ade612 100644
--- a/package.json
+++ b/package.json
@@ -1,8 +1,8 @@
 {
   "name": "inotify",
   "version": "1.3.0",
-  "author": "Camilo Aguilar <camilo@cloudescape.com>",
-  "email": "camilo@cloudescape.com",
+  "author": "Camilo Aguilar <camilo.aguilar@gmail.com>",
+  "email": "camilo.aguilar@gmail.com",
   "keywords": [
     "inotify",
     "watch",
@@ -10,6 +10,7 @@
     "watch files",
     "watch directories"
   ],
+  "license": "MIT",
   "description": "inotify bindings for v8 javascript engine",
   "repository": {
     "type": "git",