X-Git-Url: https://git.lizzy.rs/?a=blobdiff_plain;f=clippy_lints%2Fsrc%2Fdbg_macro.rs;h=48fc601e726cf6ef35c86b8192c3708731b8f2cc;hb=6d1225981177587fbb68d9c4902c770c3dbaafb0;hp=637e19bd60e32e40dba021f760c02f6995fd3610;hpb=a7fe3b2ebb35e6bca78bc103a75966105981abc4;p=rust.git diff --git a/clippy_lints/src/dbg_macro.rs b/clippy_lints/src/dbg_macro.rs index 637e19bd60e..48fc601e726 100644 --- a/clippy_lints/src/dbg_macro.rs +++ b/clippy_lints/src/dbg_macro.rs @@ -1,8 +1,8 @@ -use crate::utils::sym; use crate::utils::{snippet_opt, span_help_and_lint, span_lint_and_sugg}; +use rustc::declare_lint_pass; use rustc::lint::{EarlyContext, EarlyLintPass, LintArray, LintPass}; -use rustc::{declare_lint_pass, declare_tool_lint}; use rustc_errors::Applicability; +use rustc_session::declare_tool_lint; use syntax::ast; use syntax::source_map::Span; use syntax::tokenstream::TokenStream; @@ -32,12 +32,12 @@ impl EarlyLintPass for DbgMacro { fn check_mac(&mut self, cx: &EarlyContext<'_>, mac: &ast::Mac) { - if mac.node.path == *sym::dbg { - if let Some(sugg) = tts_span(mac.node.tts.clone()).and_then(|span| snippet_opt(cx, span)) { + if mac.path == sym!(dbg) { + if let Some(sugg) = tts_span(mac.args.inner_tokens()).and_then(|span| snippet_opt(cx, span)) { span_lint_and_sugg( cx, DBG_MACRO, - mac.span, + mac.span(), "`dbg!` macro is intended as a debugging tool", "ensure to avoid having uses of it in version control", sugg, @@ -47,7 +47,7 @@ fn check_mac(&mut self, cx: &EarlyContext<'_>, mac: &ast::Mac) { span_help_and_lint( cx, DBG_MACRO, - mac.span, + mac.span(), "`dbg!` macro is intended as a debugging tool", "ensure to avoid having uses of it in version control", ); @@ -60,9 +60,6 @@ fn check_mac(&mut self, cx: &EarlyContext<'_>, mac: &ast::Mac) { fn tts_span(tts: TokenStream) -> Option { let mut cursor = tts.into_trees(); let first = cursor.next()?.span(); - let span = match cursor.last() { - Some(tree) => first.to(tree.span()), - None => first, - }; + let span = cursor.last().map_or(first, |tree| first.to(tree.span())); Some(span) }