consider wait() calls in nested bodies

Author: y21

clippy_lints/src/zombie_processes.rs

@@ -6,6 +6,7 @@ use rustc_errors::Applicability;
 use rustc_hir::intravisit::{Visitor, walk_block, walk_expr, walk_local};
 use rustc_hir::{Expr,ExprKind,HirId,LetStmt,Node,PatKind,Stmt,StmtKind};
 use rustc_lint::{LateContext,LateLintPass};
+use rustc_middle::hir::nested_filter;
 use rustc_session::declare_lint_pass;
 use rustc_span::sym;
 use std::ops::ControlFlow;
@@ -119,6 +120,7 @@ enum WaitFinder<'a, 'tcx> {
 }
 
 impl<'tcx>Visitor<'tcx>forWaitFinder<'_,'tcx>{
+typeNestedFilter = nested_filter::OnlyBodies;
 typeResult = ControlFlow<BreakReason>;
 
 fnvisit_local(&mutself,l:&'tcxLetStmt<'tcx>) -> Self::Result{
@@ -204,6 +206,11 @@ impl<'tcx> Visitor<'tcx> for WaitFinder<'_, 'tcx> {
 
 walk_expr(self, ex)
 }
+
+fnnested_visit_map(&mutself) -> Self::Map{
+let(Self::Found(cx, _) | Self::WalkUpTo(cx, _)) = self;
+ cx.tcx.hir()
+}
 }
 
 /// This function has shared logic between the different kinds of nodes that can trigger the lint.

tests/ui/zombie_processes.rs

@@ -131,6 +131,13 @@ fn main() {
 }
  x.wait().unwrap();
 }
+
+{
+letmut x = Command::new("").spawn().unwrap();
+ std::thread::spawn(move || {
+ x.wait().unwrap();
+});
+}
 }
 
 fnprocess_child(c:Child){