Rename remaining Failures in Documentation to Panic#18773
Rename remaining Failures in Documentation to Panic#18773bors merged 1 commit intorust-lang:masterfrom subhashb:convert_remaining_failures_to_panic
Conversation
|
|
src/liballoc/heap.rs
Outdated
There was a problem hiding this comment.
This is a good change, but should probably move to a Panics section like the others have.
There was a problem hiding this comment.
This isn't a panic. Otherwise it would be nonsense to speak of the return value.
|
@steveklabnik Comments addressed. I have also removed some unnecessary changes; better verify if system actually panics in those situations, before we change wording. |
|
Conflicts resolved and rebased. |
src/librand/lib.rs
Outdated
There was a problem hiding this comment.
This should move to the end of the doc-block so that "This is a convience ..." isn't included in this subsection.
|
@subhashb sorry, didn't see that this other nit got addressed. We've had a few PRs for this, can you rebase it one more time, then leave a comment? I'll r+ asap after you do. |
|
@steveklabnik Rebased. Good to pull now. |
…nic, r=steveklabnik I have also renamed `fail` to `panic` in some non-documentation comments, where I thought it mattered. Fixes #18677 cc @steveklabnik
Just saw this when looking at rust-lang#19297 and couldn't find an issue/PR dealing with this. rust-lang#18773 seems to have missed this file. Compiler output is generated [here](https://github.com/rust-lang/rust/blob/770378a313a573776b16237a46b75bafa49072c1/src/librustc_trans/driver/mod.rs#L466). cc @steveklabnik
6 participants
I have also renamed
failtopanicin some non-documentation comments, where I thought it mattered.Fixes #18677
cc @steveklabnik