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

Lifetime defaults #5125

Merged
merged 8 commits into from
Mar 6, 2013
Merged

Conversation

nikomatsakis
Copy link
Contributor

Work towards #4846.

  • Institute new region defaults where all omitted regions get a fresh lifetime.
  • Require explicit region names except in functions.
  • Fix a bug in region parameterization inference. I've been putting this off because it will not be important when we remove RP inference in favor of explicit declarations, but then it was blocking this patch.

r? @pcwalton

bors added a commit that referenced this pull request Mar 6, 2013
… r=nikomatsakis

Work towards #4846.

- Institute new region defaults where all omitted regions get a fresh lifetime.
- Require explicit region names except in functions.
- Fix a bug in region parameterization inference.  I've been putting this off because it will not be important when we remove RP inference in favor of explicit declarations, but then it was blocking this patch.

r? @pcwalton
@bors bors closed this Mar 6, 2013
@bors bors merged commit 078fd23 into rust-lang:incoming Mar 6, 2013
@nikomatsakis nikomatsakis deleted the issue-4846-lifetime-defaults branch March 30, 2016 16:15
bors added a commit to rust-lang-ci/rust that referenced this pull request May 2, 2020
Port mitsuhiko's excessive bools lints

Closes #4 .
changelog: add `struct_excessive_bools` and `fn_params_excessive_bools` lints.
I moved is_trait_impl_item check because at first I implemented it as a late pass for some reason but then I realized it's actually an early lint. But it's a useful function to have, should I move it into a separate pr?
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.

2 participants