Auto merge of #28486 - nrc:pub-extern-crate, r=alexcrichton
Temporary 'fix' for #26775 r? @brson
This commit is contained in:
commit
d97be7bd70
2 changed files with 30 additions and 5 deletions
|
@ -5101,6 +5101,13 @@ impl<'a> Parser<'a> {
|
||||||
try!(self.expect(&token::Semi));
|
try!(self.expect(&token::Semi));
|
||||||
|
|
||||||
let last_span = self.last_span;
|
let last_span = self.last_span;
|
||||||
|
|
||||||
|
if visibility == ast::Public {
|
||||||
|
self.span_warn(mk_sp(lo, last_span.hi),
|
||||||
|
"`pub extern crate` does not work as expected and should not be used. \
|
||||||
|
Likely to become an error. Prefer `extern crate` and `pub use`.");
|
||||||
|
}
|
||||||
|
|
||||||
Ok(self.mk_item(lo,
|
Ok(self.mk_item(lo,
|
||||||
last_span.hi,
|
last_span.hi,
|
||||||
ident,
|
ident,
|
||||||
|
|
18
src/test/compile-fail/warn-pub-extern-crate.rs
Normal file
18
src/test/compile-fail/warn-pub-extern-crate.rs
Normal file
|
@ -0,0 +1,18 @@
|
||||||
|
// Copyright 2015 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.
|
||||||
|
|
||||||
|
// Check that `pub extern crate` gives a warning.
|
||||||
|
|
||||||
|
|
||||||
|
pub extern crate core; //~WARN `pub extern crate` does not work
|
||||||
|
//~^ ERROR core
|
||||||
|
|
||||||
|
fn main() {
|
||||||
|
}
|
Loading…
Add table
Add a link
Reference in a new issue