- From: Takeshi Yoshino <notifications@github.com>
- Date: Tue, 07 Mar 2017 22:34:28 -0800
- To: whatwg/streams <streams@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Wednesday, 8 March 2017 06:35:02 UTC
tyoshino commented on this pull request.
>
// The promise that was returned from writer.close(). Stored here because it may be fulfilled after the writer
// has been detached.
- this._pendingCloseRequest = undefined;
+ this._closeRequest = undefined;
+
+ // Close request is removed from _closeRequest when close() is called on the underlying sink. This prevents it
+ // from being erroneously rejected on error. If a close() call is in-flight, the request is stored here.
+ this._inFlightCloseRequest = undefined;
> We could also consolidate the inFlight indication for close and write into one variable.
I guess I should try this after landing the patch.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/streams/pull/672#discussion_r104853431
Received on Wednesday, 8 March 2017 06:35:02 UTC