rust: Use inline `format!` variables in a few places
authorColin Walters <walters@verbum.org>
Thu, 9 Jun 2022 21:51:07 +0000 (17:51 -0400)
committerColin Walters <walters@verbum.org>
Thu, 9 Jun 2022 21:51:07 +0000 (17:51 -0400)
Since our MSRV now supports it.

rust-bindings/src/object_name.rs
tests/inst/src/destructive.rs
tests/inst/src/treegen.rs

index 06ff8045fd4126c90c492f957f49cdaf6b12f342..ed4c1bfe6525ebb27bdf18e274f1666ccd687f2b 100644 (file)
@@ -86,7 +86,7 @@ mod tests {
     #[test]
     fn should_stringify_object_name() {
         let object_name = ObjectName::new("abcdef123456", ObjectType::DirTree);
-        let stringified = format!("{}", object_name);
+        let stringified = format!("{object_name}");
         assert_eq!(stringified, "abcdef123456.dirtree");
     }
 
index 352ee6d53e6b0b797833ae5fed1cbd33da750639..6cce4b7a315f7706341043a0b68da9ca167062a2 100644 (file)
@@ -363,7 +363,7 @@ fn impl_transaction_test<M: AsRef<str>>(
             booted: booted_commit.to_string(),
             orig: sysrepo_obj.resolve_rev(ORIGREF, false)?.unwrap().into(),
             prev: srvrepo_obj
-                .resolve_rev(&format!("{}^", TESTREF), false)?
+                .resolve_rev(&format!("{TESTREF}^"), false)?
                 .unwrap()
                 .into(),
             target: srvrepo_obj.resolve_rev(TESTREF, false)?.unwrap().into(),
@@ -568,7 +568,7 @@ pub(crate) fn itest_transactionality() -> Result<()> {
         ..Default::default()
     };
     with_webserver_in(&srvrepo, &webserver_opts, move |addr| {
-        let url = format!("http://{}", addr);
+        let url = format!("http://{addr}");
         bash!(
             "ostree remote delete --if-exists testrepo
              ostree remote add --set=gpg-verify=false testrepo ${url}",
index b6a3a70401ce7f8146ccbff72b182fa4e6331ad6..29bdffd22d04fd06f761b3a313d37ba76a5deddd 100644 (file)
@@ -31,9 +31,9 @@ pub(crate) fn mkvroot<P: AsRef<Path>>(p: P, v: u32) -> Result<()> {
         std::fs::create_dir_all(p.join(v))?;
     }
     let verpath = p.join("etc/.mkrootversion");
-    write_file(&verpath, &format!("{}", v))?;
-    write_file(p.join("usr/bin/somebinary"), &format!("somebinary v{}", v))?;
-    write_file(p.join("etc/someconf"), &format!("someconf v{}", v))?;
+    write_file(&verpath, &format!("{v}"))?;
+    write_file(p.join("usr/bin/somebinary"), &format!("somebinary v{v}"))?;
+    write_file(p.join("etc/someconf"), &format!("someconf v{v}"))?;
     write_file(p.join("usr/bin/vmod2"), &format!("somebinary v{}", v % 2))?;
     write_file(p.join("usr/bin/vmod3"), &format!("somebinary v{}", v % 3))?;
     Ok(())
@@ -134,7 +134,7 @@ pub(crate) fn update_os_tree<P: AsRef<Path>>(
                 tempdir.ensure_dir(v, 0o755)?;
                 let dest = tempdir.sub_dir(v)?;
                 mutated += mutate_executables_to(&src, &dest, percentage)
-                    .with_context(|| format!("Replacing binaries in {}", v))?;
+                    .with_context(|| format!("Replacing binaries in {v}"))?;
             }
         }
     }