stream: pipe should not call cleanup on handled error

[floatdrop]

If user is able to handle error (or attached on('error', ...) handler on stream - then we should not call cleanup on piped streams.

I think this two lines should be in reverse order.

We encountered this problem inside this tasks: gulpjs/gulp#91 and gulpjs/gulp#75

[juliangruber]

The semantics of a stream's error event are that it went berzerk and can't continue operating. And that is good for the common case. What you might want to do is emit warning events, or any other event without predefined semantics, and then implement your own handling.

[floatdrop]

Thank you, pretty clear explanation.