From 86f68a63940f35db7878af8bb70deb44354467e3 Mon Sep 17 00:00:00 2001 From: Armin Ronacher Date: Sat, 7 Oct 2023 17:19:27 +0200 Subject: [PATCH] Make clippy happy --- src/text/inline.rs | 4 ++-- src/text/mod.rs | 6 +++--- src/types.rs | 11 +++++++---- src/udiff.rs | 6 +++--- src/utils.rs | 17 +++++++---------- 5 files changed, 22 insertions(+), 22 deletions(-) diff --git a/src/text/inline.rs b/src/text/inline.rs index 79d6c82..c9f0f7f 100644 --- a/src/text/inline.rs +++ b/src/text/inline.rs @@ -310,7 +310,7 @@ fn test_line_ops_inline() { "Hello World\nsome stuff here\nsome more stuff here\n\nAha stuff here\nand more stuff", "Stuff\nHello World\nsome amazing stuff here\nsome more stuff here\n", ); - assert_eq!(diff.newline_terminated(), true); + assert!(diff.newline_terminated()); let changes = diff .ops() .iter() @@ -326,7 +326,7 @@ fn test_serde() { "Hello World\nsome stuff here\nsome more stuff here\n\nAha stuff here\nand more stuff", "Stuff\nHello World\nsome amazing stuff here\nsome more stuff here\n", ); - assert_eq!(diff.newline_terminated(), true); + assert!(diff.newline_terminated()); let changes = diff .ops() .iter() diff --git a/src/text/mod.rs b/src/text/mod.rs index 23408e6..0a441d1 100644 --- a/src/text/mod.rs +++ b/src/text/mod.rs @@ -632,7 +632,7 @@ fn test_unified_diff() { "Hello World\nsome stuff here\nsome more stuff here\n", "Hello World\nsome amazing stuff here\nsome more stuff here\n", ); - assert_eq!(diff.newline_terminated(), true); + assert!(diff.newline_terminated()); insta::assert_snapshot!(&diff .unified_diff() .context_radius(3) @@ -645,7 +645,7 @@ fn test_line_ops() { let a = "Hello World\nsome stuff here\nsome more stuff here\n"; let b = "Hello World\nsome amazing stuff here\nsome more stuff here\n"; let diff = TextDiff::from_lines(a, b); - assert_eq!(diff.newline_terminated(), true); + assert!(diff.newline_terminated()); let changes = diff .ops() .iter() @@ -670,7 +670,7 @@ fn test_line_ops() { #[test] fn test_virtual_newlines() { let diff = TextDiff::from_lines("a\nb", "a\nc\n"); - assert_eq!(diff.newline_terminated(), true); + assert!(diff.newline_terminated()); let changes = diff .ops() .iter() diff --git a/src/types.rs b/src/types.rs index 4efbad4..1a97add 100644 --- a/src/types.rs +++ b/src/types.rs @@ -351,18 +351,18 @@ impl DiffOp { DiffOp::Equal { old_index, len, .. } => { Some((ChangeTag::Equal, &old[old_index..old_index + len])) .into_iter() - .chain(None.into_iter()) + .chain(None) } DiffOp::Insert { new_index, new_len, .. } => Some((ChangeTag::Insert, &new[new_index..new_index + new_len])) .into_iter() - .chain(None.into_iter()), + .chain(None), DiffOp::Delete { old_index, old_len, .. } => Some((ChangeTag::Delete, &old[old_index..old_index + old_len])) .into_iter() - .chain(None.into_iter()), + .chain(None), DiffOp::Replace { old_index, old_len, @@ -370,7 +370,10 @@ impl DiffOp { new_len, } => Some((ChangeTag::Delete, &old[old_index..old_index + old_len])) .into_iter() - .chain(Some((ChangeTag::Insert, &new[new_index..new_index + new_len])).into_iter()), + .chain(Some(( + ChangeTag::Insert, + &new[new_index..new_index + new_len], + ))), } } diff --git a/src/udiff.rs b/src/udiff.rs index 1bbf60b..26701da 100644 --- a/src/udiff.rs +++ b/src/udiff.rs @@ -317,10 +317,10 @@ where /// /// `n` configures [`UnifiedDiff::context_radius`] and /// `header` configures [`UnifiedDiff::header`] when not `None`. -pub fn unified_diff<'old, 'new>( +pub fn unified_diff( alg: Algorithm, - old: &'old str, - new: &'new str, + old: &str, + new: &str, n: usize, header: Option<(&str, &str)>, ) -> String { diff --git a/src/utils.rs b/src/utils.rs index 44776fe..c95f453 100644 --- a/src/utils.rs +++ b/src/utils.rs @@ -155,7 +155,7 @@ impl<'x, T: DiffableStr + ?Sized> TextDiffRemapper<'x, T> { DiffOp::Equal { old_index, len, .. } => { Some((ChangeTag::Equal, self.old.slice(old_index..old_index + len))) .into_iter() - .chain(None.into_iter()) + .chain(None) } DiffOp::Insert { new_index, new_len, .. @@ -164,7 +164,7 @@ impl<'x, T: DiffableStr + ?Sized> TextDiffRemapper<'x, T> { self.new.slice(new_index..new_index + new_len), )) .into_iter() - .chain(None.into_iter()), + .chain(None), DiffOp::Delete { old_index, old_len, .. } => Some(( @@ -172,7 +172,7 @@ impl<'x, T: DiffableStr + ?Sized> TextDiffRemapper<'x, T> { self.old.slice(old_index..old_index + old_len), )) .into_iter() - .chain(None.into_iter()), + .chain(None), DiffOp::Replace { old_index, old_len, @@ -183,13 +183,10 @@ impl<'x, T: DiffableStr + ?Sized> TextDiffRemapper<'x, T> { self.old.slice(old_index..old_index + old_len), )) .into_iter() - .chain( - Some(( - ChangeTag::Insert, - self.new.slice(new_index..new_index + new_len), - )) - .into_iter(), - ), + .chain(Some(( + ChangeTag::Insert, + self.new.slice(new_index..new_index + new_len), + ))), } .map(|(tag, opt_val)| (tag, opt_val.expect("slice out of bounds"))) }