Auto merge of #56303 - petrochenkov:stabuseas, r=cramertj
Stabilize `underscore_imports` Closes https://github.com/rust-lang/rust/issues/48216
This commit is contained in:
commit
c9bb68f5ae
12 changed files with 11 additions and 69 deletions
|
@ -37,7 +37,7 @@ Rust MIR: a lowered representation of Rust. Also: an experiment!
|
||||||
#![feature(if_while_or_patterns)]
|
#![feature(if_while_or_patterns)]
|
||||||
#![feature(try_from)]
|
#![feature(try_from)]
|
||||||
#![feature(reverse_bits)]
|
#![feature(reverse_bits)]
|
||||||
#![feature(underscore_imports)]
|
#![cfg_attr(stage0, feature(underscore_imports))]
|
||||||
|
|
||||||
#![recursion_limit="256"]
|
#![recursion_limit="256"]
|
||||||
|
|
||||||
|
|
|
@ -400,9 +400,6 @@ declare_features! (
|
||||||
// Allows `#[repr(packed)]` attribute on structs.
|
// Allows `#[repr(packed)]` attribute on structs.
|
||||||
(active, repr_packed, "1.26.0", Some(33158), None),
|
(active, repr_packed, "1.26.0", Some(33158), None),
|
||||||
|
|
||||||
// Allows `use path as _;` and `extern crate c as _;`.
|
|
||||||
(active, underscore_imports, "1.26.0", Some(48216), None),
|
|
||||||
|
|
||||||
// Allows macro invocations in `extern {}` blocks.
|
// Allows macro invocations in `extern {}` blocks.
|
||||||
(active, macros_in_extern, "1.27.0", Some(49476), None),
|
(active, macros_in_extern, "1.27.0", Some(49476), None),
|
||||||
|
|
||||||
|
@ -694,6 +691,8 @@ declare_features! (
|
||||||
(accepted, self_struct_ctor, "1.32.0", Some(51994), None),
|
(accepted, self_struct_ctor, "1.32.0", Some(51994), None),
|
||||||
// `Self` in type definitions (RFC 2300)
|
// `Self` in type definitions (RFC 2300)
|
||||||
(accepted, self_in_typedefs, "1.32.0", Some(49303), None),
|
(accepted, self_in_typedefs, "1.32.0", Some(49303), None),
|
||||||
|
// `use path as _;` and `extern crate c as _;`
|
||||||
|
(accepted, underscore_imports, "1.33.0", Some(48216), None),
|
||||||
);
|
);
|
||||||
|
|
||||||
// If you change this, please modify `src/doc/unstable-book` as well. You must
|
// If you change this, please modify `src/doc/unstable-book` as well. You must
|
||||||
|
@ -1547,26 +1546,8 @@ impl<'a> Visitor<'a> for PostExpansionVisitor<'a> {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
fn visit_use_tree(&mut self, use_tree: &'a ast::UseTree, id: NodeId, _nested: bool) {
|
|
||||||
if let ast::UseTreeKind::Simple(Some(ident), ..) = use_tree.kind {
|
|
||||||
if ident.name == "_" {
|
|
||||||
gate_feature_post!(&self, underscore_imports, use_tree.span,
|
|
||||||
"renaming imports with `_` is unstable");
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
visit::walk_use_tree(self, use_tree, id);
|
|
||||||
}
|
|
||||||
|
|
||||||
fn visit_item(&mut self, i: &'a ast::Item) {
|
fn visit_item(&mut self, i: &'a ast::Item) {
|
||||||
match i.node {
|
match i.node {
|
||||||
ast::ItemKind::ExternCrate(_) => {
|
|
||||||
if i.ident.name == "_" {
|
|
||||||
gate_feature_post!(&self, underscore_imports, i.span,
|
|
||||||
"renaming extern crates with `_` is unstable");
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
ast::ItemKind::Static(..) |
|
ast::ItemKind::Static(..) |
|
||||||
ast::ItemKind::Const(_,_) => {
|
ast::ItemKind::Const(_,_) => {
|
||||||
if i.ident.name == "_" {
|
if i.ident.name == "_" {
|
||||||
|
|
|
@ -1,14 +0,0 @@
|
||||||
// Copyright 2018 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.
|
|
||||||
|
|
||||||
extern crate std as _; //~ ERROR renaming extern crates with `_` is unstable
|
|
||||||
use std::vec as _; //~ ERROR renaming imports with `_` is unstable
|
|
||||||
|
|
||||||
fn main() {}
|
|
|
@ -1,19 +0,0 @@
|
||||||
error[E0658]: renaming extern crates with `_` is unstable (see issue #48216)
|
|
||||||
--> $DIR/feature-gate-underscore-imports.rs:11:1
|
|
||||||
|
|
|
||||||
LL | extern crate std as _; //~ ERROR renaming extern crates with `_` is unstable
|
|
||||||
| ^^^^^^^^^^^^^^^^^^^^^^
|
|
||||||
|
|
|
||||||
= help: add #![feature(underscore_imports)] to the crate attributes to enable
|
|
||||||
|
|
||||||
error[E0658]: renaming imports with `_` is unstable (see issue #48216)
|
|
||||||
--> $DIR/feature-gate-underscore-imports.rs:12:5
|
|
||||||
|
|
|
||||||
LL | use std::vec as _; //~ ERROR renaming imports with `_` is unstable
|
|
||||||
| ^^^^^^^^^^^^^
|
|
||||||
|
|
|
||||||
= help: add #![feature(underscore_imports)] to the crate attributes to enable
|
|
||||||
|
|
||||||
error: aborting due to 2 previous errors
|
|
||||||
|
|
||||||
For more information about this error, try `rustc --explain E0658`.
|
|
|
@ -1,5 +1,3 @@
|
||||||
#![feature(underscore_imports)]
|
|
||||||
|
|
||||||
#[macro_export]
|
#[macro_export]
|
||||||
macro_rules! do_nothing {
|
macro_rules! do_nothing {
|
||||||
() => ()
|
() => ()
|
||||||
|
|
|
@ -11,7 +11,6 @@
|
||||||
// compile-pass
|
// compile-pass
|
||||||
// aux-build:underscore-imports.rs
|
// aux-build:underscore-imports.rs
|
||||||
|
|
||||||
#![feature(underscore_imports)]
|
|
||||||
#![warn(unused_imports, unused_extern_crates)]
|
#![warn(unused_imports, unused_extern_crates)]
|
||||||
|
|
||||||
#[macro_use]
|
#[macro_use]
|
||||||
|
|
|
@ -1,17 +1,17 @@
|
||||||
warning: unused import: `m::Tr1 as _`
|
warning: unused import: `m::Tr1 as _`
|
||||||
--> $DIR/basic.rs:37:9
|
--> $DIR/basic.rs:36:9
|
||||||
|
|
|
|
||||||
LL | use m::Tr1 as _; //~ WARN unused import
|
LL | use m::Tr1 as _; //~ WARN unused import
|
||||||
| ^^^^^^^^^^^
|
| ^^^^^^^^^^^
|
||||||
|
|
|
|
||||||
note: lint level defined here
|
note: lint level defined here
|
||||||
--> $DIR/basic.rs:15:9
|
--> $DIR/basic.rs:14:9
|
||||||
|
|
|
|
||||||
LL | #![warn(unused_imports, unused_extern_crates)]
|
LL | #![warn(unused_imports, unused_extern_crates)]
|
||||||
| ^^^^^^^^^^^^^^
|
| ^^^^^^^^^^^^^^
|
||||||
|
|
||||||
warning: unused import: `S as _`
|
warning: unused import: `S as _`
|
||||||
--> $DIR/basic.rs:38:9
|
--> $DIR/basic.rs:37:9
|
||||||
|
|
|
|
||||||
LL | use S as _; //~ WARN unused import
|
LL | use S as _; //~ WARN unused import
|
||||||
| ^^^^^^
|
| ^^^^^^
|
||||||
|
|
|
@ -1,8 +1,6 @@
|
||||||
// compile-pass
|
// compile-pass
|
||||||
// aux-build:duplicate.rs
|
// aux-build:duplicate.rs
|
||||||
|
|
||||||
#![feature(underscore_imports)]
|
|
||||||
|
|
||||||
extern crate duplicate;
|
extern crate duplicate;
|
||||||
|
|
||||||
#[duplicate::duplicate]
|
#[duplicate::duplicate]
|
||||||
|
|
|
@ -1,6 +1,5 @@
|
||||||
// edition:2018
|
// edition:2018
|
||||||
|
|
||||||
#![feature(underscore_imports)]
|
|
||||||
#![deny(unused_imports)]
|
#![deny(unused_imports)]
|
||||||
|
|
||||||
mod multi_segment {
|
mod multi_segment {
|
||||||
|
|
|
@ -1,17 +1,17 @@
|
||||||
error: unused import: `core::any`
|
error: unused import: `core::any`
|
||||||
--> $DIR/unused-2018.rs:7:9
|
--> $DIR/unused-2018.rs:6:9
|
||||||
|
|
|
|
||||||
LL | use core::any; //~ ERROR unused import: `core::any`
|
LL | use core::any; //~ ERROR unused import: `core::any`
|
||||||
| ^^^^^^^^^
|
| ^^^^^^^^^
|
||||||
|
|
|
|
||||||
note: lint level defined here
|
note: lint level defined here
|
||||||
--> $DIR/unused-2018.rs:4:9
|
--> $DIR/unused-2018.rs:3:9
|
||||||
|
|
|
|
||||||
LL | #![deny(unused_imports)]
|
LL | #![deny(unused_imports)]
|
||||||
| ^^^^^^^^^^^^^^
|
| ^^^^^^^^^^^^^^
|
||||||
|
|
||||||
error: unused import: `core`
|
error: unused import: `core`
|
||||||
--> $DIR/unused-2018.rs:11:9
|
--> $DIR/unused-2018.rs:10:9
|
||||||
|
|
|
|
||||||
LL | use core; //~ ERROR unused import: `core`
|
LL | use core; //~ ERROR unused import: `core`
|
||||||
| ^^^^
|
| ^^^^
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
// edition:2018
|
// edition:2018
|
||||||
|
|
||||||
#![feature(uniform_paths, underscore_imports)]
|
#![feature(uniform_paths)]
|
||||||
|
|
||||||
mod T {
|
mod T {
|
||||||
pub struct U;
|
pub struct U;
|
||||||
|
|
|
@ -2,7 +2,7 @@
|
||||||
|
|
||||||
// For the time being `macro_rules` items are treated as *very* private...
|
// For the time being `macro_rules` items are treated as *very* private...
|
||||||
|
|
||||||
#![feature(underscore_imports, decl_macro, uniform_paths)]
|
#![feature(decl_macro, uniform_paths)]
|
||||||
|
|
||||||
mod m1 {
|
mod m1 {
|
||||||
macro_rules! legacy_macro { () => () }
|
macro_rules! legacy_macro { () => () }
|
||||||
|
|
Loading…
Reference in a new issue