Add pipe to input stream

Add pipe to input stream

Using the pipe method the data from an input stream can be piped
directly to an output stream.

R=whesse@google.com, ager@google.com

BUG=
TEST=

Committed: https://code.google.com/p/dart/source/detail?r=2564

[Mads Ager (google), 2011-12-14 13:49:11]

lgtm

LGTM with a couple of comments.

http://codereview.chromium.org/8933029/diff/7002/runtime/bin/file.dart
File runtime/bin/file.dart (right):

http://codereview.chromium.org/8933029/diff/7002/runtime/bin/file.dart#newcod...
runtime/bin/file.dart:224: * Synchronously get the length of the file.
Thank you!

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
File tests/standalone/src/StreamPipeTest.dart (right):

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:5: // Echo server test program to test
socket streams.
Update comment.

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:82:
getDataFilename("tests/standalone/src/data/dart-logo.png");
Would it be simpler to just use one of the other data files that we have already
instead of using a png?

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:104: if (count++ < 10) {
Do we really need to run this 10 times?

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:263: //testFileToFilePipe1();
Commented out code.

[Søren Gjesse, 2011-12-19 07:46:02]

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
File tests/standalone/src/StreamPipeTest.dart (right):

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:5: // Echo server test program to test
socket streams.
On 2011/12/14 13:49:11, Mads Ager wrote:
> Update comment.

Removed, other comments should cover purpose.

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:82:
getDataFilename("tests/standalone/src/data/dart-logo.png");
On 2011/12/14 13:49:11, Mads Ager wrote:
> Would it be simpler to just use one of the other data files that we have
already
> instead of using a png?

The only data files we have are quite small test files. I wanted a somewhat
larger binary file, and the dart-logo.png was just at hand.

Should we maybe have a number of generic test files e.g 1kdata, 4kdata, 16kdata?

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:104: if (count++ < 10) {
On 2011/12/14 13:49:11, Mads Ager wrote:
> Do we really need to run this 10 times?

Reduced to 2.

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:263: //testFileToFilePipe1();
On 2011/12/14 13:49:11, Mads Ager wrote:
> Commented out code.

Thanks. Re-activated.

[Søren Gjesse, 2011-12-19 14:22:31]

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
File tests/standalone/src/StreamPipeTest.dart (right):

http://codereview.chromium.org/8933029/diff/7002/tests/standalone/src/StreamP...
tests/standalone/src/StreamPipeTest.dart:82:
getDataFilename("tests/standalone/src/data/dart-logo.png");
On 2011/12/19 07:46:02, Søren Gjesse wrote:
> On 2011/12/14 13:49:11, Mads Ager wrote:
> > Would it be simpler to just use one of the other data files that we have
> already
> > instead of using a png?
> 
> The only data files we have are quite small test files. I wanted a somewhat
> larger binary file, and the dart-logo.png was just at hand.
> 
> Should we maybe have a number of generic test files e.g 1kdata, 4kdata,
16kdata?

Used the existing readline_test1.dat for now.