Skip to content

Commit 582d1d8

Browse files
committed
chore(core): apply clippy autofix in rollout tests
Remove .clone() from rollout_items.push(RolloutItem::ResponseItem(###.clone())); in codex.rs since ResponseItem already takes ownership of the item, so cloning is unnecessary. Tests: - just fix -p codex-core
1 parent 06a4333 commit 582d1d8

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

codex-rs/core/src/codex.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9601,7 +9601,7 @@ mod tests {
96019601
std::iter::once(&user1),
96029602
reconstruction_turn.truncation_policy,
96039603
);
9604-
rollout_items.push(RolloutItem::ResponseItem(user1.clone()));
9604+
rollout_items.push(RolloutItem::ResponseItem(user1));
96059605

96069606
let assistant1 = ResponseItem::Message {
96079607
id: None,
@@ -9616,7 +9616,7 @@ mod tests {
96169616
std::iter::once(&assistant1),
96179617
reconstruction_turn.truncation_policy,
96189618
);
9619-
rollout_items.push(RolloutItem::ResponseItem(assistant1.clone()));
9619+
rollout_items.push(RolloutItem::ResponseItem(assistant1));
96209620

96219621
let summary1 = "summary one";
96229622
let snapshot1 = live_history
@@ -9643,7 +9643,7 @@ mod tests {
96439643
std::iter::once(&user2),
96449644
reconstruction_turn.truncation_policy,
96459645
);
9646-
rollout_items.push(RolloutItem::ResponseItem(user2.clone()));
9646+
rollout_items.push(RolloutItem::ResponseItem(user2));
96479647

96489648
let assistant2 = ResponseItem::Message {
96499649
id: None,
@@ -9658,7 +9658,7 @@ mod tests {
96589658
std::iter::once(&assistant2),
96599659
reconstruction_turn.truncation_policy,
96609660
);
9661-
rollout_items.push(RolloutItem::ResponseItem(assistant2.clone()));
9661+
rollout_items.push(RolloutItem::ResponseItem(assistant2));
96629662

96639663
let summary2 = "summary two";
96649664
let snapshot2 = live_history

0 commit comments

Comments
 (0)