Skip to content

std: Rename thread::catch_panic to panic::recover - #28128

Closed
alexcrichton wants to merge 1 commit into
rust-lang:masterfrom
alexcrichton:panic-recover
Closed

alexcrichton wants to merge 1 commit into
rust-lang:masterfrom
alexcrichton:panic-recover

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

This commit is an implementation of RFC 1236 which renames the
thread::catch_panic function to panic::recover while also removing the
Send bound (but leaving the 'static bound).

cc #27719

This commit is an implementation of [RFC 1236][rfc] which renames the
`thread::catch_panic` function to `panic::recover` while also removing the
`Send` bound (but leaving the `'static` bound).

[rfc]: rust-lang/rfcs#1236

cc rust-lang#27719
@alexcrichton

Copy link
Copy Markdown
Member Author

r? @aturon

@rust-highfive rust-highfive assigned aturon and unassigned nikomatsakis Aug 31, 2015
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@bors

bors commented Sep 2, 2015

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #28148) made this pull request unmergeable. Please resolve the merge conflicts.

@alexcrichton alexcrichton added the T-libs-api [DEPRECATED; DO NOT USE] label Sep 2, 2015
@alexcrichton

Copy link
Copy Markdown
Member Author

More discussion is happening on the RFC, so closing until that's settled.

@alexcrichton
alexcrichton deleted the panic-recover branch September 14, 2015 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-libs-api [DEPRECATED; DO NOT USE]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants