ReadStream#resume() throws TypeError: Bad argument if called immediately

[slaskis]

Here's a small reproducible example (in 0.4.0):

var tmp = require("fs").createReadStream("/dev/random");
tmp.pause() // not required, it throws even without pausing
tmp.resume()

With a process.nextTick it works:

var tmp = require("fs").createReadStream("/dev/random");
tmp.pause()
process.nextTick(function(){ tmp.resume() })

[isaacs]

Fix https://gist.github.com/833106

[slaskis]

Actually after some more testing it seems that a process.nextTick() is not always enough. However this seems to help:

tmp.on("open",function(){process.nextTick(function(){tmp.resume()})})

[isaacs]

The important thing is that the resume() will throw if it comes before the "open" event. Attached patch fixes it.

[isaacs]

Closed by 8b9dbda Don't read fs read stream if not open

[slaskis]

Great, thanks!