Skip to content

Conversation

RalfJung
Copy link
Member

The public function is called catch_unwind, the intrinsic at some point got renamed to catch_unwind -- there's no reason to have the internal implementation of this still be called try, so let's rename it to match the rest.

@rustbot
Copy link
Collaborator

rustbot commented May 24, 2025

r? @tgross35

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

@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 May 24, 2025
@rustbot
Copy link
Collaborator

rustbot commented May 24, 2025

The Miri subtree was changed

cc @rust-lang/miri

@Noratrieb
Copy link
Member

r=me

@RalfJung
Copy link
Member Author

@bors r=Noratrieb

@bors
Copy link
Collaborator

bors commented May 24, 2025

📌 Commit 1827bc0 has been approved by Noratrieb

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-review Status: Awaiting review from the assignee but also interested parties. labels May 24, 2025
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request May 24, 2025
rename internal panicking::try to catch_unwind

The public function is called `catch_unwind`, the intrinsic at some point got renamed to `catch_unwind` -- there's no reason to have the internal implementation of this still be called `try`, so let's rename it to match the rest.
jhpratt added a commit to jhpratt/rust that referenced this pull request May 24, 2025
rename internal panicking::try to catch_unwind

The public function is called `catch_unwind`, the intrinsic at some point got renamed to `catch_unwind` -- there's no reason to have the internal implementation of this still be called `try`, so let's rename it to match the rest.
bors added a commit that referenced this pull request May 24, 2025
Rollup of 6 pull requests

Successful merges:

 - #140066 (Stabilize `<[T; N]>::as_mut_slice` as `const`)
 - #141105 (additional edge cases tests for `path.rs` 🧪 )
 - #141487 (Update askama to `0.14.0`)
 - #141498 (Use C-string literals to reduce boilerplate)
 - #141505 (rename internal panicking::try to catch_unwind)
 - #141511 (Cleanup CodegenFnAttrFlags)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 76ca0e2 into rust-lang:master May 25, 2025
6 checks passed
@rustbot rustbot added this to the 1.89.0 milestone May 25, 2025
rust-timer added a commit that referenced this pull request May 25, 2025
Rollup merge of #141505 - RalfJung:catch_unwind, r=Noratrieb

rename internal panicking::try to catch_unwind

The public function is called `catch_unwind`, the intrinsic at some point got renamed to `catch_unwind` -- there's no reason to have the internal implementation of this still be called `try`, so let's rename it to match the rest.
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request May 25, 2025
Rollup of 6 pull requests

Successful merges:

 - rust-lang/rust#140066 (Stabilize `<[T; N]>::as_mut_slice` as `const`)
 - rust-lang/rust#141105 (additional edge cases tests for `path.rs` 🧪 )
 - rust-lang/rust#141487 (Update askama to `0.14.0`)
 - rust-lang/rust#141498 (Use C-string literals to reduce boilerplate)
 - rust-lang/rust#141505 (rename internal panicking::try to catch_unwind)
 - rust-lang/rust#141511 (Cleanup CodegenFnAttrFlags)

r? `@ghost`
`@rustbot` modify labels: rollup
@RalfJung RalfJung deleted the catch_unwind branch May 25, 2025 07:23
github-actions bot pushed a commit to model-checking/verify-rust-std that referenced this pull request May 26, 2025
rename internal panicking::try to catch_unwind

The public function is called `catch_unwind`, the intrinsic at some point got renamed to `catch_unwind` -- there's no reason to have the internal implementation of this still be called `try`, so let's rename it to match the rest.
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