Fix #2257 pause/resume semantics for stdin

This makes it so that the stdin TTY-wrap stream gets ref'ed on
.resume() and unref'ed on .pause()

The semantics of the names "pause" and "resume" are a bit weird, but the
important thing is that this corrects an API change from 0.4 -> 0.6
which made it impossible to read from stdin multiple times, without
knowing when it might end up being closed.  If no one has it open, this
lets the process die naturally.

LGTM'd by @ry

1 files changed, 11 insertions, 0 deletions

diff --git a/lib/tty.js b/lib/tty.js
index 78971dda31..32ba512d52 100644
--- a/lib/tty.js
+++ b/lib/tty.js
@@ -83,8 +83,19 @@ function ReadStream(fd) {
   this.on('newListener', onNewListener);
 }
 inherits(ReadStream, net.Socket);
+
 exports.ReadStream = ReadStream;
 
+ReadStream.prototype.pause = function() {
+  this._handle.unref();
+  return net.Socket.prototype.pause.call(this);
+};
+
+ReadStream.prototype.resume = function() {
+  this._handle.ref();
+  return net.Socket.prototype.resume.call(this);
+};
+
 
 ReadStream.prototype.isTTY = true;