suppress clippy lint inefficient_to_string - according to reviewers…

…' preference

crates/cargo-test-support/src/lib.rs

@@ -106,6 +106,7 @@ dependency.
 */
 
 #![allow(clippy::needless_doctest_main)] // according to @ehuss this lint is fussy
+#![allow(clippy::inefficient_to_string)] // this causes suggestions that result in `(*s).to_string()`
 
 use std::env;
 use std::ffi::OsStr;

src/cargo/lib.rs

@@ -17,6 +17,7 @@
 #![allow(clippy::type_complexity)] // there's an exceptionally complex type
 #![allow(clippy::wrong_self_convention)] // perhaps `Rc` should be special-cased in Clippy?
 #![allow(clippy::write_with_newline)] // too pedantic
+#![allow(clippy::inefficient_to_string)] // this causes suggestions that result in `(*s).to_string()`
 #![warn(clippy::needless_borrow)]
 #![warn(clippy::redundant_clone)]
 // Unit is now interned, and would probably be better as pass-by-copy, but

tests/testsuite/main.rs

@@ -4,6 +4,7 @@
 #![allow(clippy::explicit_iter_loop)]
 #![allow(clippy::redundant_closure)]
 #![allow(clippy::block_in_if_condition_stmt)] // clippy doesn't agree with rustfmt 😂
+#![allow(clippy::inefficient_to_string)] // this causes suggestions that result in `(*s).to_string()`
 #![warn(clippy::needless_borrow)]
 #![warn(clippy::redundant_clone)]