csharpstandard icon indicating copy to clipboard operation
csharpstandard copied to clipboard
verified publisher icon dotnet

Add support for Async Streams

Open RexJaeschke opened this issue 3 years ago • 4 comments

There is some overlap between this PR and PR #672 in 13.14, "The using statement" that will require reconciliation when the second of these two PRs is merged. (PR 606 was spec'd first.)

RexJaeschke avatar Aug 08 '22 11:08 RexJaeschke

A comparison that doesn't include so many unrelated changes: https://github.com/dotnet/csharpstandard/compare/draft-v7...RexJaeschke:csharpstandard:async-streams

KalleOlaviNiemitalo avatar Nov 21 '22 08:11 KalleOlaviNiemitalo

rebased on the latest draft v8 branch on 09/23/2023

BillWagner avatar Sep 26 '23 16:09 BillWagner

Summary of discussions:

  • Try to reword to avoid the brackets
  • Avoid using "stream" in normative text, but have a note
  • Merge the "Asynchronous stream creation and usage" into the iterator section
  • Reorder async and iterators so that async comes first
  • Defer the CancellationToken support until after we've got the structure right (but as a fast-follow - definitely in C# 8)

jskeet avatar May 15 '24 21:05 jskeet