clippy fixes

Author: arriqaaq

revision-derive/src/ast/attributes.rs

@@ -128,7 +128,7 @@ impl<O: AttributeOptions> Parse for FilteredAttributes<O> {
 
 			let parsed_options =
 				attr.parse_args_with(Punctuated::<O::Option, Token![,]>::parse_terminated)?;
-			options.extend(parsed_options.into_iter())
+			options.extend(parsed_options)
 		}
 
 		let options = O::finish(Span::call_site(), options)?;

src/implementations/range.rs

@@ -71,6 +71,7 @@ mod tests {
 	}
 
 	#[test]
+	#[allow(clippy::reversed_empty_ranges)]
 	fn test_range_inverted() {
 		let val = 100u32..10u32;
 		let mut mem: Vec<u8> = vec![];
@@ -186,17 +187,15 @@ mod tests {
 		let mut mem: Vec<u8> = vec![];
 		val.serialize_revisioned(&mut mem).unwrap();
 
-		let truncated = &mem[..1];
-		let result =
-			<Range<u32> as DeserializeRevisioned>::deserialize_revisioned(&mut truncated.as_ref());
+		let mut truncated = &mem[..1];
+		let result = <Range<u32> as DeserializeRevisioned>::deserialize_revisioned(&mut truncated);
 		assert!(result.is_err());
 	}
 
 	#[test]
 	fn test_range_empty_reader() {
-		let empty: &[u8] = &[];
-		let result =
-			<Range<u32> as DeserializeRevisioned>::deserialize_revisioned(&mut empty.as_ref());
+		let mut empty: &[u8] = &[];
+		let result = <Range<u32> as DeserializeRevisioned>::deserialize_revisioned(&mut empty);
 		assert!(result.is_err());
 	}
 

src/implementations/system_time.rs

@@ -137,17 +137,15 @@ mod tests {
 		let mut mem: Vec<u8> = vec![];
 		val.serialize_revisioned(&mut mem).unwrap();
 
-		let truncated = &mem[..2];
-		let result =
-			<SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut truncated.as_ref());
+		let mut truncated = &mem[..2];
+		let result = <SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut truncated);
 		assert!(result.is_err());
 	}
 
 	#[test]
 	fn test_system_time_empty_reader() {
-		let empty: &[u8] = &[];
-		let result =
-			<SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut empty.as_ref());
+		let mut empty: &[u8] = &[];
+		let result = <SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut empty);
 		assert!(result.is_err());
 	}
 
@@ -212,9 +210,8 @@ mod tests {
 		let mut mem: Vec<u8> = vec![];
 		val.serialize_revisioned(&mut mem).unwrap();
 
-		let secs_only = &mem[..mem.len() - 1];
-		let result =
-			<SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut secs_only.as_ref());
+		let mut secs_only = &mem[..mem.len() - 1];
+		let result = <SystemTime as DeserializeRevisioned>::deserialize_revisioned(&mut secs_only);
 		assert!(result.is_err());
 	}
 }