Cargo fmt
This commit is contained in:
@@ -79,7 +79,16 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for BoxedLocal {
|
|||||||
|
|
||||||
let fn_def_id = cx.tcx.hir().local_def_id_from_hir_id(hir_id);
|
let fn_def_id = cx.tcx.hir().local_def_id_from_hir_id(hir_id);
|
||||||
let region_scope_tree = &cx.tcx.region_scope_tree(fn_def_id);
|
let region_scope_tree = &cx.tcx.region_scope_tree(fn_def_id);
|
||||||
ExprUseVisitor::new(&mut v, cx.tcx, fn_def_id, cx.param_env, region_scope_tree, cx.tables, None).consume_body(body);
|
ExprUseVisitor::new(
|
||||||
|
&mut v,
|
||||||
|
cx.tcx,
|
||||||
|
fn_def_id,
|
||||||
|
cx.param_env,
|
||||||
|
region_scope_tree,
|
||||||
|
cx.tables,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
.consume_body(body);
|
||||||
|
|
||||||
for node in v.set {
|
for node in v.set {
|
||||||
span_lint(
|
span_lint(
|
||||||
|
|||||||
@@ -1662,7 +1662,16 @@ fn check_for_mutation(
|
|||||||
};
|
};
|
||||||
let def_id = def_id::DefId::local(body.hir_id.owner);
|
let def_id = def_id::DefId::local(body.hir_id.owner);
|
||||||
let region_scope_tree = &cx.tcx.region_scope_tree(def_id);
|
let region_scope_tree = &cx.tcx.region_scope_tree(def_id);
|
||||||
ExprUseVisitor::new(&mut delegate, cx.tcx, def_id, cx.param_env, region_scope_tree, cx.tables, None).walk_expr(body);
|
ExprUseVisitor::new(
|
||||||
|
&mut delegate,
|
||||||
|
cx.tcx,
|
||||||
|
def_id,
|
||||||
|
cx.param_env,
|
||||||
|
region_scope_tree,
|
||||||
|
cx.tables,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
.walk_expr(body);
|
||||||
delegate.mutation_span()
|
delegate.mutation_span()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -137,8 +137,16 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for NeedlessPassByValue {
|
|||||||
} = {
|
} = {
|
||||||
let mut ctx = MovedVariablesCtxt::new(cx);
|
let mut ctx = MovedVariablesCtxt::new(cx);
|
||||||
let region_scope_tree = &cx.tcx.region_scope_tree(fn_def_id);
|
let region_scope_tree = &cx.tcx.region_scope_tree(fn_def_id);
|
||||||
euv::ExprUseVisitor::new(&mut ctx, cx.tcx, fn_def_id, cx.param_env, region_scope_tree, cx.tables, None)
|
euv::ExprUseVisitor::new(
|
||||||
.consume_body(body);
|
&mut ctx,
|
||||||
|
cx.tcx,
|
||||||
|
fn_def_id,
|
||||||
|
cx.param_env,
|
||||||
|
region_scope_tree,
|
||||||
|
cx.tables,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
.consume_body(body);
|
||||||
ctx
|
ctx
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -16,7 +16,16 @@ pub fn mutated_variables<'a, 'tcx: 'a>(expr: &'tcx Expr, cx: &'a LateContext<'a,
|
|||||||
};
|
};
|
||||||
let def_id = def_id::DefId::local(expr.hir_id.owner);
|
let def_id = def_id::DefId::local(expr.hir_id.owner);
|
||||||
let region_scope_tree = &cx.tcx.region_scope_tree(def_id);
|
let region_scope_tree = &cx.tcx.region_scope_tree(def_id);
|
||||||
ExprUseVisitor::new(&mut delegate, cx.tcx, def_id, cx.param_env, region_scope_tree, cx.tables, None).walk_expr(expr);
|
ExprUseVisitor::new(
|
||||||
|
&mut delegate,
|
||||||
|
cx.tcx,
|
||||||
|
def_id,
|
||||||
|
cx.param_env,
|
||||||
|
region_scope_tree,
|
||||||
|
cx.tables,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
.walk_expr(expr);
|
||||||
|
|
||||||
if delegate.skip {
|
if delegate.skip {
|
||||||
return None;
|
return None;
|
||||||
@@ -32,7 +41,7 @@ pub fn is_potentially_mutated<'a, 'tcx: 'a>(
|
|||||||
if let Res::Local(id) = variable.res {
|
if let Res::Local(id) = variable.res {
|
||||||
mutated_variables(expr, cx).map_or(true, |mutated| mutated.contains(&id))
|
mutated_variables(expr, cx).map_or(true, |mutated| mutated.contains(&id))
|
||||||
} else {
|
} else {
|
||||||
return true
|
return true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user