Skip to content

Don't fuse in MapWindows#154190

Open
Jules-Bertholet wants to merge 1 commit intorust-lang:mainfrom
Jules-Bertholet:nonfused
Open

Don't fuse in MapWindows#154190
Jules-Bertholet wants to merge 1 commit intorust-lang:mainfrom
Jules-Bertholet:nonfused

Conversation

@Jules-Bertholet
Copy link
Contributor

@Jules-Bertholet Jules-Bertholet commented Mar 21, 2026

cc #87155

Fusing makes the iterator larger, slower, more complicated, and less useful. Users who need fusing behavior can always use .fuse(), but there is no way to get non-fusing behavior from the fused version.

@rustbot label A-iterators

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 21, 2026
@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Mar 21, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 21, 2026

r? @scottmcm

rustbot has assigned @scottmcm.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @scottmcm, libs
  • @scottmcm, libs expanded to 8 candidates
  • Random selection from Mark-Simulacrum, scottmcm

@rustbot rustbot added the A-iterators Area: Iterators label Mar 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-iterators Area: Iterators S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants