Skip to content

Conversation

@nxsaken
Copy link
Contributor

@nxsaken nxsaken commented Oct 30, 2025

Feature: const_control_flow
Tracking issue: #148739

This PR constifies the methods on ControlFlow with a dependency on #143874.

@rustbot rustbot added 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. labels Oct 30, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 30, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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

#[rustc_const_unstable(feature = "const_control_flow", issue = "none")]
pub const fn map_break<T, F>(self, f: F) -> ControlFlow<T, C>
where
F: [const] FnOnce(B) -> T + [const] Destruct,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might it be good to have something like [const] (A+B+C)?

@Mark-Simulacrum
Copy link
Member

r=me with a tracking issue, I think.

@Mark-Simulacrum Mark-Simulacrum added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 8, 2025
@nxsaken nxsaken force-pushed the const_control_flow_1 branch from 6d56cae to 3175799 Compare November 9, 2025 09:34
@rustbot
Copy link
Collaborator

rustbot commented Nov 9, 2025

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.

@nxsaken
Copy link
Contributor Author

nxsaken commented Nov 9, 2025

@Mark-Simulacrum ping

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 9, 2025

📌 Commit 3175799 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Nov 9, 2025
bors added a commit that referenced this pull request Nov 9, 2025
Rollup of 4 pull requests

Successful merges:

 - #148248 (Constify `ControlFlow` methods without unstable bounds)
 - #148285 (Constify `ControlFlow` methods with unstable bounds)
 - #148510 (compiletest: Do the known-directives check only once, and improve its error message)
 - #148655 (Fix invalid macro tag generation for keywords which can be followed by values)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 5591d7e into rust-lang:master Nov 10, 2025
11 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Nov 10, 2025
rust-timer added a commit that referenced this pull request Nov 10, 2025
Rollup merge of #148285 - nxsaken:const_control_flow_1, r=Mark-Simulacrum

Constify `ControlFlow` methods with unstable bounds

Feature: `const_control_flow`
Tracking issue: #148739

This PR constifies the methods on `ControlFlow` with a dependency on #143874.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

5 participants