Fix incorrect mutable suggestion information for binding in ref pattern.

For ref pattern in func param, the mutability suggestion has to apply to the binding.

For example: `fn foo(&x: &i32)` -> `fn foo(&(mut x): &i32)`

fixes rust-lang#122415

Author: surechen

compiler/rustc_borrowck/src/diagnostics/mutability_errors.rs

@@ -728,25 +728,40 @@ impl<'a, 'tcx> MirBorrowckCtxt<'a, 'tcx> {
             _ => local_decl.source_info.span,
         };
 
+        // With ref-binding patterns, the mutability suggestion has to apply to
+        // the binding, not the reference (which would be a type error):
+        //
+        // `let &b = a;` -> `let &(mut b) = a;`
+        // or
+        // `fn foo(&x: &i32)` -> `fn foo(&(mut x): &i32)`
         let def_id = self.body.source.def_id();
-        let hir_id = if let Some(local_def_id) = def_id.as_local()
+        let is_ref_binding_patterns = if let Some(local_def_id) = def_id.as_local()
             && let Some(body_id) = self.infcx.tcx.hir().maybe_body_owned_by(local_def_id)
         {
             let body = self.infcx.tcx.hir().body(body_id);
-            BindingFinder { span: pat_span }.visit_body(body).break_value()
+
+            if let Some(hir_id) = (BindingFinder { span: pat_span }).visit_body(body).break_value()
+                && let hir::Node::Local(hir::Local {
+                    pat: hir::Pat { kind: hir::PatKind::Ref(_, _), .. },
+                    ..
+                }) = self.infcx.tcx.hir_node(hir_id)
+            {
+                true
+            } else {
+                body.params.iter().any(|param| {
+                    if let hir::Pat { kind: hir::PatKind::Ref(_, _), span, .. } = param.pat
+                        && *span == pat_span
+                    {
+                        return true;
+                    }
+                    false
+                })
+            }
         } else {
-            None
+            false
         };
 
-        // With ref-binding patterns, the mutability suggestion has to apply to
-        // the binding, not the reference (which would be a type error):
-        //
-        // `let &b = a;` -> `let &(mut b) = a;`
-        if let Some(hir_id) = hir_id
-            && let hir::Node::Local(hir::Local {
-                pat: hir::Pat { kind: hir::PatKind::Ref(_, _), .. },
-                ..
-            }) = self.infcx.tcx.hir_node(hir_id)
+        if is_ref_binding_patterns
             && let Ok(name) =
                 self.infcx.tcx.sess.source_map().span_to_snippet(local_decl.source_info.span)
         {

src/tools/tidy/src/issues.txt

@@ -3466,7 +3466,6 @@
 "ui/pattern/issue-106552.rs",
 "ui/pattern/issue-106862.rs",
 "ui/pattern/issue-110508.rs",
-"ui/pattern/issue-114896.rs",
 "ui/pattern/issue-115599.rs",
 "ui/pattern/issue-11577.rs",
 "ui/pattern/issue-117626.rs",

tests/ui/pattern/patkind-ref-binding-issue-114896.fixed

@@ -0,0 +1,10 @@
+//@ run-rustfix
+#![allow(dead_code)]
+
+fn main() {
+    fn x(a: &char) {
+        let &(mut b) = a;
+        b.make_ascii_uppercase();
+//~^ cannot borrow `b` as mutable, as it is not declared as mutable
+    }
+}

tests/ui/pattern/issue-114896.rs renamed to tests/ui/pattern/patkind-ref-binding-issue-114896.rs

@@ -1,3 +1,6 @@
+//@ run-rustfix
+#![allow(dead_code)]
+
 fn main() {
     fn x(a: &char) {
         let &b = a;

tests/ui/pattern/issue-114896.stderr renamed to tests/ui/pattern/patkind-ref-binding-issue-114896.stderr

@@ -1,5 +1,5 @@
 error[E0596]: cannot borrow `b` as mutable, as it is not declared as mutable
-  --> $DIR/issue-114896.rs:4:9
+  --> $DIR/patkind-ref-binding-issue-114896.rs:7:9
    |
 LL |         let &b = a;
    |             -- help: consider changing this to be mutable: `&(mut b)`

tests/ui/pattern/patkind-ref-binding-issue-122415.fixed

@@ -0,0 +1,11 @@
+//@ run-rustfix
+#![allow(dead_code)]
+
+fn mutate(_y: &mut i32) {}
+
+fn foo(&(mut x): &i32) {
+    mutate(&mut x);
+    //~^ ERROR cannot borrow `x` as mutable
+}
+
+fn main() {}

tests/ui/pattern/patkind-ref-binding-issue-122415.rs

@@ -0,0 +1,11 @@
+//@ run-rustfix
+#![allow(dead_code)]
+
+fn mutate(_y: &mut i32) {}
+
+fn foo(&x: &i32) {
+    mutate(&mut x);
+    //~^ ERROR cannot borrow `x` as mutable
+}
+
+fn main() {}

tests/ui/pattern/patkind-ref-binding-issue-122415.stderr

@@ -0,0 +1,11 @@
+error[E0596]: cannot borrow `x` as mutable, as it is not declared as mutable
+  --> $DIR/patkind-ref-binding-issue-122415.rs:7:12
+   |
+LL | fn foo(&x: &i32) {
+   |        -- help: consider changing this to be mutable: `&(mut x)`
+LL |     mutate(&mut x);
+   |            ^^^^^^ cannot borrow as mutable
+
+error: aborting due to 1 previous error
+
+For more information about this error, try `rustc --explain E0596`.