Skip to content

Commit

Permalink
Stabilize impl_header_lifetime_elision in 2015
Browse files Browse the repository at this point in the history
It's already stable in 2018; this finishes the stabilization.
  • Loading branch information
scottmcm committed Oct 20, 2018
1 parent 89be71a commit 3739357
Show file tree
Hide file tree
Showing 27 changed files with 37 additions and 141 deletions.
2 changes: 1 addition & 1 deletion src/libcore/lib.rs
Expand Up @@ -87,7 +87,7 @@
#![feature(doc_spotlight)]
#![feature(extern_types)]
#![feature(fundamental)]
#![feature(impl_header_lifetime_elision)]
#![cfg_attr(stage0, feature(impl_header_lifetime_elision))]
#![feature(intrinsics)]
#![feature(lang_items)]
#![feature(link_llvm_intrinsics)]
Expand Down
2 changes: 1 addition & 1 deletion src/libcore/tests/lib.rs
Expand Up @@ -19,7 +19,7 @@
#![feature(flt2dec)]
#![feature(fmt_internals)]
#![feature(hashmap_internals)]
#![feature(impl_header_lifetime_elision)]
#![cfg_attr(stage0, feature(impl_header_lifetime_elision))]
#![feature(pattern)]
#![feature(range_is_empty)]
#![feature(raw)]
Expand Down
13 changes: 4 additions & 9 deletions src/librustc/hir/lowering.rs
Expand Up @@ -125,9 +125,8 @@ pub struct LoweringContext<'a> {

// Whether or not in-band lifetimes are being collected. This is used to
// indicate whether or not we're in a place where new lifetimes will result
// in in-band lifetime definitions, such a function or an impl header.
// This will always be false unless the `in_band_lifetimes` or
// `impl_header_lifetime_elision` feature is enabled.
// in in-band lifetime definitions, such a function or an impl header,
// including implicit lifetimes from `impl_header_lifetime_elision`.
is_collecting_in_band_lifetimes: bool,

// Currently in-scope lifetimes defined in impl headers, fn headers, or HRTB.
Expand Down Expand Up @@ -709,12 +708,8 @@ impl<'a> LoweringContext<'a> {
assert!(self.lifetimes_to_define.is_empty());
let old_anonymous_lifetime_mode = self.anonymous_lifetime_mode;

if self.sess.features_untracked().impl_header_lifetime_elision {
self.anonymous_lifetime_mode = anonymous_lifetime_mode;
self.is_collecting_in_band_lifetimes = true;
} else if self.sess.features_untracked().in_band_lifetimes {
self.is_collecting_in_band_lifetimes = true;
}
self.anonymous_lifetime_mode = anonymous_lifetime_mode;
self.is_collecting_in_band_lifetimes = true;

let (in_band_ty_params, res) = f(self);

Expand Down
2 changes: 1 addition & 1 deletion src/librustc/lib.rs
Expand Up @@ -67,7 +67,7 @@
#![feature(step_trait)]
#![feature(integer_atomics)]
#![feature(test)]
#![feature(impl_header_lifetime_elision)]
#![cfg_attr(stage0, feature(impl_header_lifetime_elision))]
#![feature(in_band_lifetimes)]
#![feature(macro_at_most_once_rep)]
#![feature(crate_visibility_modifier)]
Expand Down
2 changes: 1 addition & 1 deletion src/librustc_data_structures/lib.rs
Expand Up @@ -21,7 +21,7 @@
html_root_url = "https://doc.rust-lang.org/nightly/")]

#![feature(in_band_lifetimes)]
#![feature(impl_header_lifetime_elision)]
#![cfg_attr(stage0, feature(impl_header_lifetime_elision))]
#![feature(unboxed_closures)]
#![feature(fn_traits)]
#![feature(unsize)]
Expand Down
2 changes: 1 addition & 1 deletion src/librustc_mir/lib.rs
Expand Up @@ -16,7 +16,7 @@ Rust MIR: a lowered representation of Rust. Also: an experiment!

#![feature(nll)]
#![feature(in_band_lifetimes)]
#![feature(impl_header_lifetime_elision)]
#![cfg_attr(stage0, feature(impl_header_lifetime_elision))]
#![feature(slice_patterns)]
#![feature(slice_sort_by_cached_key)]
#![feature(box_patterns)]
Expand Down
7 changes: 3 additions & 4 deletions src/libsyntax/feature_gate.rs
Expand Up @@ -462,10 +462,6 @@ declare_features! (

(active, abi_amdgpu_kernel, "1.29.0", Some(51575), None),

// impl<I:Iterator> Iterator for &mut Iterator
// impl Debug for Foo<'_>
(active, impl_header_lifetime_elision, "1.30.0", Some(15872), Some(Edition::Edition2018)),

// Support for arbitrary delimited token streams in non-macro attributes
(active, unrestricted_attribute_tokens, "1.30.0", Some(44690), None),

Expand Down Expand Up @@ -684,6 +680,9 @@ declare_features! (
(accepted, min_const_fn, "1.31.0", Some(53555), None),
// Scoped lints
(accepted, tool_lints, "1.31.0", Some(44690), None),
// impl<I:Iterator> Iterator for &mut Iterator
// impl Debug for Foo<'_>
(accepted, impl_header_lifetime_elision, "1.31.0", Some(15872), None),
);

// If you change this, please modify src/doc/unstable-book as well. You must
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

3 changes: 0 additions & 3 deletions src/test/ui/feature-gates/feature-gate-in_band_lifetimes.rs
Expand Up @@ -10,9 +10,6 @@

#![allow(warnings)]

// Make sure this related feature didn't accidentally enable this
#![feature(impl_header_lifetime_elision)]

fn foo(x: &'x u8) -> &'x u8 { x }
//~^ ERROR use of undeclared lifetime name
//~^^ ERROR use of undeclared lifetime name
Expand Down
34 changes: 17 additions & 17 deletions src/test/ui/feature-gates/feature-gate-in_band_lifetimes.stderr
@@ -1,101 +1,101 @@
error[E0261]: use of undeclared lifetime name `'x`
--> $DIR/feature-gate-in_band_lifetimes.rs:16:12
--> $DIR/feature-gate-in_band_lifetimes.rs:13:12
|
LL | fn foo(x: &'x u8) -> &'x u8 { x }
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'x`
--> $DIR/feature-gate-in_band_lifetimes.rs:16:23
--> $DIR/feature-gate-in_band_lifetimes.rs:13:23
|
LL | fn foo(x: &'x u8) -> &'x u8 { x }
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:28:12
--> $DIR/feature-gate-in_band_lifetimes.rs:25:12
|
LL | impl<'a> X<'b> {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:30:27
--> $DIR/feature-gate-in_band_lifetimes.rs:27:27
|
LL | fn inner_2(&self) -> &'b u8 {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:36:8
--> $DIR/feature-gate-in_band_lifetimes.rs:33:8
|
LL | impl X<'b> {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:38:27
--> $DIR/feature-gate-in_band_lifetimes.rs:35:27
|
LL | fn inner_3(&self) -> &'b u8 {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'a`
--> $DIR/feature-gate-in_band_lifetimes.rs:46:9
--> $DIR/feature-gate-in_band_lifetimes.rs:43:9
|
LL | impl Y<&'a u8> {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'a`
--> $DIR/feature-gate-in_band_lifetimes.rs:48:25
--> $DIR/feature-gate-in_band_lifetimes.rs:45:25
|
LL | fn inner(&self) -> &'a u8 {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:56:27
--> $DIR/feature-gate-in_band_lifetimes.rs:53:27
|
LL | fn any_lifetime() -> &'b u8;
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:58:27
--> $DIR/feature-gate-in_band_lifetimes.rs:55:27
|
LL | fn borrowed_lifetime(&'b self) -> &'b u8;
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:58:40
--> $DIR/feature-gate-in_band_lifetimes.rs:55:40
|
LL | fn borrowed_lifetime(&'b self) -> &'b u8;
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'a`
--> $DIR/feature-gate-in_band_lifetimes.rs:63:14
--> $DIR/feature-gate-in_band_lifetimes.rs:60:14
|
LL | impl MyTrait<'a> for Y<&'a u8> {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'a`
--> $DIR/feature-gate-in_band_lifetimes.rs:63:25
--> $DIR/feature-gate-in_band_lifetimes.rs:60:25
|
LL | impl MyTrait<'a> for Y<&'a u8> {
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'a`
--> $DIR/feature-gate-in_band_lifetimes.rs:66:31
--> $DIR/feature-gate-in_band_lifetimes.rs:63:31
|
LL | fn my_lifetime(&self) -> &'a u8 { self.0 }
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:68:27
--> $DIR/feature-gate-in_band_lifetimes.rs:65:27
|
LL | fn any_lifetime() -> &'b u8 { &0 }
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:70:27
--> $DIR/feature-gate-in_band_lifetimes.rs:67:27
|
LL | fn borrowed_lifetime(&'b self) -> &'b u8 { &*self.0 }
| ^^ undeclared lifetime

error[E0261]: use of undeclared lifetime name `'b`
--> $DIR/feature-gate-in_band_lifetimes.rs:70:40
--> $DIR/feature-gate-in_band_lifetimes.rs:67:40
|
LL | fn borrowed_lifetime(&'b self) -> &'b u8 { &*self.0 }
| ^^ undeclared lifetime
Expand Down
2 changes: 0 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/assoc-type.rs
Expand Up @@ -13,8 +13,6 @@

#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

trait MyTrait {
type Output;
}
Expand Down
4 changes: 2 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/assoc-type.stderr
@@ -1,11 +1,11 @@
error[E0106]: missing lifetime specifier
--> $DIR/assoc-type.rs:23:19
--> $DIR/assoc-type.rs:21:19
|
LL | type Output = &i32;
| ^ expected lifetime parameter

error[E0106]: missing lifetime specifier
--> $DIR/assoc-type.rs:28:20
--> $DIR/assoc-type.rs:26:20
|
LL | type Output = &'_ i32;
| ^^ expected lifetime parameter
Expand Down
2 changes: 0 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/dyn-trait.rs
Expand Up @@ -13,8 +13,6 @@

#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

use std::fmt::Debug;

// Equivalent to `Box<dyn Debug + 'static>`:
Expand Down
6 changes: 3 additions & 3 deletions src/test/ui/impl-header-lifetime-elision/dyn-trait.stderr
@@ -1,11 +1,11 @@
error[E0495]: cannot infer an appropriate lifetime due to conflicting requirements
--> $DIR/dyn-trait.rs:32:16
--> $DIR/dyn-trait.rs:30:16
|
LL | static_val(x); //~ ERROR cannot infer
| ^
|
note: first, the lifetime cannot outlive the lifetime 'a as defined on the function body at 31:26...
--> $DIR/dyn-trait.rs:31:26
note: first, the lifetime cannot outlive the lifetime 'a as defined on the function body at 29:26...
--> $DIR/dyn-trait.rs:29:26
|
LL | fn with_dyn_debug_static<'a>(x: Box<dyn Debug + 'a>) {
| ^^
Expand Down
Expand Up @@ -12,8 +12,6 @@

#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

// This works for functions...
fn foo<'a>(x: &str, y: &'a str) {}

Expand Down
2 changes: 0 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/path-elided.rs
Expand Up @@ -9,8 +9,6 @@
// except according to those terms.
#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

trait MyTrait { }

struct Foo<'a> { x: &'a u32 }
Expand Down
@@ -1,5 +1,5 @@
error[E0106]: missing lifetime specifier
--> $DIR/path-elided.rs:18:18
--> $DIR/path-elided.rs:16:18
|
LL | impl MyTrait for Foo {
| ^^^ expected lifetime parameter
Expand Down
2 changes: 0 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/path-underscore.rs
Expand Up @@ -14,8 +14,6 @@

#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

trait MyTrait { }

struct Foo<'a> { x: &'a u32 }
Expand Down
2 changes: 0 additions & 2 deletions src/test/ui/impl-header-lifetime-elision/ref-underscore.rs
Expand Up @@ -14,8 +14,6 @@

#![allow(warnings)]

#![feature(impl_header_lifetime_elision)]

trait MyTrait { }

impl MyTrait for &i32 {
Expand Down

0 comments on commit 3739357

Please sign in to comment.