Skip to content

fix: Remove unnecessary clones#911

Merged
seanmonstar merged 1 commit into
hyperium:masterfrom
FranciscoTGouveia:remove-clones
Jun 8, 2026
Merged

fix: Remove unnecessary clones#911
seanmonstar merged 1 commit into
hyperium:masterfrom
FranciscoTGouveia:remove-clones

Conversation

@FranciscoTGouveia

Copy link
Copy Markdown
Contributor

If I am not overlooking anything, these clones can be safely removed since they aren't used anywhere else later.

CC @nunoplopes

@seanmonstar

Copy link
Copy Markdown
Member

Looks like one of the clones was needed after all. 🤓

@FranciscoTGouveia

Copy link
Copy Markdown
Contributor Author

Oh, this was an oversight on my end.
I have only run cargo test --all-features, and pending_send_request_gets_reset_by_peer_properly wasn't triggered.
The PR should be fixed now.

@seanmonstar seanmonstar merged commit c813cc8 into hyperium:master Jun 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants