Rollup merge of rust-lang#137712 - meithecatte:extract-binding-mode, r=oli-obk

jieyouxu · web-flow · commit 222aaba5a18a · 2025-02-28T21:42:00.000+08:00
Clean up TypeckResults::extract_binding_mode

- Remove the `Option` from the result type, as `None` is never returned.
- Document the difference from the `BindingMode` in `PatKind::Binding`.
diff --git a/clippy_utils/src/lib.rs b/clippy_utils/src/lib.rs
@@ -1094,7 +1094,6 @@ pub fn capture_local_usage(cx: &LateContext<'_>, e: &Expr<'_>) -> CaptureKind {
         pat.each_binding_or_first(&mut |_, id, span, _| match cx
             .typeck_results()
             .extract_binding_mode(cx.sess(), id, span)
-            .unwrap()
             .0
         {
             ByRef::No if !is_copy(cx, cx.typeck_results().node_type(id)) => {

Original file line number	Diff line number	Diff line change
`@@ -1094,7 +1094,6 @@ pub fn capture_local_usage(cx: &LateContext<'_>, e: &Expr<'_>) -> CaptureKind {`
`1094`	`1094`	`pat.each_binding_or_first(&mut \|_, id, span, _\| match cx`
`1095`	`1095`	`.typeck_results()`
`1096`	`1096`	`.extract_binding_mode(cx.sess(), id, span)`
`1097`		`- .unwrap()`
`1098`	`1097`	`.0`
`1099`	`1098`	`{`
`1100`	`1099`	`ByRef::No if !is_copy(cx, cx.typeck_results().node_type(id)) => {`