2015-08-16 01:54:43 -05:00
|
|
|
|
use rustc::lint::*;
|
2015-09-03 09:42:17 -05:00
|
|
|
|
use rustc_front::hir::*;
|
|
|
|
|
use reexport::*;
|
2015-08-26 17:31:35 -05:00
|
|
|
|
use syntax::codemap::{ExpnInfo, Span, ExpnFormat};
|
2015-09-06 13:57:06 -05:00
|
|
|
|
use rustc::front::map::Node::*;
|
2015-08-25 16:21:38 -05:00
|
|
|
|
use rustc::middle::def_id::DefId;
|
2015-06-07 05:05:14 -05:00
|
|
|
|
use rustc::middle::ty;
|
2015-08-16 01:54:43 -05:00
|
|
|
|
use std::borrow::Cow;
|
2015-06-01 15:30:34 -05:00
|
|
|
|
|
2015-08-21 11:48:36 -05:00
|
|
|
|
// module DefPaths for certain structs/enums we check for
|
|
|
|
|
pub const OPTION_PATH: [&'static str; 3] = ["core", "option", "Option"];
|
|
|
|
|
pub const RESULT_PATH: [&'static str; 3] = ["core", "result", "Result"];
|
|
|
|
|
pub const STRING_PATH: [&'static str; 3] = ["collections", "string", "String"];
|
|
|
|
|
pub const VEC_PATH: [&'static str; 3] = ["collections", "vec", "Vec"];
|
|
|
|
|
pub const LL_PATH: [&'static str; 3] = ["collections", "linked_list", "LinkedList"];
|
|
|
|
|
|
2015-09-06 03:53:55 -05:00
|
|
|
|
/// returns true this expn_info was expanded by any macro
|
|
|
|
|
pub fn in_macro(cx: &Context, span: Span) -> bool {
|
|
|
|
|
cx.sess().codemap().with_expn_info(span.expn_id,
|
|
|
|
|
|info| info.map_or(false, |i| {
|
|
|
|
|
match i.callee.format {
|
|
|
|
|
ExpnFormat::CompilerExpansion(..) => false,
|
|
|
|
|
_ => true,
|
2015-08-26 17:31:35 -05:00
|
|
|
|
}
|
2015-09-06 03:53:55 -05:00
|
|
|
|
}))
|
2015-06-01 15:30:34 -05:00
|
|
|
|
}
|
2015-06-07 05:05:14 -05:00
|
|
|
|
|
2015-09-06 03:53:55 -05:00
|
|
|
|
/// returns true if the macro that expanded the crate was outside of
|
|
|
|
|
/// the current crate or was a compiler plugin
|
2015-06-07 05:05:14 -05:00
|
|
|
|
pub fn in_external_macro(cx: &Context, span: Span) -> bool {
|
2015-09-06 03:53:55 -05:00
|
|
|
|
/// invokes in_macro with the expansion info of the given span
|
|
|
|
|
/// slightly heavy, try to use this after other checks have already happened
|
|
|
|
|
fn in_macro_ext(cx: &Context, opt_info: Option<&ExpnInfo>) -> bool {
|
|
|
|
|
// no ExpnInfo = no macro
|
|
|
|
|
opt_info.map_or(false, |info| {
|
|
|
|
|
match info.callee.format {
|
|
|
|
|
ExpnFormat::CompilerExpansion(..) => {
|
|
|
|
|
if info.callee.name() == "closure expansion" {
|
|
|
|
|
return false;
|
|
|
|
|
}
|
|
|
|
|
},
|
|
|
|
|
ExpnFormat::MacroAttribute(..) => {
|
|
|
|
|
// these are all plugins
|
|
|
|
|
return true;
|
|
|
|
|
},
|
|
|
|
|
_ => (),
|
|
|
|
|
}
|
|
|
|
|
// no span for the callee = external macro
|
|
|
|
|
info.callee.span.map_or(true, |span| {
|
|
|
|
|
// no snippet = external macro or compiler-builtin expansion
|
|
|
|
|
cx.sess().codemap().span_to_snippet(span).ok().map_or(true, |code|
|
|
|
|
|
// macro doesn't start with "macro_rules"
|
|
|
|
|
// = compiler plugin
|
|
|
|
|
!code.starts_with("macro_rules")
|
|
|
|
|
)
|
|
|
|
|
})
|
|
|
|
|
})
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-11 13:22:20 -05:00
|
|
|
|
cx.sess().codemap().with_expn_info(span.expn_id,
|
2015-09-06 03:53:55 -05:00
|
|
|
|
|info| in_macro_ext(cx, info))
|
2015-06-07 05:05:14 -05:00
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
/// check if a DefId's path matches the given absolute type path
|
|
|
|
|
/// usage e.g. with
|
|
|
|
|
/// `match_def_path(cx, id, &["core", "option", "Option"])`
|
|
|
|
|
pub fn match_def_path(cx: &Context, def_id: DefId, path: &[&str]) -> bool {
|
2015-08-25 11:26:20 -05:00
|
|
|
|
cx.tcx.with_path(def_id, |iter| iter.zip(path).all(|(nm, p)| nm.name() == p))
|
2015-06-07 05:05:14 -05:00
|
|
|
|
}
|
|
|
|
|
|
2015-08-21 12:00:33 -05:00
|
|
|
|
/// check if type is struct or enum type with given def path
|
|
|
|
|
pub fn match_type(cx: &Context, ty: ty::Ty, path: &[&str]) -> bool {
|
|
|
|
|
match ty.sty {
|
|
|
|
|
ty::TyEnum(ref adt, _) | ty::TyStruct(ref adt, _) => {
|
|
|
|
|
match_def_path(cx, adt.did, path)
|
|
|
|
|
}
|
|
|
|
|
_ => {
|
|
|
|
|
false
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-23 09:32:50 -05:00
|
|
|
|
/// check if method call given in "expr" belongs to given trait
|
|
|
|
|
pub fn match_trait_method(cx: &Context, expr: &Expr, path: &[&str]) -> bool {
|
|
|
|
|
let method_call = ty::MethodCall::expr(expr.id);
|
|
|
|
|
let trt_id = cx.tcx.tables
|
|
|
|
|
.borrow().method_map.get(&method_call)
|
|
|
|
|
.and_then(|callee| cx.tcx.trait_of_item(callee.def_id));
|
|
|
|
|
if let Some(trt_id) = trt_id {
|
|
|
|
|
match_def_path(cx, trt_id, path)
|
|
|
|
|
} else {
|
|
|
|
|
false
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-06-07 05:05:14 -05:00
|
|
|
|
/// match a Path against a slice of segment string literals, e.g.
|
|
|
|
|
/// `match_path(path, &["std", "rt", "begin_unwind"])`
|
|
|
|
|
pub fn match_path(path: &Path, segments: &[&str]) -> bool {
|
2015-08-11 08:07:21 -05:00
|
|
|
|
path.segments.iter().rev().zip(segments.iter().rev()).all(
|
2015-08-16 01:33:10 -05:00
|
|
|
|
|(a, b)| &a.identifier.name == b)
|
2015-06-07 05:05:14 -05:00
|
|
|
|
}
|
2015-07-09 10:02:21 -05:00
|
|
|
|
|
2015-09-06 14:03:09 -05:00
|
|
|
|
/// get the name of the item the expression is in, if available
|
|
|
|
|
pub fn get_item_name(cx: &Context, expr: &Expr) -> Option<Name> {
|
2015-09-06 13:57:06 -05:00
|
|
|
|
let parent_id = cx.tcx.map.get_parent(expr.id);
|
|
|
|
|
match cx.tcx.map.find(parent_id) {
|
|
|
|
|
Some(NodeItem(&Item{ ref ident, .. })) |
|
|
|
|
|
Some(NodeTraitItem(&TraitItem{ id: _, ref ident, .. })) |
|
|
|
|
|
Some(NodeImplItem(&ImplItem{ id: _, ref ident, .. })) => {
|
2015-09-06 14:03:09 -05:00
|
|
|
|
Some(ident.name)
|
2015-09-06 13:57:06 -05:00
|
|
|
|
},
|
|
|
|
|
_ => None,
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-07-09 10:02:21 -05:00
|
|
|
|
/// convert a span to a code snippet if available, otherwise use default, e.g.
|
|
|
|
|
/// `snippet(cx, expr.span, "..")`
|
|
|
|
|
pub fn snippet<'a>(cx: &Context, span: Span, default: &'a str) -> Cow<'a, str> {
|
2015-08-11 08:07:21 -05:00
|
|
|
|
cx.sess().codemap().span_to_snippet(span).map(From::from).unwrap_or(Cow::Borrowed(default))
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-12 06:14:14 -05:00
|
|
|
|
/// convert a span (from a block) to a code snippet if available, otherwise use default, e.g.
|
|
|
|
|
/// `snippet(cx, expr.span, "..")`
|
|
|
|
|
/// This trims the code of indentation, except for the first line
|
|
|
|
|
/// Use it for blocks or block-like things which need to be printed as such
|
|
|
|
|
pub fn snippet_block<'a>(cx: &Context, span: Span, default: &'a str) -> Cow<'a, str> {
|
|
|
|
|
let snip = snippet(cx, span, default);
|
|
|
|
|
trim_multiline(snip, true)
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-29 04:41:06 -05:00
|
|
|
|
/// Like snippet_block, but add braces if the expr is not an ExprBlock
|
|
|
|
|
pub fn expr_block<'a>(cx: &Context, expr: &Expr, default: &'a str) -> Cow<'a, str> {
|
|
|
|
|
let code = snippet_block(cx, expr.span, default);
|
|
|
|
|
if let ExprBlock(_) = expr.node {
|
|
|
|
|
code
|
|
|
|
|
} else {
|
|
|
|
|
Cow::Owned(format!("{{ {} }}", code))
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-12 06:14:14 -05:00
|
|
|
|
/// Trim indentation from a multiline string
|
|
|
|
|
/// with possibility of ignoring the first line
|
2015-08-13 12:50:00 -05:00
|
|
|
|
pub fn trim_multiline(s: Cow<str>, ignore_first: bool) -> Cow<str> {
|
2015-08-21 10:11:34 -05:00
|
|
|
|
let s_space = trim_multiline_inner(s, ignore_first, ' ');
|
|
|
|
|
let s_tab = trim_multiline_inner(s_space, ignore_first, '\t');
|
|
|
|
|
trim_multiline_inner(s_tab, ignore_first, ' ')
|
2015-08-13 02:57:30 -05:00
|
|
|
|
}
|
|
|
|
|
|
2015-08-13 12:50:00 -05:00
|
|
|
|
fn trim_multiline_inner(s: Cow<str>, ignore_first: bool, ch: char) -> Cow<str> {
|
2015-08-12 06:14:14 -05:00
|
|
|
|
let x = s.lines().skip(ignore_first as usize)
|
2015-08-13 08:59:12 -05:00
|
|
|
|
.filter_map(|l| { if l.len() > 0 { // ignore empty lines
|
|
|
|
|
Some(l.char_indices()
|
|
|
|
|
.find(|&(_,x)| x != ch)
|
|
|
|
|
.unwrap_or((l.len(), ch)).0)
|
|
|
|
|
} else {None}})
|
2015-08-12 06:14:14 -05:00
|
|
|
|
.min().unwrap_or(0);
|
|
|
|
|
if x > 0 {
|
2015-08-13 08:59:12 -05:00
|
|
|
|
Cow::Owned(s.lines().enumerate().map(|(i,l)| if (ignore_first && i == 0) ||
|
|
|
|
|
l.len() == 0 {
|
2015-08-12 06:14:14 -05:00
|
|
|
|
l
|
|
|
|
|
} else {
|
|
|
|
|
l.split_at(x).1
|
|
|
|
|
}).collect::<Vec<_>>()
|
|
|
|
|
.join("\n"))
|
|
|
|
|
} else {
|
|
|
|
|
s
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-11 08:07:21 -05:00
|
|
|
|
/// get a parent expr if any – this is useful to constrain a lint
|
|
|
|
|
pub fn get_parent_expr<'c>(cx: &'c Context, e: &Expr) -> Option<&'c Expr> {
|
|
|
|
|
let map = &cx.tcx.map;
|
|
|
|
|
let node_id : NodeId = e.id;
|
|
|
|
|
let parent_id : NodeId = map.get_parent_node(node_id);
|
|
|
|
|
if node_id == parent_id { return None; }
|
2015-08-11 13:22:20 -05:00
|
|
|
|
map.find(parent_id).and_then(|node|
|
2015-08-11 08:07:21 -05:00
|
|
|
|
if let NodeExpr(parent) = node { Some(parent) } else { None } )
|
2015-07-09 10:02:21 -05:00
|
|
|
|
}
|
2015-07-16 01:53:02 -05:00
|
|
|
|
|
2015-07-26 09:53:11 -05:00
|
|
|
|
#[cfg(not(feature="structured_logging"))]
|
|
|
|
|
pub fn span_lint(cx: &Context, lint: &'static Lint, sp: Span, msg: &str) {
|
2015-08-11 08:07:21 -05:00
|
|
|
|
cx.span_lint(lint, sp, msg);
|
2015-08-27 00:39:40 -05:00
|
|
|
|
if cx.current_level(lint) != Level::Allow {
|
|
|
|
|
cx.sess().fileline_help(sp, &format!("for further information visit \
|
|
|
|
|
https://github.com/Manishearth/rust-clippy/wiki#{}",
|
|
|
|
|
lint.name_lower()))
|
|
|
|
|
}
|
2015-07-26 09:53:11 -05:00
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
#[cfg(feature="structured_logging")]
|
|
|
|
|
pub fn span_lint(cx: &Context, lint: &'static Lint, sp: Span, msg: &str) {
|
2015-08-11 08:07:21 -05:00
|
|
|
|
// lint.name / lint.desc is can give details of the lint
|
|
|
|
|
// cx.sess().codemap() has all these nice functions for line/column/snippet details
|
|
|
|
|
// http://doc.rust-lang.org/syntax/codemap/struct.CodeMap.html#method.span_to_string
|
|
|
|
|
cx.span_lint(lint, sp, msg);
|
2015-08-27 00:39:40 -05:00
|
|
|
|
if cx.current_level(lint) != Level::Allow {
|
|
|
|
|
cx.sess().fileline_help(sp, &format!("for further information visit \
|
|
|
|
|
https://github.com/Manishearth/rust-clippy/wiki#{}",
|
|
|
|
|
lint.name_lower()))
|
|
|
|
|
}
|
2015-08-11 08:07:21 -05:00
|
|
|
|
}
|
|
|
|
|
|
2015-08-11 13:22:20 -05:00
|
|
|
|
pub fn span_help_and_lint(cx: &Context, lint: &'static Lint, span: Span,
|
2015-08-11 08:07:21 -05:00
|
|
|
|
msg: &str, help: &str) {
|
2015-09-02 01:19:47 -05:00
|
|
|
|
cx.span_lint(lint, span, msg);
|
2015-08-11 08:07:21 -05:00
|
|
|
|
if cx.current_level(lint) != Level::Allow {
|
2015-08-27 00:39:40 -05:00
|
|
|
|
cx.sess().fileline_help(span, &format!("{}\nfor further information \
|
|
|
|
|
visit https://github.com/Manishearth/rust-clippy/wiki#{}",
|
|
|
|
|
help, lint.name_lower()))
|
2015-08-11 08:07:21 -05:00
|
|
|
|
}
|
2015-07-26 09:53:11 -05:00
|
|
|
|
}
|
2015-08-11 13:57:21 -05:00
|
|
|
|
|
2015-09-02 01:19:47 -05:00
|
|
|
|
pub fn span_note_and_lint(cx: &Context, lint: &'static Lint, span: Span,
|
|
|
|
|
msg: &str, note_span: Span, note: &str) {
|
|
|
|
|
cx.span_lint(lint, span, msg);
|
|
|
|
|
if cx.current_level(lint) != Level::Allow {
|
|
|
|
|
if note_span == span {
|
|
|
|
|
cx.sess().fileline_note(note_span, note)
|
|
|
|
|
} else {
|
|
|
|
|
cx.sess().span_note(note_span, note)
|
|
|
|
|
}
|
|
|
|
|
cx.sess().fileline_help(span, &format!("for further information visit \
|
|
|
|
|
https://github.com/Manishearth/rust-clippy/wiki#{}",
|
|
|
|
|
lint.name_lower()))
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-11 13:57:21 -05:00
|
|
|
|
/// return the base type for references and raw pointers
|
2015-08-13 09:28:11 -05:00
|
|
|
|
pub fn walk_ptrs_ty(ty: ty::Ty) -> ty::Ty {
|
2015-08-11 13:57:21 -05:00
|
|
|
|
match ty.sty {
|
|
|
|
|
ty::TyRef(_, ref tm) | ty::TyRawPtr(ref tm) => walk_ptrs_ty(tm.ty),
|
|
|
|
|
_ => ty
|
|
|
|
|
}
|
|
|
|
|
}
|
2015-08-12 00:48:00 -05:00
|
|
|
|
|
2015-08-25 11:38:08 -05:00
|
|
|
|
/// return the base type for references and raw pointers, and count reference depth
|
|
|
|
|
pub fn walk_ptrs_ty_depth(ty: ty::Ty) -> (ty::Ty, usize) {
|
|
|
|
|
fn inner(ty: ty::Ty, depth: usize) -> (ty::Ty, usize) {
|
|
|
|
|
match ty.sty {
|
|
|
|
|
ty::TyRef(_, ref tm) | ty::TyRawPtr(ref tm) => inner(tm.ty, depth + 1),
|
|
|
|
|
_ => (ty, depth)
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
inner(ty, 0)
|
|
|
|
|
}
|
|
|
|
|
|
2015-08-12 14:11:32 -05:00
|
|
|
|
/// Produce a nested chain of if-lets and ifs from the patterns:
|
2015-08-12 00:48:00 -05:00
|
|
|
|
///
|
2015-08-12 14:11:32 -05:00
|
|
|
|
/// if_let_chain! {
|
|
|
|
|
/// [
|
|
|
|
|
/// Some(y) = x,
|
|
|
|
|
/// y.len() == 2,
|
|
|
|
|
/// Some(z) = y,
|
|
|
|
|
/// ],
|
2015-08-12 00:48:00 -05:00
|
|
|
|
/// {
|
|
|
|
|
/// block
|
|
|
|
|
/// }
|
|
|
|
|
/// }
|
|
|
|
|
///
|
|
|
|
|
/// becomes
|
|
|
|
|
///
|
|
|
|
|
/// if let Some(y) = x {
|
2015-08-12 14:11:32 -05:00
|
|
|
|
/// if y.len() == 2 {
|
|
|
|
|
/// if let Some(z) = y {
|
|
|
|
|
/// block
|
|
|
|
|
/// }
|
2015-08-12 00:48:00 -05:00
|
|
|
|
/// }
|
|
|
|
|
/// }
|
|
|
|
|
#[macro_export]
|
|
|
|
|
macro_rules! if_let_chain {
|
2015-08-12 14:11:32 -05:00
|
|
|
|
([let $pat:pat = $expr:expr, $($tt:tt)+], $block:block) => {
|
2015-08-12 00:48:00 -05:00
|
|
|
|
if let $pat = $expr {
|
2015-08-12 14:11:32 -05:00
|
|
|
|
if_let_chain!{ [$($tt)+], $block }
|
2015-08-12 00:48:00 -05:00
|
|
|
|
}
|
|
|
|
|
};
|
2015-08-12 14:11:32 -05:00
|
|
|
|
([let $pat:pat = $expr:expr], $block:block) => {
|
2015-08-12 00:48:00 -05:00
|
|
|
|
if let $pat = $expr {
|
|
|
|
|
$block
|
|
|
|
|
}
|
|
|
|
|
};
|
2015-08-12 14:11:32 -05:00
|
|
|
|
([$expr:expr, $($tt:tt)+], $block:block) => {
|
|
|
|
|
if $expr {
|
|
|
|
|
if_let_chain!{ [$($tt)+], $block }
|
|
|
|
|
}
|
|
|
|
|
};
|
|
|
|
|
([$expr:expr], $block:block) => {
|
|
|
|
|
if $expr {
|
|
|
|
|
$block
|
|
|
|
|
}
|
|
|
|
|
};
|
2015-08-12 00:48:00 -05:00
|
|
|
|
}
|