Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove all uses of the 'self lifetime from the codebase and make it illegal to use self as a lifetime name #10889

Closed
bstrie opened this issue Dec 10, 2013 · 0 comments · Fixed by #10897
Closed

Remove all uses of the 'self lifetime from the codebase and make it illegal to use self as a lifetime name #10889

bstrie opened this issue Dec 10, 2013 · 0 comments · Fixed by #10897

Comments

@bstrie
Copy link
Contributor

bstrie commented Dec 10, 2013

The follow-up to #10675.
@erik erik mentioned this issue Dec 10, 2013
Merged
bors added a commit that referenced this issue Dec 11, 2013
@bors
auto merge of #10897 : boredomist/rust/remove-self-lifetime, r=brson
1b12dca 
Also remove all instances of 'self within the codebase.

This fixes #10889.

To make reviewing easier the following files were modified with more than a dumb text replacement:

- `src/test/compile-fail/lifetime-no-keyword.rs`
- `src/test/compile-fail/lifetime-obsoleted-self.rs`
- `src/test/compile-fail/regions-free-region-ordering-incorrect.rs`
- `src/libsyntax/parse/lexer.rs`
@bors bors closed this as completed in 5731ca3 Dec 11, 2013
cadencemarseille added a commit to cadencemarseille/rust-pcre that referenced this issue Dec 13, 2013
@cadencemarseille
Change 'self to 'a
86e7adb 
Update for rust-lang/rust#10889
flip1995 pushed a commit to flip1995/rust that referenced this issue Jun 30, 2023
@bors
Auto merge of rust-lang#10889 - blyxyas:link-config-desc, r=flip1995
5164458 
Add configuration values auto markdown links to `CHANGELOG.md`

Related to [this Zulip discussion](https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/Revisiting.20.20Clippy's.20configuration.20discover/near/363119950).

Now `cargo dev update_lints` also includes configurations as Markdown links, so changes to configuration can be linked directly.
It links to [this file](https://github.com/rust-lang/rust-clippy/blob/master/book/src/lint_configuration.md) (`book/src/lint_configuration.md`) because a new changelog may be deployed before the website is updated. So, in order to avoid broken links, this is the simplest way.

changelog:none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make 'self lifetime illegal. erik/rust
1 participant
@bstrie