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.

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

lint for catch used as identifier #48791

Closed
nikomatsakis opened this issue Mar 6, 2018 · 3 comments
Closed

lint for catch used as identifier #48791

nikomatsakis opened this issue Mar 6, 2018 · 3 comments
Labels
A-lints Area: Lints (warnings about flaws in source code) such as unused_mut. I-needs-decision Issue: In need of a decision. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue. WG-epoch Working group: Epoch (2018) management

Comments

@nikomatsakis
Copy link
Contributor

In the new epoch, we would like to reserve the catch keyword for future use. Therefore, we should add a lint that catch is going to be a reserved keyword in the future.

@nikomatsakis nikomatsakis added I-needs-decision Issue: In need of a decision. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Mar 6, 2018
@nikomatsakis nikomatsakis added T-lang Relevant to the language team, which will review and decide on the PR/issue. WG-epoch Working group: Epoch (2018) management labels Mar 6, 2018
@Centril Centril added the A-lints Area: Lints (warnings about flaws in source code) such as unused_mut. label Mar 7, 2018
@scottmcm
Copy link
Member

scottmcm commented Mar 7, 2018

Others that may or may not be part of this: https://internals.rust-lang.org/t/keywords-to-reserve-for-rust2018-epoch/6720

@nikomatsakis
Copy link
Contributor Author

Yep, I plan to go through and vet that list, opening issues where appropriate.

@Centril
Copy link
Contributor

Centril commented May 3, 2018

Closing this and tracking a possible lint for try at #50412

@Centril Centril closed this as completed May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-lints Area: Lints (warnings about flaws in source code) such as unused_mut. I-needs-decision Issue: In need of a decision. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-lang Relevant to the language team, which will review and decide on the PR/issue. WG-epoch Working group: Epoch (2018) management
Projects
None yet
Development

No branches or pull requests

3 participants