Warn when using a 'static
lifetime bound
Previously a `'static` lifetime bound would result in an `undeclared lifetime` error when compiling, even though it could be considered valid. However, it is unnecessary to use it as a lifetime bound so we present the user with a warning instead and suggest using the `'static` lifetime directly, in place of the lifetime parameter.
This commit is contained in:
parent
1ae1a19ba6
commit
e7949d0013
2 changed files with 28 additions and 2 deletions
|
@ -29,7 +29,7 @@ use syntax::ast;
|
|||
use syntax::attr;
|
||||
use syntax::ptr::P;
|
||||
use syntax::symbol::keywords;
|
||||
use syntax_pos::Span;
|
||||
use syntax_pos::{mk_sp, Span};
|
||||
use errors::DiagnosticBuilder;
|
||||
use util::nodemap::{NodeMap, NodeSet, FxHashSet, FxHashMap, DefIdMap};
|
||||
use rustc_back::slice;
|
||||
|
@ -1464,7 +1464,17 @@ impl<'a, 'tcx> LifetimeContext<'a, 'tcx> {
|
|||
self.check_lifetime_def_for_shadowing(old_scope, &lifetime_i.lifetime);
|
||||
|
||||
for bound in &lifetime_i.bounds {
|
||||
self.resolve_lifetime_ref(bound);
|
||||
if !bound.is_static() {
|
||||
self.resolve_lifetime_ref(bound);
|
||||
} else {
|
||||
self.insert_lifetime(bound, Region::Static);
|
||||
let full_span = mk_sp(lifetime_i.lifetime.span.lo, bound.span.hi);
|
||||
self.sess.struct_span_warn(full_span,
|
||||
&format!("unnecessary lifetime parameter `{}`", lifetime_i.lifetime.name))
|
||||
.help(&format!("you can use the `'static` lifetime directly, in place \
|
||||
of `{}`", lifetime_i.lifetime.name))
|
||||
.emit();
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
16
src/test/compile-fail/static-lifetime-bound.rs
Normal file
16
src/test/compile-fail/static-lifetime-bound.rs
Normal file
|
@ -0,0 +1,16 @@
|
|||
// Copyright 2017 The Rust Project Developers. See the COPYRIGHT
|
||||
// file at the top-level directory of this distribution and at
|
||||
// http://rust-lang.org/COPYRIGHT.
|
||||
//
|
||||
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
|
||||
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
|
||||
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
|
||||
// option. This file may not be copied, modified, or distributed
|
||||
// except according to those terms.
|
||||
|
||||
fn f<'a: 'static>(_: &'a i32) {} //~WARN unnecessary lifetime parameter `'a`
|
||||
|
||||
fn main() {
|
||||
let x = 0;
|
||||
f(&x); //~ERROR does not live long enough
|
||||
}
|
Loading…
Add table
Add a link
Reference in a new issue