Skip to content

refactor: TryFutures are Futures now, into_future is no longer need#2878

Open
yhx-12243 wants to merge 11 commits into
rust-lang:mainfrom
yhx-12243:remove-into-future
Open

refactor: TryFutures are Futures now, into_future is no longer need#2878
yhx-12243 wants to merge 11 commits into
rust-lang:mainfrom
yhx-12243:remove-into-future

Conversation

@yhx-12243

Copy link
Copy Markdown

Reduce a wrapper for IntoFuture struct, reduce code complexity.

@yhx-12243

Copy link
Copy Markdown
Author

AFAIK there are almost no crates use these IntoFutures and IntoStreams. But if one want the backward compatibility, one can define type IntoFuture<T> = T and type IntoStream<T> = T or just bump to v0.4.0, both is OK.

@taiki-e

taiki-e commented Oct 2, 2024

Copy link
Copy Markdown
Member

This appears to be a duplicate of #2763?

@yhx-12243

Copy link
Copy Markdown
Author

This appears to be a duplicate of #2763?

Maybe to say it is an extension better.

@rustbot

This comment has been minimized.

@rustbot rustbot added A-future Area: futures::future A-stream Area: futures::stream labels Jul 18, 2026
@rustbot

This comment has been minimized.

@rustbot rustbot added A-compat Area: futures::compat A-sink Area: futures::sink labels Jul 18, 2026
@rustbot

rustbot commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-compat Area: futures::compat A-future Area: futures::future A-sink Area: futures::sink A-stream Area: futures::stream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants