fix: simplify

core/consensus/slots/src/aux_schema.rs

@@ -95,7 +95,7 @@ pub fn check_equivocation<C, H, P>(
 		}
 	}
 
-	// TODO: Having two vectors is super ugly, but I don't know a way around.
+	// TODO: Is there a better way?
 	let mut keys_to_delete = vec![];
 	let mut keys_as_slice_to_delete = vec![];
 
@@ -105,19 +105,13 @@ pub fn check_equivocation<C, H, P>(
 		let first_slot = slot - PRUNING_BOUND;
 		let last_slot = slot - MAX_SLOT_CAPACITY;
 
-		for _ in first_slot..last_slot {
-			keys_to_delete.push(prefix.clone());
-		}
-
-		let mut s = first_slot;
-		for p in keys_to_delete.iter_mut() {
+		for s in first_slot..last_slot {
+			let mut p = prefix.clone();
 			s.using_encoded(|s| p.extend(s));
-			s += 1;
+			keys_to_delete.push(p);
 		}
 
-		for p in keys_to_delete.iter() {
-			keys_as_slice_to_delete.push(p.as_slice());
-		}
+		keys_as_slice_to_delete.extend(keys_to_delete.iter().map(|p| p.as_slice()));
 	}
 
 	v.push((header, signer));