diff options
| author | Brendan Ashworth <brendan.ashworth@me.com> | 2015-05-28 06:42:33 +0300 |
|---|---|---|
| committer | Brendan Ashworth <brendan.ashworth@me.com> | 2015-05-30 10:42:52 +0300 |
| commit | a65762cab6edf33d87a129e406f633dfa445bb23 (patch) | |
| tree | e20abda72ba91ff131eaef251e40cc68f26fa4a7 | |
| parent | 8606793999979829df002c2ab69235a05605fc4d (diff) | |
src: remove old code
The Socket writable only change was added and implemented in the
constructor around 5885f464f0ad372efa7e, but this was never removed.
The libev counter issue is no longer prudent; the test remains in
test/sequential/test-regress-GH-1726.
PR-URL: https://github.com/nodejs/io.js/pull/1819
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
| -rw-r--r-- | src/node.js | 19 |
1 files changed, 0 insertions, 19 deletions
diff --git a/src/node.js b/src/node.js index 065c337e743..4521afbf0cb 100644 --- a/src/node.js +++ b/src/node.js @@ -578,12 +578,6 @@ var tty = NativeModule.require('tty'); stream = new tty.WriteStream(fd); stream._type = 'tty'; - - // Hack to have stream not keep the event loop alive. - // See https://github.com/joyent/node/issues/1726 - if (stream._handle && stream._handle.unref) { - stream._handle.unref(); - } break; case 'FILE': @@ -600,20 +594,7 @@ readable: false, writable: true }); - - // FIXME Should probably have an option in net.Socket to create a - // stream from an existing fd which is writable only. But for now - // we'll just add this hack and set the `readable` member to false. - // Test: ./node test/fixtures/echo.js < /etc/passwd - stream.readable = false; - stream.read = null; stream._type = 'pipe'; - - // FIXME Hack to have stream not keep the event loop alive. - // See https://github.com/joyent/node/issues/1726 - if (stream._handle && stream._handle.unref) { - stream._handle.unref(); - } break; default: |