Add unpipe method

[tinchoz49]

Hi @mafintosh, this is my first attempt to add the unpipe method. #11

[mafintosh]

Curious, what is your usecase for unpipe? I haven't found any, since is why I haven't added it, but can be convinced

[tinchoz49]

Yes, I never use it until today.

I'm working on a re-implementation of aws-lambda-fastify and I'm building a Readable Stream with streamx to simulate the http.IncomingMessage.

So the tests are not passing because of this: https://github.com/fastify/fastify-multipart/blob/master/index.js#L500 (first time I saw someone using unpipe)

[tinchoz49]

btw I know that I can just extend from http.IncomingMessage and use the node Readable stream but with streamx I get a performance improve of almost 20k of req/seq more. The stream state lifecycle in streamx is more simple and faster.

[tinchoz49]

I don't like the unpipe but maybe we can added only for compatibility support.

[mafintosh]

Okay nice! I can be convinced :)

[James4Ever0]

unpipe could be added in order to fix issues like this when working with webtorrent and fluent-ffmpeg. Otherwise I have to monkey patch this object.

[tforster]

I don't like the unpipe but maybe we can added only for compatibility support.

Is there a better/different implementation/option for unpipe? I am using it in one of my projects and it is preventing me from upgrading to vinyl-fs 4.

[phated]

Is there a better/different implementation/option for unpipe? I am using it in one of my projects and it is preventing me from upgrading to vinyl-fs 4.

Sounds like your code is just written incorrectly.

[tforster]

Is there a better/different implementation/option for unpipe? I am using it in one of my projects and it is preventing me from upgrading to vinyl-fs 4.

Sounds like your code is just written incorrectly.

How so? unpipe is a valid stream method.