Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.
Stream select macro #2262
Merged
Stream select macro #2262
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
taiki-e
requested changes
Jun 16, 2021
Thanks for the PR and sorry for the late review!
I think it makes sense to add a feature like this (this is actually a stream version of simple select), but I don't want to add features that rely on Fused{Future,Stream} because Fused{Future,Stream} will be removed in the next major version.
taiki-e
added
the
S-waiting-on-author
|
@taiki-e I've removed the FusedStream dependency, this is now ready for review. |
taiki-e
removed
the
S-waiting-on-author
|
@taiki-e ready for re-review, thank you! |
taiki-e
reviewed
Jul 30, 2021
taiki-e
approved these changes
Jul 30, 2021
taiki-e
added
the
0.3-backport: pending
taiki-e
pushed a commit
that referenced
this issue
Aug 28, 2021
Merged
taiki-e
added
0.3-backport: completed
and removed
0.3-backport: pending
taiki-e
pushed a commit
that referenced
this issue
Aug 30, 2021
Merged
|
Published in 0.3.17 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Adds a new procedural macro to the futures crate that I've found some use for.
stream_select!works similarly to theselect_allfunction, but doesn't require the streams to all be the same type. Nor does it use aBox<dyn Stream>to accomplish that.It's implemented in terms of the, but represents a type rather than multiple branches of execution. In my use case I would feed the stream fromselect!macrostream_select!into aselect!statement, as many of the streams I want toselect!over should be treated identically in theselect!statement.Reviewers may ask "Why not just use the
selectfunction repeatedly?" And the answer to that is that recursive use of theselectfunction results in an unfair stream, with priority being given to streams on the upper layers.