Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add derive(Clone) where possible #382

merged 1 commit into from Jan 1, 2020


Copy link

@andersk andersk commented Nov 10, 2019

No description provided.

Signed-off-by: Anders Kaseorg <>
Copy link

Would be great to have this merged, this PR is quite necessary when you have an iterator chain that eventually needs to be cloned (e.g. for .cycle()) but some itertools combinator in the chain fails to be Clone...

Copy link

Looks good to me. Maybe we can simplify the manual impls after merging this (e.g. via a macro similar to debug_fmt_fields).

Copy link

jswrenn commented Jan 1, 2020


bors r+

@jswrenn jswrenn added this to the next milestone Jan 1, 2020
bors bot added a commit that referenced this pull request Jan 1, 2020
382: Add derive(Clone) where possible r=jswrenn a=andersk

Co-authored-by: Anders Kaseorg <>
Copy link

bors bot commented Jan 1, 2020

Build succeeded

And happy new year from bors! 🎉

@bors bors bot merged commit 1e97bf2 into rust-itertools:master Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants