Auto merge of #28152 - llogiq:master, r=Manishearth

This changes a lot of `.to_string()` to `.to_owned()`, removes a few redundant closures, and changes some `match`es to `if let`s.

I'm currently in the process of trying out clippy and acting on its suggestions. I started with compiletest, because we use it to test clippy, too. If this finds positive reception, I may continue refactoring other parts of the rust codebase.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/compiletest/errors.rs b/src/compiletest/errors.rs
index 4b2a3e0283d..a3ad022ebd5 100644
--- a/src/compiletest/errors.rs
+++ b/src/compiletest/errors.rs
@@ -76,7 +76,7 @@ fn parse_expected(last_nonfollow_error: Option<usize>,
     let letters = line[kind_start..].chars();
     let msg = letters.skip_while(|c| c.is_whitespace())
                      .skip_while(|c| !c.is_whitespace())
-                     .collect::<String>().trim().to_string();
+                     .collect::<String>().trim().to_owned();
 
     let (which, line) = if follow {
         assert!(adjusts == 0, "use either //~| or //~^, not both.");