cargo dev fmt

Author: Henri Lunnikivi

clippy_lints/src/field_reassign_with_default.rs

@@ -1,10 +1,10 @@
 use crate::utils::{match_def_path, paths, qpath_res, snippet, span_lint_and_note};
 use if_chain::if_chain;
-use rustc_hir::def::Res;
 use rustc_data_structures::fx::FxHashMap;
+use rustc_hir::def::Res;
 use rustc_hir::{Block, Expr, ExprKind, PatKind, QPath, Stmt, StmtKind};
-use rustc_span::symbol::{Ident, Symbol};
 use rustc_middle::ty::{self, Adt, TyS};
+use rustc_span::symbol::{Ident, Symbol};
 
 use rustc_lint::{LateContext, LateLintPass};
 use rustc_session::{declare_lint_pass, declare_tool_lint};
@@ -41,11 +41,11 @@ declare_lint_pass!(FieldReassignWithDefault => [FIELD_REASSIGN_WITH_DEFAULT]);
 
 impl LateLintPass<'_> for FieldReassignWithDefault {
     fn check_block(&mut self, cx: &LateContext<'_>, block: &Block<'_>) {
-
         // find all binding statements like `let mut _ = T::default()` where `T::default()` is the
         // `default` method of the `Default` trait. and store statement index in current block being
         // checked and the name of the bound variable
-        let binding_statements_using_default: Vec<(usize, Symbol, &TyS<'_>)> = enumerate_bindings_using_default(cx, block);
+        let binding_statements_using_default: Vec<(usize, Symbol, &TyS<'_>)> =
+            enumerate_bindings_using_default(cx, block);
 
         // start from the `let mut _ = _::default();` and look at all the following
         // statements, see if they re-assign the fields of the binding
@@ -69,7 +69,9 @@ impl LateLintPass<'_> for FieldReassignWithDefault {
                 // shadow a binding
                 else {
                     // find out if and which field was set by this `consequtive_statement`
-                    if let Some((field_ident, assign_rhs)) = field_reassigned_by_stmt(consequtive_statement, &binding_name) {
+                    if let Some((field_ident, assign_rhs)) =
+                        field_reassigned_by_stmt(consequtive_statement, &binding_name)
+                    {
                         // extract and store the assigned value for help message
                         let value_snippet = snippet(cx, assign_rhs.span, "..");
 
@@ -91,9 +93,10 @@ impl LateLintPass<'_> for FieldReassignWithDefault {
                 let stmt = &block.stmts[stmt_idx];
 
                 if let StmtKind::Local(preceding_local) = &stmt.kind {
-
                     // if all fields of the struct are not assigned, add `.. Default::default()` to the suggestion.
-                    let ext_with_default = !fields_of_type(&binding_type).iter().all(|field| assigned_fields.contains_key(&field.name));
+                    let ext_with_default = !fields_of_type(&binding_type)
+                        .iter()
+                        .all(|field| assigned_fields.contains_key(&field.name));
 
                     let field_list = assigned_fields
                         .into_iter()
@@ -116,15 +119,18 @@ impl LateLintPass<'_> for FieldReassignWithDefault {
                         Some(preceding_local.span),
                         &format!("consider initializing the variable with `{}`", sugg),
                     );
-
                 }
             }
         }
     }
 }
 
-/// Returns the block indices, identifiers and types of bindings set as `Default::default()`, except for when the pattern type is a tuple.
-fn enumerate_bindings_using_default<'cx, 'hir>(cx: &LateContext<'cx>, block: &Block<'hir>) -> Vec<(usize, Symbol, &'cx TyS<'cx>)> {
+/// Returns the block indices, identifiers and types of bindings set as `Default::default()`, except
+/// for when the pattern type is a tuple.
+fn enumerate_bindings_using_default<'cx, 'hir>(
+    cx: &LateContext<'cx>,
+    block: &Block<'hir>,
+) -> Vec<(usize, Symbol, &'cx TyS<'cx>)> {
     block
         .stmts
         .iter()
@@ -155,7 +161,8 @@ fn enumerate_bindings_using_default<'cx, 'hir>(cx: &LateContext<'cx>, block: &Bl
                     None
                 }
             }
-        }).collect()
+        })
+        .collect()
 }
 
 fn stmt_shadows_binding(this: &Stmt<'_>, shadowed: &Symbol) -> bool {
@@ -198,4 +205,4 @@ fn fields_of_type<'a>(ty: &'a TyS<'_>) -> Vec<Ident> {
         }
     }
     vec![]
-}
+}