Account for changes from #14396

Author: smoelius

clippy_lints/src/strings.rs

@@ -496,21 +496,9 @@ impl<'tcx> LateLintPass<'tcx> for StringToString {
                 if path.ident.name == sym::to_string
                     && let ty = cx.typeck_results().expr_ty(self_arg)
                     && is_type_lang_item(cx, ty.peel_refs(), LangItem::String)
+                    && let Some(parent_span) = is_called_from_map_like(cx, expr)
                 {
-                    if let Some(parent_span) = is_called_from_map_like(cx, expr) {
-                        suggest_cloned_string_to_string(cx, parent_span);
-                    } else {
-                        #[expect(clippy::collapsible_span_lint_calls, reason = "rust-clippy#7797")]
-                        span_lint_and_then(
-                            cx,
-                            STRING_TO_STRING,
-                            expr.span,
-                            "`to_string()` called on a `String`",
-                            |diag| {
-                                diag.help("consider using `.clone()`");
-                            },
-                        );
-                    }
+                    suggest_cloned_string_to_string(cx, parent_span);
                 }
             },
             ExprKind::Path(QPath::TypeRelative(ty, segment)) => {

tests/ui/string_to_string.fixed

@@ -1,8 +1,21 @@
-#![warn(clippy::implicit_clone)]
-#![allow(clippy::redundant_clone)]
+#![warn(clippy::implicit_clone, clippy::string_to_string)]
+#![allow(clippy::redundant_clone, clippy::unnecessary_literal_unwrap)]
 
 fn main() {
     let mut message = String::from("Hello");
     let mut v = message.clone();
     //~^ ERROR: implicitly cloning a `String` by calling `to_string` on its dereferenced type
+
+    let variable1 = String::new();
+    let v = &variable1;
+    let variable2 = Some(v);
+    let _ = variable2.map(|x| {
+        println!();
+        x.clone()
+        //~^ ERROR: implicitly cloning a `String` by calling `to_string` on its dereferenced type
+    });
+
+    let x = Some(String::new());
+    let _ = x.unwrap_or_else(|| v.clone());
+    //~^ ERROR: implicitly cloning a `String` by calling `to_string` on its dereferenced type
 }

tests/ui/string_to_string.rs

@@ -12,10 +12,10 @@ fn main() {
     let _ = variable2.map(|x| {
         println!();
         x.to_string()
+        //~^ ERROR: implicitly cloning a `String` by calling `to_string` on its dereferenced type
     });
-    //~^^ string_to_string
 
     let x = Some(String::new());
     let _ = x.unwrap_or_else(|| v.to_string());
-    //~^ string_to_string
+    //~^ ERROR: implicitly cloning a `String` by calling `to_string` on its dereferenced type
 }

tests/ui/string_to_string.stderr

@@ -7,21 +7,17 @@ LL |     let mut v = message.to_string();
    = note: `-D clippy::implicit-clone` implied by `-D warnings`
    = help: to override `-D warnings` add `#[allow(clippy::implicit_clone)]`
 
-error: `to_string()` called on a `String`
+error: implicitly cloning a `String` by calling `to_string` on its dereferenced type
   --> tests/ui/string_to_string.rs:14:9
    |
 LL |         x.to_string()
-   |         ^^^^^^^^^^^^^
-   |
-   = help: consider using `.clone()`
+   |         ^^^^^^^^^^^^^ help: consider using: `x.clone()`
 
-error: `to_string()` called on a `String`
+error: implicitly cloning a `String` by calling `to_string` on its dereferenced type
   --> tests/ui/string_to_string.rs:19:33
    |
 LL |     let _ = x.unwrap_or_else(|| v.to_string());
-   |                                 ^^^^^^^^^^^^^
-   |
-   = help: consider using `.clone()`
+   |                                 ^^^^^^^^^^^^^ help: consider using: `v.clone()`
 
 error: aborting due to 3 previous errors