Skip to content

Rename begin_unwind lang item and core function #17428

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Sep 25, 2014

Conversation

fhahn
Copy link
Contributor

@fhahn fhahn commented Sep 21, 2014

This is a PR for #16114 and includes to following things:

  • Rename begin_unwind lang item to fail_fmt
  • Rename core::failure::begin_unwind to fail_impl
  • Rename fail_ lang item to fail

@rust-highfive
Copy link
Contributor

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!

#[cfg(not(stage0))]
#[cold] #[inline(never)] // this is the slow path, always
#[lang="fail"]
fn fail_(expr_file_line: &(&'static str, &'static str, uint)) -> ! {
Copy link
Member

Choose a reason for hiding this comment

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

Could the name of this function just be fail?

@alexcrichton
Copy link
Member

Thanks for this!

@fhahn fhahn force-pushed the issue-16114-rename-begin-unwind-2 branch from af05c95 to 20ccb2b Compare September 22, 2014 21:25
@fhahn
Copy link
Contributor Author

fhahn commented Sep 22, 2014

I've applied all changes. I've added a new commit for the renaming, because rebasing and squashing resulted in conflicts. If that's a problem I can try harder and produce a patch with fewer commits ;)

@fhahn fhahn force-pushed the issue-16114-rename-begin-unwind-2 branch from 20ccb2b to 88cc1e6 Compare September 22, 2014 21:42
@fhahn fhahn force-pushed the issue-16114-rename-begin-unwind-2 branch from 88cc1e6 to 3729004 Compare September 23, 2014 08:20
@fhahn fhahn force-pushed the issue-16114-rename-begin-unwind-2 branch from 3729004 to d06f1aa Compare September 24, 2014 22:23
@fhahn fhahn force-pushed the issue-16114-rename-begin-unwind-2 branch from d06f1aa to c8b767d Compare September 24, 2014 23:09
bors added a commit that referenced this pull request Sep 25, 2014
…r=alexcrichton

This is a PR for #16114 and includes to following things:

* Rename `begin_unwind` lang item to `fail_fmt`
*  Rename `core::failure::begin_unwind` to `fail_impl`
* Rename `fail_` lang item to `fail`
@bors bors closed this Sep 25, 2014
@bors bors merged commit c8b767d into rust-lang:master Sep 25, 2014
@fhahn fhahn deleted the issue-16114-rename-begin-unwind-2 branch September 25, 2014 08:22
lnicola pushed a commit to lnicola/rust that referenced this pull request Jun 23, 2024
feat: add space after specific keywords in completion

fix rust-lang#17428.

When completing some specific keywords, it would be convenient if r-a could automatically add a space afterwards.

This PR implements this feature for the following keywords:

- Visibility: `pub`, `pub(crate)`, `pub(super)`, `pub(in xxx)`
- Pattern: `ref` / `mut`
- Others: `unsafe` / `for` / `where`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants