2021-03-25 18:29:11 +00:00
|
|
|
use clippy_utils::diagnostics::span_lint_and_sugg;
|
2021-12-30 14:10:43 +00:00
|
|
|
use clippy_utils::in_macro;
|
2021-03-25 18:29:11 +00:00
|
|
|
use clippy_utils::source::snippet;
|
2020-06-23 15:05:22 +00:00
|
|
|
use hir::def::{DefKind, Res};
|
2020-02-26 12:40:31 +00:00
|
|
|
use if_chain::if_chain;
|
|
|
|
use rustc_ast::ast;
|
2020-06-23 15:05:22 +00:00
|
|
|
use rustc_data_structures::fx::{FxHashMap, FxHashSet};
|
2020-02-26 12:40:31 +00:00
|
|
|
use rustc_errors::Applicability;
|
2020-06-23 15:05:22 +00:00
|
|
|
use rustc_hir as hir;
|
|
|
|
use rustc_lint::{LateContext, LateLintPass, LintContext};
|
|
|
|
use rustc_session::{declare_tool_lint, impl_lint_pass};
|
2021-04-08 15:50:13 +00:00
|
|
|
use rustc_span::{edition::Edition, sym, Span};
|
2020-02-26 12:40:31 +00:00
|
|
|
|
|
|
|
declare_clippy_lint! {
|
2021-07-29 10:16:06 +00:00
|
|
|
/// ### What it does
|
|
|
|
/// Checks for `#[macro_use] use...`.
|
2020-02-26 12:40:31 +00:00
|
|
|
///
|
2021-07-29 10:16:06 +00:00
|
|
|
/// ### Why is this bad?
|
|
|
|
/// Since the Rust 2018 edition you can import
|
2020-02-26 12:40:31 +00:00
|
|
|
/// macro's directly, this is considered idiomatic.
|
|
|
|
///
|
2021-07-29 10:16:06 +00:00
|
|
|
/// ### Example
|
2020-10-09 10:45:29 +00:00
|
|
|
/// ```rust,ignore
|
2020-02-26 12:40:31 +00:00
|
|
|
/// #[macro_use]
|
2020-10-09 10:45:29 +00:00
|
|
|
/// use some_macro;
|
2020-02-26 12:40:31 +00:00
|
|
|
/// ```
|
2021-12-06 11:33:31 +00:00
|
|
|
#[clippy::version = "1.44.0"]
|
2020-03-05 18:22:17 +00:00
|
|
|
pub MACRO_USE_IMPORTS,
|
2020-02-26 12:40:31 +00:00
|
|
|
pedantic,
|
|
|
|
"#[macro_use] is no longer needed"
|
|
|
|
}
|
|
|
|
|
2020-06-23 15:05:22 +00:00
|
|
|
#[derive(Clone, Debug, PartialEq, Eq)]
|
|
|
|
struct PathAndSpan {
|
|
|
|
path: String,
|
|
|
|
span: Span,
|
|
|
|
}
|
|
|
|
|
2021-05-21 17:35:49 +00:00
|
|
|
/// `MacroRefData` includes the name of the macro.
|
2020-06-23 15:05:22 +00:00
|
|
|
#[derive(Debug, Clone)]
|
|
|
|
pub struct MacroRefData {
|
|
|
|
name: String,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl MacroRefData {
|
2021-05-21 17:35:49 +00:00
|
|
|
pub fn new(name: String) -> Self {
|
|
|
|
Self { name }
|
2020-06-23 15:05:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Default)]
|
|
|
|
#[allow(clippy::module_name_repetitions)]
|
|
|
|
pub struct MacroUseImports {
|
|
|
|
/// the actual import path used and the span of the attribute above it.
|
|
|
|
imports: Vec<(String, Span)>,
|
|
|
|
/// the span of the macro reference, kept to ensure only one reference is used per macro call.
|
|
|
|
collected: FxHashSet<Span>,
|
|
|
|
mac_refs: Vec<MacroRefData>,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl_lint_pass!(MacroUseImports => [MACRO_USE_IMPORTS]);
|
|
|
|
|
|
|
|
impl MacroUseImports {
|
2020-06-25 20:41:36 +00:00
|
|
|
fn push_unique_macro(&mut self, cx: &LateContext<'_>, span: Span) {
|
2020-06-23 15:05:22 +00:00
|
|
|
let call_site = span.source_callsite();
|
|
|
|
let name = snippet(cx, cx.sess().source_map().span_until_char(call_site, '!'), "_");
|
2021-09-28 17:03:12 +00:00
|
|
|
if span.source_callee().is_some() && !self.collected.contains(&call_site) {
|
|
|
|
let name = if name.contains("::") {
|
|
|
|
name.split("::").last().unwrap().to_string()
|
|
|
|
} else {
|
|
|
|
name.to_string()
|
|
|
|
};
|
2020-06-23 15:05:22 +00:00
|
|
|
|
2021-09-28 17:03:12 +00:00
|
|
|
self.mac_refs.push(MacroRefData::new(name));
|
|
|
|
self.collected.insert(call_site);
|
2020-06-23 15:05:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-06-25 20:41:36 +00:00
|
|
|
fn push_unique_macro_pat_ty(&mut self, cx: &LateContext<'_>, span: Span) {
|
2020-06-23 15:05:22 +00:00
|
|
|
let call_site = span.source_callsite();
|
|
|
|
let name = snippet(cx, cx.sess().source_map().span_until_char(call_site, '!'), "_");
|
2021-09-28 17:03:12 +00:00
|
|
|
if span.source_callee().is_some() && !self.collected.contains(&call_site) {
|
|
|
|
self.mac_refs.push(MacroRefData::new(name.to_string()));
|
|
|
|
self.collected.insert(call_site);
|
2020-06-23 15:05:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2020-06-25 20:41:36 +00:00
|
|
|
impl<'tcx> LateLintPass<'tcx> for MacroUseImports {
|
|
|
|
fn check_item(&mut self, cx: &LateContext<'_>, item: &hir::Item<'_>) {
|
2020-02-26 12:40:31 +00:00
|
|
|
if_chain! {
|
2021-01-02 15:29:43 +00:00
|
|
|
if cx.sess().opts.edition >= Edition::Edition2018;
|
2020-06-23 15:05:22 +00:00
|
|
|
if let hir::ItemKind::Use(path, _kind) = &item.kind;
|
2021-01-24 12:17:54 +00:00
|
|
|
let attrs = cx.tcx.hir().attrs(item.hir_id());
|
2021-04-08 15:50:13 +00:00
|
|
|
if let Some(mac_attr) = attrs.iter().find(|attr| attr.has_name(sym::macro_use));
|
2020-06-23 15:05:22 +00:00
|
|
|
if let Res::Def(DefKind::Mod, id) = path.res;
|
2021-04-22 09:31:13 +00:00
|
|
|
if !id.is_local();
|
2020-02-26 12:40:31 +00:00
|
|
|
then {
|
2020-06-23 15:05:22 +00:00
|
|
|
for kid in cx.tcx.item_children(id).iter() {
|
|
|
|
if let Res::Def(DefKind::Macro(_mac_type), mac_id) = kid.res {
|
|
|
|
let span = mac_attr.span;
|
|
|
|
let def_path = cx.tcx.def_path_str(mac_id);
|
|
|
|
self.imports.push((def_path, span));
|
|
|
|
}
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
if in_macro(item.span) {
|
|
|
|
self.push_unique_macro_pat_ty(cx, item.span);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2020-06-25 20:41:36 +00:00
|
|
|
fn check_attribute(&mut self, cx: &LateContext<'_>, attr: &ast::Attribute) {
|
2020-06-23 15:05:22 +00:00
|
|
|
if in_macro(attr.span) {
|
|
|
|
self.push_unique_macro(cx, attr.span);
|
|
|
|
}
|
|
|
|
}
|
2020-06-25 20:41:36 +00:00
|
|
|
fn check_expr(&mut self, cx: &LateContext<'_>, expr: &hir::Expr<'_>) {
|
2020-06-23 15:05:22 +00:00
|
|
|
if in_macro(expr.span) {
|
|
|
|
self.push_unique_macro(cx, expr.span);
|
|
|
|
}
|
|
|
|
}
|
2020-06-25 20:41:36 +00:00
|
|
|
fn check_stmt(&mut self, cx: &LateContext<'_>, stmt: &hir::Stmt<'_>) {
|
2020-06-23 15:05:22 +00:00
|
|
|
if in_macro(stmt.span) {
|
|
|
|
self.push_unique_macro(cx, stmt.span);
|
|
|
|
}
|
|
|
|
}
|
2020-06-25 20:41:36 +00:00
|
|
|
fn check_pat(&mut self, cx: &LateContext<'_>, pat: &hir::Pat<'_>) {
|
2020-06-23 15:05:22 +00:00
|
|
|
if in_macro(pat.span) {
|
|
|
|
self.push_unique_macro_pat_ty(cx, pat.span);
|
|
|
|
}
|
|
|
|
}
|
2020-06-25 20:41:36 +00:00
|
|
|
fn check_ty(&mut self, cx: &LateContext<'_>, ty: &hir::Ty<'_>) {
|
2020-06-23 15:05:22 +00:00
|
|
|
if in_macro(ty.span) {
|
|
|
|
self.push_unique_macro_pat_ty(cx, ty.span);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
#[allow(clippy::too_many_lines)]
|
2021-09-12 09:58:27 +00:00
|
|
|
fn check_crate_post(&mut self, cx: &LateContext<'_>) {
|
2020-06-23 15:05:22 +00:00
|
|
|
let mut used = FxHashMap::default();
|
|
|
|
let mut check_dup = vec![];
|
|
|
|
for (import, span) in &self.imports {
|
|
|
|
let found_idx = self.mac_refs.iter().position(|mac| import.ends_with(&mac.name));
|
|
|
|
|
|
|
|
if let Some(idx) = found_idx {
|
2021-02-11 14:04:38 +00:00
|
|
|
self.mac_refs.remove(idx);
|
2020-06-23 15:05:22 +00:00
|
|
|
let seg = import.split("::").collect::<Vec<_>>();
|
|
|
|
|
|
|
|
match seg.as_slice() {
|
|
|
|
// an empty path is impossible
|
|
|
|
// a path should always consist of 2 or more segments
|
|
|
|
[] | [_] => return,
|
|
|
|
[root, item] => {
|
|
|
|
if !check_dup.contains(&(*item).to_string()) {
|
|
|
|
used.entry(((*root).to_string(), span))
|
|
|
|
.or_insert_with(Vec::new)
|
|
|
|
.push((*item).to_string());
|
|
|
|
check_dup.push((*item).to_string());
|
|
|
|
}
|
2021-05-20 10:30:31 +00:00
|
|
|
},
|
2020-06-23 15:05:22 +00:00
|
|
|
[root, rest @ ..] => {
|
|
|
|
if rest.iter().all(|item| !check_dup.contains(&(*item).to_string())) {
|
|
|
|
let filtered = rest
|
|
|
|
.iter()
|
|
|
|
.filter_map(|item| {
|
|
|
|
if check_dup.contains(&(*item).to_string()) {
|
|
|
|
None
|
|
|
|
} else {
|
|
|
|
Some((*item).to_string())
|
|
|
|
}
|
|
|
|
})
|
|
|
|
.collect::<Vec<_>>();
|
|
|
|
used.entry(((*root).to_string(), span))
|
|
|
|
.or_insert_with(Vec::new)
|
|
|
|
.push(filtered.join("::"));
|
|
|
|
check_dup.extend(filtered);
|
|
|
|
} else {
|
|
|
|
let rest = rest.to_vec();
|
|
|
|
used.entry(((*root).to_string(), span))
|
|
|
|
.or_insert_with(Vec::new)
|
|
|
|
.push(rest.join("::"));
|
|
|
|
check_dup.extend(rest.iter().map(ToString::to_string));
|
|
|
|
}
|
2021-05-20 10:30:31 +00:00
|
|
|
},
|
2020-06-23 15:05:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
let mut suggestions = vec![];
|
|
|
|
for ((root, span), path) in used {
|
|
|
|
if path.len() == 1 {
|
2021-06-03 06:41:37 +00:00
|
|
|
suggestions.push((span, format!("{}::{}", root, path[0])));
|
2020-06-23 15:05:22 +00:00
|
|
|
} else {
|
2021-06-03 06:41:37 +00:00
|
|
|
suggestions.push((span, format!("{}::{{{}}}", root, path.join(", "))));
|
2020-06-23 15:05:22 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
// If mac_refs is not empty we have encountered an import we could not handle
|
|
|
|
// such as `std::prelude::v1::foo` or some other macro that expands to an import.
|
|
|
|
if self.mac_refs.is_empty() {
|
|
|
|
for (span, import) in suggestions {
|
|
|
|
let help = format!("use {};", import);
|
2020-02-26 12:40:31 +00:00
|
|
|
span_lint_and_sugg(
|
2020-06-23 15:05:22 +00:00
|
|
|
cx,
|
2020-03-05 18:22:17 +00:00
|
|
|
MACRO_USE_IMPORTS,
|
2020-06-23 15:05:22 +00:00
|
|
|
*span,
|
|
|
|
"`macro_use` attributes are no longer needed in the Rust 2018 edition",
|
2020-02-26 12:40:31 +00:00
|
|
|
"remove the attribute and import the macro directly, try",
|
|
|
|
help,
|
2020-06-23 15:05:22 +00:00
|
|
|
Applicability::MaybeIncorrect,
|
2021-06-03 06:41:37 +00:00
|
|
|
);
|
2020-02-26 12:40:31 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|