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

Implement Clone for PRNGs #20483

Merged
merged 2 commits into from Jan 6, 2015

Conversation

Projects
None yet
3 participants
@nagisa
Copy link
Contributor

nagisa commented Jan 3, 2015

  • Implement (derive) Clone for ChaChaRng, Isaac*Rng, StdRng and ThreadRng;
  • Derive XorShiftRng Clone implementation instead of implementing it explicitly.

OsRng is the only Rng which does not implement Clone or Copy after this patch because of its dependence on Reader.

r? @huonw I guess?

@nagisa nagisa changed the title Rng copy Implement Clone for PRNGs Jan 3, 2015

@nagisa nagisa force-pushed the nagisa:rng-copy branch from 10a5e0e to 6ca1f0c Jan 5, 2015

@huonw

This comment has been minimized.

Copy link

huonw commented on 6ca1f0c Jan 5, 2015

r+

This comment has been minimized.

Copy link

huonw replied Jan 5, 2015

rollup

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 6, 2015

rollup merge of rust-lang#20483: nagisa/rng-copy
* Implement (derive) `Clone` for `ChaChaRng`, `Isaac*Rng`, `StdRng` and `ThreadRng`;
* Derive `XorShiftRng` `Clone` implementation instead of implementing it explicitly.

`OsRng` is the only Rng which does not implement `Clone` or `Copy` after this patch because of its dependence on `Reader`.

r? @huonw I guess?

@bors bors merged commit 6ca1f0c into rust-lang:master Jan 6, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.