From 03f5009c343a073b245e3712707e80bed5853c88 Mon Sep 17 00:00:00 2001 From: Kyle Bloom Date: Tue, 31 Jan 2023 17:29:34 +0000 Subject: [PATCH] fix: Clippy uninlined format args --- benches/read_metadata.rs | 5 +---- examples/extract.rs | 2 +- examples/extract_lorem.rs | 2 +- examples/file_info.rs | 2 +- examples/stdin_info.rs | 4 ++-- examples/write_dir.rs | 10 +++++----- examples/write_sample.rs | 4 ++-- src/compression.rs | 6 +++--- src/result.rs | 6 +++--- src/write.rs | 5 ++--- tests/end_to_end.rs | 2 +- tests/zip64_large.rs | 2 +- 12 files changed, 23 insertions(+), 27 deletions(-) diff --git a/benches/read_metadata.rs b/benches/read_metadata.rs index 51f1f69e..95334b1c 100644 --- a/benches/read_metadata.rs +++ b/benches/read_metadata.rs @@ -17,10 +17,7 @@ fn generate_random_archive(count_files: usize, file_size: usize) -> Vec { let bytes = vec![0u8; file_size]; for i in 0..count_files { - let name = format!( - "file_deadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeef_{}.dat", - i - ); + let name = format!("file_deadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeef_{i}.dat"); writer.start_file(name, options).unwrap(); writer.write_all(&bytes).unwrap(); } diff --git a/examples/extract.rs b/examples/extract.rs index 3519ebaf..3ffb03d7 100644 --- a/examples/extract.rs +++ b/examples/extract.rs @@ -26,7 +26,7 @@ fn real_main() -> i32 { { let comment = file.comment(); if !comment.is_empty() { - println!("File {} comment: {}", i, comment); + println!("File {i} comment: {comment}"); } } diff --git a/examples/extract_lorem.rs b/examples/extract_lorem.rs index 4b3773a0..bc50abe1 100644 --- a/examples/extract_lorem.rs +++ b/examples/extract_lorem.rs @@ -25,7 +25,7 @@ fn real_main() -> i32 { let mut contents = String::new(); file.read_to_string(&mut contents).unwrap(); - println!("{}", contents); + println!("{contents}"); 0 } diff --git a/examples/file_info.rs b/examples/file_info.rs index 0513c3a3..6a2adc58 100644 --- a/examples/file_info.rs +++ b/examples/file_info.rs @@ -30,7 +30,7 @@ fn real_main() -> i32 { { let comment = file.comment(); if !comment.is_empty() { - println!("Entry {} comment: {}", i, comment); + println!("Entry {i} comment: {comment}"); } } diff --git a/examples/stdin_info.rs b/examples/stdin_info.rs index 10d7aa8b..a609916a 100644 --- a/examples/stdin_info.rs +++ b/examples/stdin_info.rs @@ -20,12 +20,12 @@ fn real_main() -> i32 { ); match file.read(&mut buf) { Ok(n) => println!("The first {} bytes are: {:?}", n, &buf[0..n]), - Err(e) => println!("Could not read the file: {:?}", e), + Err(e) => println!("Could not read the file: {e:?}"), }; } Ok(None) => break, Err(e) => { - println!("Error encountered while reading zip: {:?}", e); + println!("Error encountered while reading zip: {e:?}"); return 1; } } diff --git a/examples/write_dir.rs b/examples/write_dir.rs index 20e599c1..3b043528 100644 --- a/examples/write_dir.rs +++ b/examples/write_dir.rs @@ -54,8 +54,8 @@ fn real_main() -> i32 { continue; } match doit(src_dir, dst_file, method.unwrap()) { - Ok(_) => println!("done: {} written to {}", src_dir, dst_file), - Err(e) => println!("Error: {:?}", e), + Ok(_) => println!("done: {src_dir} written to {dst_file}"), + Err(e) => println!("Error: {e:?}"), } } @@ -84,18 +84,18 @@ where // Write file or directory explicitly // Some unzip tools unzip files with directory paths correctly, some do not! if path.is_file() { - println!("adding file {:?} as {:?} ...", path, name); + println!("adding file {path:?} as {name:?} ..."); #[allow(deprecated)] zip.start_file_from_path(name, options)?; let mut f = File::open(path)?; f.read_to_end(&mut buffer)?; - zip.write_all(&*buffer)?; + zip.write_all(&buffer)?; buffer.clear(); } else if !name.as_os_str().is_empty() { // Only if not root! Avoids path spec / warning // and mapname conversion failed error on unzip - println!("adding dir {:?} as {:?} ...", path, name); + println!("adding dir {path:?} as {name:?} ..."); #[allow(deprecated)] zip.add_directory_from_path(name, options)?; } diff --git a/examples/write_sample.rs b/examples/write_sample.rs index 97a5515b..2e45cb1e 100644 --- a/examples/write_sample.rs +++ b/examples/write_sample.rs @@ -14,8 +14,8 @@ fn real_main() -> i32 { let filename = &*args[1]; match doit(filename) { - Ok(_) => println!("File written to {}", filename), - Err(e) => println!("Error: {:?}", e), + Ok(_) => println!("File written to {filename}"), + Err(e) => println!("Error: {e:?}"), } 0 diff --git a/src/compression.rs b/src/compression.rs index abd8b530..baec9399 100644 --- a/src/compression.rs +++ b/src/compression.rs @@ -141,7 +141,7 @@ impl CompressionMethod { impl fmt::Display for CompressionMethod { fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result { // Just duplicate what the Debug format looks like, i.e, the enum key: - write!(f, "{:?}", self) + write!(f, "{self:?}") } } @@ -195,8 +195,8 @@ mod test { #[test] fn to_display_fmt() { fn check_match(method: CompressionMethod) { - let debug_str = format!("{:?}", method); - let display_str = format!("{}", method); + let debug_str = format!("{method:?}"); + let display_str = format!("{method}"); assert_eq!(debug_str, display_str); } diff --git a/src/result.rs b/src/result.rs index 72a30e48..95eb4667 100644 --- a/src/result.rs +++ b/src/result.rs @@ -44,9 +44,9 @@ impl From for ZipError { impl fmt::Display for ZipError { fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result { match self { - ZipError::Io(err) => write!(fmt, "{}", err), - ZipError::InvalidArchive(err) => write!(fmt, "invalid Zip archive: {}", err), - ZipError::UnsupportedArchive(err) => write!(fmt, "unsupported Zip archive: {}", err), + ZipError::Io(err) => write!(fmt, "{err}"), + ZipError::InvalidArchive(err) => write!(fmt, "invalid Zip archive: {err}"), + ZipError::UnsupportedArchive(err) => write!(fmt, "unsupported Zip archive: {err}"), ZipError::FileNotFound => write!(fmt, "specified file not found in archive"), } } diff --git a/src/write.rs b/src/write.rs index e43cc85f..980b1444 100644 --- a/src/write.rs +++ b/src/write.rs @@ -848,7 +848,7 @@ impl Drop for ZipWriter { fn drop(&mut self) { if !self.inner.is_closed() { if let Err(e) = self.finalize() { - let _ = write!(io::stderr(), "ZipWriter drop failed: {:?}", e); + let _ = write!(io::stderr(), "ZipWriter drop failed: {e:?}"); } } } @@ -1211,8 +1211,7 @@ fn validate_extra_data(file: &ZipFileData) -> ZipResult<()> { return Err(ZipError::Io(io::Error::new( io::ErrorKind::Other, format!( - "Extra data header ID {:#06} requires crate feature \"unreserved\"", - kind, + "Extra data header ID {kind:#06} requires crate feature \"unreserved\"", ), ))); } diff --git a/tests/end_to_end.rs b/tests/end_to_end.rs index 25d0c54d..09e7ce47 100644 --- a/tests/end_to_end.rs +++ b/tests/end_to_end.rs @@ -13,7 +13,7 @@ fn end_to_end() { for &method in SUPPORTED_COMPRESSION_METHODS { let file = &mut Cursor::new(Vec::new()); - println!("Writing file with {} compression", method); + println!("Writing file with {method} compression"); write_test_archive(file, method).expect("Couldn't write test zip archive"); println!("Checking file contents"); diff --git a/tests/zip64_large.rs b/tests/zip64_large.rs index 3d10a318..468ef198 100644 --- a/tests/zip64_large.rs +++ b/tests/zip64_large.rs @@ -205,7 +205,7 @@ fn zip64_large() { match file.read_exact(&mut buf) { Ok(()) => println!("The first {} bytes are: {:?}", buf.len(), buf), - Err(e) => println!("Could not read the file: {:?}", e), + Err(e) => println!("Could not read the file: {e:?}"), }; } }