style: simplify string formatting for readability

hamirmahal · hamirmahal · commit 4bdee80afa3d · 2024-10-06T09:33:58.000-07:00
diff --git a/build.rs b/build.rs
@@ -5,7 +5,7 @@ fn main() {
         Some(true) => {}
         // rustc version too small or can't figure it out
         _ => {
-            eprintln!("'fd' requires rustc >= {}", min_version);
+            eprintln!("'fd' requires rustc >= {min_version}");
             std::process::exit(1);
         }
     }
diff --git a/src/exec/command.rs b/src/exec/command.rs
@@ -103,7 +103,7 @@ pub fn handle_cmd_error(cmd: Option<&Command>, err: io::Error) -> ExitCode {
             ExitCode::GeneralError
         }
         (_, err) => {
-            print_error(format!("Problem while executing command: {}", err));
+            print_error(format!("Problem while executing command: {err}"));
             ExitCode::GeneralError
         }
     }
diff --git a/src/filter/time.rs b/src/filter/time.rs
@@ -149,21 +149,21 @@ mod tests {
         // Timestamp only supported via '@' prefix
         assert!(TimeFilter::before(&ref_time, &ref_timestamp.to_string()).is_none());
         assert!(
-            TimeFilter::before(&ref_time, &format!("@{}", ref_timestamp))
+            TimeFilter::before(&ref_time, &format!("@{ref_timestamp}"))
                 .unwrap()
                 .applies_to(&t1m_ago)
         );
         assert!(
-            !TimeFilter::before(&ref_time, &format!("@{}", ref_timestamp))
+            !TimeFilter::before(&ref_time, &format!("@{ref_timestamp}"))
                 .unwrap()
                 .applies_to(&t1s_later)
         );
         assert!(
-            !TimeFilter::after(&ref_time, &format!("@{}", ref_timestamp))
+            !TimeFilter::after(&ref_time, &format!("@{ref_timestamp}"))
                 .unwrap()
                 .applies_to(&t1m_ago)
         );
-        assert!(TimeFilter::after(&ref_time, &format!("@{}", ref_timestamp))
+        assert!(TimeFilter::after(&ref_time, &format!("@{ref_timestamp}"))
             .unwrap()
             .applies_to(&t1s_later));
     }
diff --git a/src/hyperlink.rs b/src/hyperlink.rs
@@ -35,7 +35,7 @@ fn encode(f: &mut Formatter, byte: u8) -> fmt::Result {
         #[cfg(windows)]
         b'\\' => f.write_char('/'),
         _ => {
-            write!(f, "%{:02X}", byte)
+            write!(f, "%{byte:02X}")
         }
     }
 }
diff --git a/src/main.rs b/src/main.rs
@@ -63,7 +63,7 @@ fn main() {
             exit_code.exit();
         }
         Err(err) => {
-            eprintln!("[fd error]: {:#}", err);
+            eprintln!("[fd error]: {err:#}");
             ExitCode::GeneralError.exit();
         }
     }
diff --git a/src/output.rs b/src/output.rs
@@ -17,7 +17,7 @@ pub fn print_entry<W: Write>(stdout: &mut W, entry: &DirEntry, config: &Config)
     let mut has_hyperlink = false;
     if config.hyperlink {
         if let Some(url) = PathUrl::new(entry.path()) {
-            write!(stdout, "\x1B]8;;{}\x1B\\", url)?;
+            write!(stdout, "\x1B]8;;{url}\x1B\\")?;
             has_hyperlink = true;
         }
     }
@@ -143,7 +143,7 @@ fn print_entry_uncolorized_base<W: Write>(
     if let Some(ref separator) = config.path_separator {
         *path_string.to_mut() = replace_path_separator(&path_string, separator);
     }
-    write!(stdout, "{}", path_string)?;
+    write!(stdout, "{path_string}")?;
     print_trailing_slash(stdout, entry, config, None)
 }
 
diff --git a/src/walk.rs b/src/walk.rs
@@ -252,7 +252,7 @@ impl<'a, W: Write> ReceiverBuffer<'a, W> {
     fn print(&mut self, entry: &DirEntry) -> Result<(), ExitCode> {
         if let Err(e) = output::print_entry(&mut self.stdout, entry, self.config) {
             if e.kind() != ::std::io::ErrorKind::BrokenPipe {
-                print_error(format!("Could not write to output: {}", e));
+                print_error(format!("Could not write to output: {e}"));
                 return Err(ExitCode::GeneralError);
             }
         }
@@ -377,8 +377,7 @@ impl WorkerState {
                         Some(ignore::Error::Partial(_)) => (),
                         Some(err) => {
                             print_error(format!(
-                                "Malformed pattern in global ignore file. {}.",
-                                err
+                                "Malformed pattern in global ignore file. {err}."
                             ));
                         }
                         None => (),
@@ -392,7 +391,7 @@ impl WorkerState {
             match result {
                 Some(ignore::Error::Partial(_)) => (),
                 Some(err) => {
-                    print_error(format!("Malformed pattern in custom ignore file. {}.", err));
+                    print_error(format!("Malformed pattern in custom ignore file. {err}."));
                 }
                 None => (),
             }
diff --git a/tests/testenv/mod.rs b/tests/testenv/mod.rs
@@ -104,9 +104,9 @@ fn format_output_error(args: &[&str], expected: &str, actual: &str) -> String {
     let diff_text = diff::lines(expected, actual)
         .into_iter()
         .map(|diff| match diff {
-            diff::Result::Left(l) => format!("-{}", l),
-            diff::Result::Both(l, _) => format!(" {}", l),
-            diff::Result::Right(r) => format!("+{}", r),
+            diff::Result::Left(l) => format!("-{l}"),
+            diff::Result::Both(l, _) => format!(" {l}"),
+            diff::Result::Right(r) => format!("+{r}"),
         })
         .collect::<Vec<_>>()
         .join("\n");
@@ -290,7 +290,7 @@ impl TestEnv {
     pub fn assert_failure_with_error(&self, args: &[&str], expected: &str) {
         let status = self.assert_error_subdirectory(".", args, Some(expected));
         if status.success() {
-            panic!("error '{}' did not occur.", expected);
+            panic!("error '{expected}' did not occur.");
         }
     }
 
diff --git a/tests/tests.rs b/tests/tests.rs
@@ -597,7 +597,7 @@ fn test_full_path() {
     te.assert_output(
         &[
             "--full-path",
-            &format!("^{prefix}.*three.*foo$", prefix = prefix),
+            &format!("^{prefix}.*three.*foo$"),
         ],
         "one/two/three/d.foo
         one/two/three/directory_foo/",
@@ -1518,7 +1518,7 @@ fn test_symlink_as_absolute_root() {
     let (te, abs_path) = get_test_env_with_abs_path(DEFAULT_DIRS, DEFAULT_FILES);
 
     te.assert_output(
-        &["", &format!("{abs_path}/symlink", abs_path = abs_path)],
+        &["", &format!("{abs_path}/symlink")],
         &format!(
             "{abs_path}/symlink/c.foo
             {abs_path}/symlink/C.Foo2
@@ -1543,7 +1543,7 @@ fn test_symlink_and_full_path() {
         &[
             "--absolute-path",
             "--full-path",
-            &format!("^{prefix}.*three", prefix = prefix),
+            &format!("^{prefix}.*three"),
         ],
         &format!(
             "{abs_path}/{expected_path}/three/
@@ -1563,8 +1563,8 @@ fn test_symlink_and_full_path_abs_path() {
     te.assert_output(
         &[
             "--full-path",
-            &format!("^{prefix}.*symlink.*three", prefix = prefix),
-            &format!("{abs_path}/symlink", abs_path = abs_path),
+            &format!("^{prefix}.*symlink.*three"),
+            &format!("{abs_path}/symlink"),
         ],
         &format!(
             "{abs_path}/symlink/three/
@@ -2337,7 +2337,7 @@ fn test_owner_current_user() {
 fn test_owner_current_group() {
     let te = TestEnv::new(DEFAULT_DIRS, DEFAULT_FILES);
     let gid = Gid::current();
-    te.assert_output(&["--owner", &format!(":{}", gid), "a.foo"], "a.foo");
+    te.assert_output(&["--owner", &format!(":{gid}"), "a.foo"], "a.foo");
     if let Ok(Some(group)) = Group::from_gid(gid) {
         te.assert_output(&["--owner", &format!(":{}", group.name), "a.foo"], "a.foo");
     }
@@ -2616,7 +2616,7 @@ fn test_invalid_cwd() {
         .unwrap();
 
     if !output.status.success() {
-        panic!("{:?}", output);
+        panic!("{output:?}");
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ fn main() {`
`5`	`5`	`Some(true) => {}`
`6`	`6`	`// rustc version too small or can't figure it out`
`7`	`7`	`_ => {`
`8`		`- eprintln!("'fd' requires rustc >= {}", min_version);`
	`8`	`+ eprintln!("'fd' requires rustc >= {min_version}");`
`9`	`9`	`std::process::exit(1);`
`10`	`10`	`}`
`11`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ pub fn handle_cmd_error(cmd: Option<&Command>, err: io::Error) -> ExitCode {`
`103`	`103`	`ExitCode::GeneralError`
`104`	`104`	`}`
`105`	`105`	`(_, err) => {`
`106`		`- print_error(format!("Problem while executing command: {}", err));`
	`106`	`+ print_error(format!("Problem while executing command: {err}"));`
`107`	`107`	`ExitCode::GeneralError`
`108`	`108`	`}`
`109`	`109`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ fn encode(f: &mut Formatter, byte: u8) -> fmt::Result {`
`35`	`35`	`#[cfg(windows)]`
`36`	`36`	`b'\\' => f.write_char('/'),`
`37`	`37`	`_ => {`
`38`		`- write!(f, "%{:02X}", byte)`
	`38`	`+ write!(f, "%{byte:02X}")`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ fn main() {`
`63`	`63`	`exit_code.exit();`
`64`	`64`	`}`
`65`	`65`	`Err(err) => {`
`66`		`- eprintln!("[fd error]: {:#}", err);`
	`66`	`+ eprintln!("[fd error]: {err:#}");`
`67`	`67`	`ExitCode::GeneralError.exit();`
`68`	`68`	`}`
`69`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ pub fn print_entry<W: Write>(stdout: &mut W, entry: &DirEntry, config: &Config)`
`17`	`17`	`let mut has_hyperlink = false;`
`18`	`18`	`if config.hyperlink {`
`19`	`19`	`if let Some(url) = PathUrl::new(entry.path()) {`
`20`		`- write!(stdout, "\x1B]8;;{}\x1B\\", url)?;`
	`20`	`+ write!(stdout, "\x1B]8;;{url}\x1B\\")?;`
`21`	`21`	`has_hyperlink = true;`
`22`	`22`	`}`
`23`	`23`	`}`
`@@ -143,7 +143,7 @@ fn print_entry_uncolorized_base<W: Write>(`
`143`	`143`	`if let Some(ref separator) = config.path_separator {`
`144`	`144`	`*path_string.to_mut() = replace_path_separator(&path_string, separator);`
`145`	`145`	`}`
`146`		`- write!(stdout, "{}", path_string)?;`
	`146`	`+ write!(stdout, "{path_string}")?;`
`147`	`147`	`print_trailing_slash(stdout, entry, config, None)`
`148`	`148`	`}`
`149`	`149`
Original file line number	Diff line number	Diff line change
`@@ -252,7 +252,7 @@ impl<'a, W: Write> ReceiverBuffer<'a, W> {`
`252`	`252`	`fn print(&mut self, entry: &DirEntry) -> Result<(), ExitCode> {`
`253`	`253`	`if let Err(e) = output::print_entry(&mut self.stdout, entry, self.config) {`
`254`	`254`	`if e.kind() != ::std::io::ErrorKind::BrokenPipe {`
`255`		`- print_error(format!("Could not write to output: {}", e));`
	`255`	`+ print_error(format!("Could not write to output: {e}"));`
`256`	`256`	`return Err(ExitCode::GeneralError);`
`257`	`257`	`}`
`258`	`258`	`}`
`@@ -377,8 +377,7 @@ impl WorkerState {`
`377`	`377`	`Some(ignore::Error::Partial(_)) => (),`
`378`	`378`	`Some(err) => {`
`379`	`379`	`print_error(format!(`
`380`		`- "Malformed pattern in global ignore file. {}.",`
`381`		`- err`
	`380`	`+ "Malformed pattern in global ignore file. {err}."`
`382`	`381`	`));`
`383`	`382`	`}`
`384`	`383`	`None => (),`
`@@ -392,7 +391,7 @@ impl WorkerState {`
`392`	`391`	`match result {`
`393`	`392`	`Some(ignore::Error::Partial(_)) => (),`
`394`	`393`	`Some(err) => {`
`395`		`- print_error(format!("Malformed pattern in custom ignore file. {}.", err));`
	`394`	`+ print_error(format!("Malformed pattern in custom ignore file. {err}."));`
`396`	`395`	`}`
`397`	`396`	`None => (),`
`398`	`397`	`}`