Swap span_lint for span_lint_and_sugg

This implements a machine applicable suggestion to any matched usage of
`.as_ref().take()``

Author: b-NC

clippy_lints/src/methods/option_take_on_temporary.rs

@@ -1,27 +1,30 @@
-use clippy_utils::diagnostics::span_lint;
+use clippy_utils::diagnostics::span_lint_and_sugg;
+use clippy_utils::source::snippet_with_applicability;
 use clippy_utils::ty::is_type_diagnostic_item;
+use rustc_errors::Applicability;
 use rustc_hir::Expr;
 use rustc_lint::LateContext;
 use rustc_span::sym;
 
 use super::OPTION_TAKE_ON_TEMPORARY;
 
-pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>, _recv: &'tcx Expr<'_>) {
+pub(super) fn check<'tcx>(cx: &LateContext<'tcx>, expr: &'tcx Expr<'_>, recv: &'tcx Expr<'_>) {
     // Checks if expression type is equal to sym::Option and if the expr is not a syntactic place
     if is_expr_option(cx, expr) & !expr.is_syntactic_place_expr() {
-        span_lint(cx, OPTION_TAKE_ON_TEMPORARY, expr.span, "Format test");
+        let mut applicability = Applicability::MachineApplicable;
+        span_lint_and_sugg(
+            cx,
+            OPTION_TAKE_ON_TEMPORARY,
+            expr.span,
+            "Called `Option::take()` on a temporary value",
+            "try",
+            format!(
+                "{}",
+                snippet_with_applicability(cx, recv.span, "..", &mut applicability)
+            ),
+            Applicability::MachineApplicable,
+        );
     }
-    /*    if_chain! {
-        is_expr_option(cx, expr);
-        then {
-            span_lint(
-                cx,
-                OPTION_TAKE_ON_TEMPORARY,
-                expr.span,
-                "Format test"
-            );
-        }
-    };*/
 }
 
 fn is_expr_option(cx: &LateContext<'_>, expr: &Expr<'_>) -> bool {

tests/ui/option_take_on_temporary.fixed

@@ -0,0 +1,7 @@
+// run-rustfix
+
+fn main() {
+    println!("Testing option_take_on_temporary");
+    let x = Some(3);
+    let y = x.as_ref();
+}

tests/ui/option_take_on_temporary.rs

@@ -1,3 +1,5 @@
+// run-rustfix
+
 fn main() {
     println!("Testing option_take_on_temporary");
     let x = Some(3);

tests/ui/option_take_on_temporary.stderr

@@ -1,8 +1,8 @@
-error: Format test
-  --> $DIR/option_take_on_temporary.rs:4:13
+error: Called `Option::take()` on a temporary value
+  --> $DIR/option_take_on_temporary.rs:6:13
    |
 LL |     let y = x.as_ref().take();
-   |             ^^^^^^^^^^^^^^^^^
+   |             ^^^^^^^^^^^^^^^^^ help: try: `x.as_ref()`
    |
    = note: `-D clippy::option-take-on-temporary` implied by `-D warnings`