Wrap stdout/stderr/stdin to only expose the relevant methods.

Wrap stdout/stderr/stdin to only expose the relevant methods.


BUG=https://code.google.com/p/dart/issues/detail?id=9439,https://code.google.com/p/dart/issues/detail?id=9593

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

[Søren Gjesse, 2013-04-08 12:10:32]

lgtm

[Anders Johnsen, 2013-04-08 12:13:14]

Committed patchset #1 manually as r21063 (presubmit successful).

[ahe, 2013-04-08 12:25:25]

DBC

https://codereview.chromium.org/13787003/diff/1/sdk/lib/io/stdio.dart
File sdk/lib/io/stdio.dart (right):

https://codereview.chromium.org/13787003/diff/1/sdk/lib/io/stdio.dart#newcode44
sdk/lib/io/stdio.dart:44: Future<T> consume(Stream<List<int>> stream) =>
_ioSink.consume(stream);
The use of T without having T declared causes a warning from apidoc, and breaks
a dart2js test which attempts to ensure our API is free of warnings.

[Anders Johnsen, 2013-06-11 12:15:04]

https://codereview.chromium.org/13787003/diff/1/sdk/lib/io/stdio.dart
File sdk/lib/io/stdio.dart (right):

https://codereview.chromium.org/13787003/diff/1/sdk/lib/io/stdio.dart#newcode44
sdk/lib/io/stdio.dart:44: Future<T> consume(Stream<List<int>> stream) =>
_ioSink.consume(stream);
On 2013/04/08 12:25:25, ahe wrote:
> The use of T without having T declared causes a warning from apidoc, and
breaks
> a dart2js test which attempts to ensure our API is free of warnings.

Yeah, bad testing from my side. Sorry

https://codereview.chromium.org/13647019