[unnecessary_literal_unwrap]: don't lint if binding initializer is expn

Author: y21

clippy_lints/src/methods/unnecessary_literal_unwrap.rs

@@ -29,6 +29,11 @@ pub(super) fn check(
     args: &[hir::Expr<'_>],
 ) {
     let init = clippy_utils::expr_or_init(cx, recv);
+    if init.span.from_expansion() {
+        // don't lint if the receiver or binding initializer comes from a macro
+        // (e.g. `let x = option_env!(..); x.unwrap()`)
+        return;
+    }
 
     let (constructor, call_args, ty) = if let hir::ExprKind::Call(call, call_args) = init.kind {
         let Some(qpath) = call.qpath_opt() else { return };

tests/ui/unnecessary_literal_unwrap.fixed

@@ -68,6 +68,11 @@ fn unwrap_methods_result() {
     1;
 }
 
+fn unwrap_from_binding() {
+    let val = option_env!("does_not_exist");
+    let _ = val.unwrap_or("");
+}
+
 fn main() {
     unwrap_option_some();
     unwrap_option_none();

tests/ui/unnecessary_literal_unwrap.rs

@@ -68,6 +68,11 @@ fn unwrap_methods_result() {
     Ok::<_, ()>(1).unwrap_or_else(|_| 2);
 }
 
+fn unwrap_from_binding() {
+    let val = option_env!("does_not_exist");
+    let _ = val.unwrap_or("");
+}
+
 fn main() {
     unwrap_option_some();
     unwrap_option_none();