Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "trybuild"
-version = "1.0.91"
+version = "1.0.92"
 authors = ["David Tolnay <dtolnay@gmail.com>"]
 categories = ["development-tools::testing"]
 description = "Test harness for ui tests of compiler diagnostics"

README.md

@@ -75,7 +75,7 @@ To test just one source file, use:
 ```
 cargo test -- ui trybuild=example.rs
 ```
-where `ui` is the name of the `#[test]` funtion that invokes `trybuild`, and
+where `ui` is the name of the `#[test]` function that invokes `trybuild`, and
 `example.rs` is the name of the file to test.
 
 <br>

src/env.rs

@@ -1,23 +1,17 @@
 use crate::error::{Error, Result};
 use std::env;
 
-#[derive(PartialEq, Debug)]
+#[derive(PartialEq, Default, Debug)]
 pub(crate) enum Update {
+    #[default]
     Wip,
     Overwrite,
 }
 
-impl Default for Update {
-    fn default() -> Self {
-        Update::Wip
-    }
-}
-
 impl Update {
     pub fn env() -> Result<Self> {
-        let var = match env::var_os("TRYBUILD") {
-            Some(var) => var,
-            None => return Ok(Update::default()),
+        let Some(var) = env::var_os("TRYBUILD") else {
+            return Ok(Update::default());
         };
 
         match var.as_os_str().to_str() {

src/flock.rs

@@ -115,9 +115,8 @@ fn create(path: &Path) -> Option<File> {
             },
         };
 
-        let modified = match metadata.modified() {
-            Ok(modified) => modified,
-            Err(_) => return None,
+        let Ok(modified) = metadata.modified() else {
+            return None;
         };
 
         let now = SystemTime::now();

src/lib.rs

@@ -208,7 +208,7 @@
 //! makes error messages that we care about substantially worse, it is also
 //! important to catch and report as a compiler issue.
 
-#![doc(html_root_url = "https://docs.rs/trybuild/1.0.91")]
+#![doc(html_root_url = "https://docs.rs/trybuild/1.0.92")]
 #![allow(
     clippy::collapsible_if,
     clippy::default_trait_access,
@@ -231,6 +231,7 @@
     clippy::too_many_lines,
     clippy::trivially_copy_pass_by_ref,
     clippy::uninhabited_references,
+    clippy::uninlined_format_args,
     clippy::unused_self,
     clippy::while_let_on_iterator,
 )]

src/manifest.rs

@@ -38,8 +38,9 @@ pub(crate) struct Package {
     pub publish: bool,
 }
 
-#[derive(Serialize, Deserialize, Debug)]
+#[derive(Serialize, Deserialize, Default, Debug)]
 pub(crate) enum Edition {
+    #[default]
     #[serde(rename = "2015")]
     E2015,
     #[serde(rename = "2018")]
@@ -75,12 +76,6 @@ pub(crate) struct Workspace {
     pub dependencies: Map<String, Dependency>,
 }
 
-impl Default for Edition {
-    fn default() -> Self {
-        Edition::E2018
-    }
-}
-
 impl AsRef<OsStr> for Name {
     fn as_ref(&self) -> &OsStr {
         self.0.as_ref()

src/normalize.rs

@@ -401,7 +401,9 @@ impl<'a> Filter<'a> {
             let trimmed_line = trimmed_line
                 .strip_prefix("= note: ")
                 .unwrap_or(trimmed_line);
-            if trimmed_line.starts_with("the full type name has been written to") {
+            if trimmed_line.starts_with("the full type name has been written to")
+                || trimmed_line.starts_with("the full name for the type has been written to")
+            {
                 return None;
             }
         }
@@ -516,9 +518,8 @@ fn unindent(diag: String, normalization: Normalization) -> String {
         }
 
         let mut ahead = lines.clone();
-        let next_line = match ahead.next() {
-            Some(line) => line,
-            None => continue,
+        let Some(next_line) = ahead.next() else {
+            continue;
         };
 
         if let IndentedLineKind::Code(indent) = indented_line_kind(next_line, normalization) {

src/run.rs

@@ -264,9 +264,8 @@ impl Runner {
         let mut features = source_manifest.features;
         for (feature, enables) in &mut features {
             enables.retain(|en| {
-                let dep_name = match en.strip_prefix("dep:") {
-                    Some(dep_name) => dep_name,
-                    None => return false,
+                let Some(dep_name) = en.strip_prefix("dep:") else {
+                    return false;
                 };
                 if let Some(Dependency { optional: true, .. }) = dependencies.get(dep_name) {
                     return true;
@@ -622,9 +621,8 @@ fn parse_cargo_json(
     let mut remaining = &*String::from_utf8_lossy(stdout);
     let mut seen = Set::new();
     while !remaining.is_empty() {
-        let begin = match remaining.find("{\"reason\":") {
-            Some(begin) => begin,
-            None => break,
+        let Some(begin) = remaining.find("{\"reason\":") else {
+            break;
         };
         let (nonmessage, rest) = remaining.split_at(begin);
         nonmessage_stdout.push_str(nonmessage);
@@ -643,9 +641,8 @@ fn parse_cargo_json(
         }
         if let Ok(de) = serde_json::from_str::<CargoMessage>(message) {
             if de.message.level != "failure-note" {
-                let (name, test) = match path_map.get(&de.target.src_path) {
-                    Some(test) => test,
-                    None => continue,
+                let Some((name, test)) = path_map.get(&de.target.src_path) else {
+                    continue;
                 };
                 let entry = map
                     .entry(de.target.src_path)