From 2498e2934495a58794bd0aa498b6297b234655f3 Mon Sep 17 00:00:00 2001 From: Roman Reiss Date: Mon, 26 Oct 2015 13:30:54 +0100 Subject: [PATCH] src: Revert "nix stdin _readableState.reading" This reverts 8cee8f5 which was causing stdin to behave strangely on Windows 8 and 10. The suspected explanation for the issue is that there might be a race condition occuring when stdin._readableState.reading is set indirectly through `push('')`. PR-URL: https://github.com/nodejs/node/pull/3490 Fixes: https://github.com/nodejs/node/issues/2996 Fixes: https://github.com/nodejs/node/issues/2504 Reviewed-By: Trevor Norris Reviewed-By: Sakthipriyan Vairamani Reviewed-By: Rod Vagg --- src/node.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/node.js b/src/node.js index ed74480289a9da..57a803cfc44312 100644 --- a/src/node.js +++ b/src/node.js @@ -726,7 +726,7 @@ // not-reading state. if (stdin._handle && stdin._handle.readStop) { stdin._handle.reading = false; - stdin.push(''); + stdin._readableState.reading = false; stdin._handle.readStop(); } @@ -735,7 +735,7 @@ stdin.on('pause', function() { if (!stdin._handle) return; - stdin.push(''); + stdin._readableState.reading = false; stdin._handle.reading = false; stdin._handle.readStop(); });