Stack overflow if high water mark is too high

[Twipped]

Since the lib uses recursion to pull down and fill itself up, I'm running into problems with hitting the top of the call stack when using a high water mark over, say, 4000. I didn't investigate too deeply into what the limit was.

An option to have it invoke read via setImmediate every N iterations could solve this without much performance penalty.

[dominictarr]

another solution is to use:
https://github.com/dominictarr/looper
which can detect when the cb is sync, and then uses iteration in that case.
this avoids the overhead of setImmediate, or the problem of having to figure out what to set N at.

[Twipped]

I ended up solving it by adding this directly above PHW:

function breakStack (limit) {
    var count = 0;
    return function (read) {
        return function (abort, cb) {
            read(abort, function (err, data) {
                if(err) return cb(err)

                if (count % limit) {
                    cb(null, data);
                } else {
                    setImmediate(() => cb(null, data));
                }
            })
        }
    }
}

[dominictarr]

can you make a PR with a test case?

[Twipped]

I'll see what I can do. Now that this script is actually working I need to move on to other tasks, but I'll try to