Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(202)

Side by Side Diff: sdk/lib/async/stream_transformers.dart

Issue 2864443002: Revert "Throw when adding something to a closed sink and improve documentation." (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « sdk/lib/async/stream_controller.dart ('k') | sdk/lib/core/sink.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 part of dart.async; 5 part of dart.async;
6 6
7 /** 7 /**
8 * Wraps an [_EventSink] so it exposes only the [EventSink] interface. 8 * Wraps an [_EventSink] so it exposes only the [EventSink] interface.
9 */ 9 */
10 class _EventSinkWrapper<T> implements EventSink<T> { 10 class _EventSinkWrapper<T> implements EventSink<T> {
(...skipping 191 matching lines...) Expand 10 before | Expand all | Expand 10 after
202 * Wraps handlers (from [StreamTransformer.fromHandlers]) into an `EventSink`. 202 * Wraps handlers (from [StreamTransformer.fromHandlers]) into an `EventSink`.
203 * 203 *
204 * This way we can reuse the code from [_StreamSinkTransformer]. 204 * This way we can reuse the code from [_StreamSinkTransformer].
205 */ 205 */
206 class _HandlerEventSink<S, T> implements EventSink<S> { 206 class _HandlerEventSink<S, T> implements EventSink<S> {
207 final _TransformDataHandler<S, T> _handleData; 207 final _TransformDataHandler<S, T> _handleData;
208 final _TransformErrorHandler<T> _handleError; 208 final _TransformErrorHandler<T> _handleError;
209 final _TransformDoneHandler<T> _handleDone; 209 final _TransformDoneHandler<T> _handleDone;
210 210
211 /// The output sink where the handlers should send their data into. 211 /// The output sink where the handlers should send their data into.
212 EventSink<T> _sink; 212 final EventSink<T> _sink;
213 213
214 _HandlerEventSink( 214 _HandlerEventSink(
215 this._handleData, this._handleError, this._handleDone, this._sink) { 215 this._handleData, this._handleError, this._handleDone, this._sink);
216 if (_sink == null) {
217 throw new ArgumentError("The provided sink must not be null.");
218 }
219 }
220
221 bool get _isClosed => _sink == null;
222 216
223 void add(S data) { 217 void add(S data) {
224 if (_isClosed) throw new StateError("Sink is closed");
225 if (_handleData != null) { 218 if (_handleData != null) {
226 _handleData(data, _sink); 219 _handleData(data, _sink);
227 } else { 220 } else {
228 _sink.add(data as T); 221 _sink.add(data as T);
229 } 222 }
230 } 223 }
231 224
232 void addError(Object error, [StackTrace stackTrace]) { 225 void addError(Object error, [StackTrace stackTrace]) {
233 if (_isClosed) throw new StateError("Sink is closed");
234 if (_handleError != null) { 226 if (_handleError != null) {
235 _handleError(error, stackTrace, _sink); 227 _handleError(error, stackTrace, _sink);
236 } else { 228 } else {
237 _sink.addError(error, stackTrace); 229 _sink.addError(error, stackTrace);
238 } 230 }
239 } 231 }
240 232
241 void close() { 233 void close() {
242 if (_isClosed) return;
243 var sink = _sink;
244 _sink = null;
245 if (_handleDone != null) { 234 if (_handleDone != null) {
246 _handleDone(sink); 235 _handleDone(_sink);
247 } else { 236 } else {
248 sink.close(); 237 _sink.close();
249 } 238 }
250 } 239 }
251 } 240 }
252 241
253 /** 242 /**
254 * A StreamTransformer that transformers events with the given handlers. 243 * A StreamTransformer that transformers events with the given handlers.
255 * 244 *
256 * Note that this transformer can only be used once. 245 * Note that this transformer can only be used once.
257 */ 246 */
258 class _StreamHandlerTransformer<S, T> extends _StreamSinkTransformer<S, T> { 247 class _StreamHandlerTransformer<S, T> extends _StreamSinkTransformer<S, T> {
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 StreamSubscription<T> listen(void onData(T event), 300 StreamSubscription<T> listen(void onData(T event),
312 {Function onError, void onDone(), bool cancelOnError}) { 301 {Function onError, void onDone(), bool cancelOnError}) {
313 cancelOnError = identical(true, cancelOnError); 302 cancelOnError = identical(true, cancelOnError);
314 StreamSubscription<T> result = _transformer(_stream, cancelOnError); 303 StreamSubscription<T> result = _transformer(_stream, cancelOnError);
315 result.onData(onData); 304 result.onData(onData);
316 result.onError(onError); 305 result.onError(onError);
317 result.onDone(onDone); 306 result.onDone(onDone);
318 return result; 307 return result;
319 } 308 }
320 } 309 }
OLDNEW
« no previous file with comments | « sdk/lib/async/stream_controller.dart ('k') | sdk/lib/core/sink.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698