squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Refactor if_chain

Browse source 
Co-authored-by: Philipp Krones <hello@philkrones.com>
This commit is contained in:
Jeremy Stucki 2019-08-11 21:02:01 +02:00
parent b651f19eb8
commit 5fd7d44f36
No known key found for this signature in database
GPG key ID: EEFCA93148042655
1 changed files with 25 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

49
clippy_lints/src/methods/mod.rs
View file

@ -2174,38 +2174,39 @@ fn lint_flat_map_identity<'a, 'tcx>(
if match_trait_method(cx, expr, &paths::ITERATOR);
if flat_map_args.len() == 2;
if let hir::ExprKind::Closure(_, _, body_id, _, _) = flat_map_args[1].node;
let body = cx.tcx.hir().body(body_id);
if body.arguments.len() == 1;
if let hir::PatKind::Binding(_, _, binding_ident, _) = body.arguments[0].pat.node;
if let hir::ExprKind::Path(hir::QPath::Resolved(_, ref path)) = body.value.node;
if path.segments.len() == 1;
if path.segments[0].ident.as_str() == binding_ident.as_str();
then {
let msg = "called `flat_map(|x| x)` on an `Iterator`. \
This can be simplified by calling `flatten().`";
span_lint(cx, FLAT_MAP_IDENTITY, expr.span, msg);
}
}
if_chain! {
if let hir::ExprKind::Closure(_, _, body_id, _, _) = flat_map_args[1].node;
let body = cx.tcx.hir().body(body_id);
if_chain! {
if match_trait_method(cx, expr, &paths::ITERATOR);
if body.arguments.len() == 1;
if let hir::PatKind::Binding(_, _, binding_ident, _) = body.arguments[0].pat.node;
if let hir::ExprKind::Path(hir::QPath::Resolved(_, ref path)) = body.value.node;
if flat_map_args.len() == 2;
if path.segments.len() == 1;
if path.segments[0].ident.as_str() == binding_ident.as_str();
let expr = &flat_map_args[1];
then {
let msg = "called `flat_map(|x| x)` on an `Iterator`. \
This can be simplified by calling `flatten().`";
span_lint(cx, FLAT_MAP_IDENTITY, expr.span, msg);
}
}
if let hir::ExprKind::Path(ref qpath) = expr.node;
if_chain! {
let expr = &flat_map_args[1];
if match_qpath(qpath, &paths::STD_CONVERT_IDENTITY);
if let hir::ExprKind::Path(ref qpath) = expr.node;
then {
let msg = "called `flat_map(std::convert::identity)` on an `Iterator`. \
This can be simplified by calling `flatten().`";
span_lint(cx, FLAT_MAP_IDENTITY, expr.span, msg);
if match_qpath(qpath, &paths::STD_CONVERT_IDENTITY);
then {
let msg = "called `flat_map(std::convert::identity)` on an `Iterator`. \
This can be simplified by calling `flatten().`";
span_lint(cx, FLAT_MAP_IDENTITY, expr.span, msg);
}
}
}
}
}