fix(packages/bindings#write): do not call native binding for empty buffers

Author: HipsterBrown

packages/bindings/lib/darwin.js

@@ -59,13 +59,16 @@ class DarwinBinding extends AbstractBinding {
   }
 
   write(buffer) {
-    this.writeOperation = super
-      .write(buffer)
-      .then(() => unixWrite.call(this, buffer))
-      .then(() => {
-        this.writeOperation = null
-      })
-    return this.writeOperation
+    if (buffer.length > 0) {
+      this.writeOperation = super
+        .write(buffer)
+        .then(() => unixWrite.call(this, buffer))
+        .then(() => {
+          this.writeOperation = null
+        })
+      return this.writeOperation
+    }
+    return Promise.resolve()
   }
 
   update(options) {

packages/bindings/lib/linux.js

@@ -59,13 +59,16 @@ class LinuxBinding extends AbstractBinding {
   }
 
   write(buffer) {
-    this.writeOperation = super
-      .write(buffer)
-      .then(() => unixWrite.call(this, buffer))
-      .then(() => {
-        this.writeOperation = null
-      })
-    return this.writeOperation
+    if (buffer.length > 0) {
+      this.writeOperation = super
+        .write(buffer)
+        .then(() => unixWrite.call(this, buffer))
+        .then(() => {
+          this.writeOperation = null
+        })
+      return this.writeOperation
+    }
+    return Promise.resolve()
   }
 
   update(options) {

packages/bindings/lib/win32.js

@@ -66,13 +66,16 @@ class WindowsBinding extends AbstractBinding {
   }
 
   write(buffer) {
-    this.writeOperation = super
-      .write(buffer)
-      .then(() => promisify(binding.write)(this.fd, buffer))
-      .then(() => {
-        this.writeOperation = null
-      })
-    return this.writeOperation
+    if (buffer.length > 0) {
+      this.writeOperation = super
+        .write(buffer)
+        .then(() => promisify(binding.write)(this.fd, buffer))
+        .then(() => {
+          this.writeOperation = null
+        })
+      return this.writeOperation
+    }
+    return Promise.resolve()
   }
 
   update(options) {