PR Followups

Author: andrewpollack

clippy_lints/src/while_let_some_result.rs

@@ -38,11 +38,11 @@ declare_lint_pass!(WhileLetSomeResult => [WHILE_LET_SOME_RESULT]);
 
 impl<'tcx> LateLintPass<'tcx> for WhileLetSomeResult {
     fn check_expr(&mut self, cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>) {
-        if_chain! { //begin checking variables
-            if let ExprKind::Match(op, body, MatchSource::WhileLetDesugar { .. }) = expr.kind; //test if expr is `while let`
-            if let ExprKind::MethodCall(_, ok_span, result_types, _) = op.kind; //check is expr.ok() has type Result<T,E>.ok(, _)
-            if let PatKind::TupleStruct(QPath::Resolved(_, x), y, _)  = body[0].pat.kind; //get operation
-            if method_chain_args(op, &["ok"]).is_some(); //test to see if using ok() methoduse std::marker::Sized;
+        if_chain! {
+            if let ExprKind::Match(op, body, MatchSource::WhileLetDesugar { .. }) = expr.kind; // test if expr is `while let`
+            if let ExprKind::MethodCall(_, ok_span, result_types, _) = op.kind;
+            if let PatKind::TupleStruct(QPath::Resolved(_, x), y, _)  = body[0].pat.kind; // get operation
+            if method_chain_args(op, &["ok"]).is_some(); // test to see if using ok() methoduse std::marker::Sized;
             if is_type_diagnostic_item(cx, cx.typeck_results().expr_ty(&result_types[0]), sym::result_type);
             if rustc_hir_pretty::to_string(rustc_hir_pretty::NO_ANN, |s| s.print_path(x, false)) == "Some";