Use `multipart_suggestion` to create an applicable suggestion.

The "consider explicitly droping" can now suggest a machine applicable
suggestion now.

diff --git a/compiler/rustc_lint/src/let_underscore.rs b/compiler/rustc_lint/src/let_underscore.rs
index 520c865cc1922d690c6860de17d953a0afa14d15..18661e8c5059c714dab92c3a87a0211c59e75302 100644 (file)
--- a/compiler/rustc_lint/src/let_underscore.rs
+++ b/compiler/rustc_lint/src/let_underscore.rs
@@ -154,11 +154,13 @@ fn build_and_emit_lint(
             "_unused",
             Applicability::MachineApplicable,
         )
-        .span_suggestion_verbose(
-            init_span,
+        .multipart_suggestion(
             "consider explicitly droping with `std::mem::drop`",
-            "drop(...)",
-            Applicability::HasPlaceholders,
+            vec![
+                (init_span.shrink_to_lo(), "drop(".to_string()),
+                (init_span.shrink_to_hi(), ")".to_string()),
+            ],
+            Applicability::MachineApplicable,
         )
         .emit();
 }

diff --git a/src/test/ui/lint/let_underscore/let_underscore_drop.stderr b/src/test/ui/lint/let_underscore/let_underscore_drop.stderr
index 5034f682bb76bf73b0b8f2783352701fc9a8a9f2..dfac6d3f7418593bd7fa5e9bac20f4a63341e60b 100644 (file)
--- a/src/test/ui/lint/let_underscore/let_underscore_drop.stderr
+++ b/src/test/ui/lint/let_underscore/let_underscore_drop.stderr
@@ -11,8 +11,8 @@ LL |     let _unused = NontrivialDrop;
    |         ~~~~~~~
 help: consider explicitly droping with `std::mem::drop`
    |
-LL |     let _ = drop(...);
-   |             ~~~~~~~~~
+LL |     let _ = drop(NontrivialDrop);
+   |             +++++              +
 
 warning: 1 warning emitted
 

diff --git a/src/test/ui/lint/let_underscore/let_underscore_lock.stderr b/src/test/ui/lint/let_underscore/let_underscore_lock.stderr
index 7ff42cb152441bf8aa89e97f66cae963b47e4f6b..f37483ddd96d844ccad13bc7842905e6e537cdce 100644 (file)
--- a/src/test/ui/lint/let_underscore/let_underscore_lock.stderr
+++ b/src/test/ui/lint/let_underscore/let_underscore_lock.stderr
@@ -11,8 +11,8 @@ LL |     let _unused = data.lock().unwrap();
    |         ~~~~~~~
 help: consider explicitly droping with `std::mem::drop`
    |
-LL |     let _ = drop(...);
-   |             ~~~~~~~~~
+LL |     let _ = drop(data.lock().unwrap());
+   |             +++++                    +
 
 error: aborting due to previous error