rustc: Polish off in_external_macro
This commit polishes off this new function to compile on newer rustc as well as update and add a suite of test cases to work with this new check for lints.
This commit is contained in:
parent
dd0808dd24
commit
8adf08c437
5 changed files with 112 additions and 39 deletions
|
|
@ -7,9 +7,18 @@
|
|||
// <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.
|
||||
#![feature(nll)]
|
||||
#![deny(elided_lifetime_in_path)]
|
||||
|
||||
fn main() {
|
||||
format!("foo {}", 22)
|
||||
#[macro_export]
|
||||
macro_rules! bar {
|
||||
() => {use std::string::ToString;}
|
||||
}
|
||||
|
||||
#[macro_export]
|
||||
macro_rules! baz {
|
||||
($i:item) => ($i)
|
||||
}
|
||||
|
||||
#[macro_export]
|
||||
macro_rules! baz2 {
|
||||
($($i:tt)*) => ($($i)*)
|
||||
}
|
||||
28
src/test/ui/lint/lints-in-foreign-macros.rs
Normal file
28
src/test/ui/lint/lints-in-foreign-macros.rs
Normal file
|
|
@ -0,0 +1,28 @@
|
|||
// 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.
|
||||
|
||||
// aux-build:lints-in-foreign-macros.rs
|
||||
// compile-pass
|
||||
|
||||
#![warn(unused_imports)]
|
||||
|
||||
#[macro_use]
|
||||
extern crate lints_in_foreign_macros;
|
||||
|
||||
macro_rules! foo {
|
||||
() => {use std::string::ToString;} //~ WARN: unused import
|
||||
}
|
||||
|
||||
mod a { foo!(); }
|
||||
mod b { bar!(); }
|
||||
mod c { baz!(use std::string::ToString;); } //~ WARN: unused import
|
||||
mod d { baz2!(use std::string::ToString;); } //~ WARN: unused import
|
||||
|
||||
fn main() {}
|
||||
27
src/test/ui/lint/lints-in-foreign-macros.stderr
Normal file
27
src/test/ui/lint/lints-in-foreign-macros.stderr
Normal file
|
|
@ -0,0 +1,27 @@
|
|||
warning: unused import: `std::string::ToString`
|
||||
--> $DIR/lints-in-foreign-macros.rs:20:16
|
||||
|
|
||||
LL | () => {use std::string::ToString;} //~ WARN: unused import
|
||||
| ^^^^^^^^^^^^^^^^^^^^^
|
||||
...
|
||||
LL | mod a { foo!(); }
|
||||
| ------- in this macro invocation
|
||||
|
|
||||
note: lint level defined here
|
||||
--> $DIR/lints-in-foreign-macros.rs:14:9
|
||||
|
|
||||
LL | #![warn(unused_imports)]
|
||||
| ^^^^^^^^^^^^^^
|
||||
|
||||
warning: unused import: `std::string::ToString`
|
||||
--> $DIR/lints-in-foreign-macros.rs:25:18
|
||||
|
|
||||
LL | mod c { baz!(use std::string::ToString;); } //~ WARN: unused import
|
||||
| ^^^^^^^^^^^^^^^^^^^^^
|
||||
|
||||
warning: unused import: `std::string::ToString`
|
||||
--> $DIR/lints-in-foreign-macros.rs:26:19
|
||||
|
|
||||
LL | mod d { baz2!(use std::string::ToString;); } //~ WARN: unused import
|
||||
| ^^^^^^^^^^^^^^^^^^^^^
|
||||
|
||||
Loading…
Add table
Add a link
Reference in a new issue