Address review comments

Author: scottgerring

clippy_lints/src/unit_types/let_unit_value.rs

@@ -82,20 +82,18 @@ pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, local: &'tcx LetStmt<'_>) {
 
                     // If this is a binding pattern, we need to add suggestions to remove any usages
                     // of the variable
-                    if let PatKind::Binding(_, binding_hir_id, ident, ..) = local.pat.kind {
-                        if let Some(body_id) = cx.enclosing_body.as_ref() {
-                            let body = cx.tcx.hir().body(*body_id);
-
-                            // Collect variable usages
-                            let mut visitor = UnitVariableCollector::new(binding_hir_id);
-                            walk_body(&mut visitor, body);
-
-                            // Add suggestions for replacing variable usages
-                            let has_usages = !visitor.spans.is_empty();
-                            if has_usages {
-                                suggestions.extend(visitor.spans.into_iter().map(|span| (span, "()".to_string())));
-                            }
-                        }
+                    if let PatKind::Binding(_, binding_hir_id, ident, ..) = local.pat.kind
+                        && let Some(body_id) = cx.enclosing_body.as_ref()
+                    {
+                        let body = cx.tcx.hir().body(*body_id);
+
+                        // Collect variable usages
+                        let mut visitor = UnitVariableCollector::new(binding_hir_id);
+                        walk_body(&mut visitor, body);
+
+                        // Add suggestions for replacing variable usages
+                        let has_usages = !visitor.spans.is_empty();
+                        suggestions.extend(visitor.spans.into_iter().map(|span| (span, "()".to_string())));
                     }
 
                     // Emit appropriate diagnostic based on whether there are usages of the let binding