From 517367ef1969fb9d0f691acb2a644401926f8194 Mon Sep 17 00:00:00 2001 From: mscdex Date: Tue, 2 Jul 2013 23:30:52 -0400 Subject: [PATCH] Parser: check for unexpected readable-stream module behavior for node v0.8 --- lib/Parser.js | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/lib/Parser.js b/lib/Parser.js index 167b633..f726245 100644 --- a/lib/Parser.js +++ b/lib/Parser.js @@ -50,9 +50,17 @@ function Parser(stream, debug) { inherits(Parser, EventEmitter); Parser.prototype.setStream = function(stream) { - if (/^v0\.8\./.test(process.version)) - stream = new ReadableStream().wrap(stream); - + if (/^v0\.8\./.test(process.version)) { + var newStream = new ReadableStream(), + wrapped = newStream.wrap(stream); + // as of now, 07/02/2013, the readable-stream module is behind in updates + // and has different behavior for the return value of wrap() compared to + // node v0.10+, so we check for that here ... + if (wrapped) + stream = wrapped; + else + stream = newStream; + } this._stream = stream; };