Skip to content
New issue

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

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

Already on GitHub? # to your account

implement RFC 1521 #33420

Merged
merged 1 commit into from
May 8, 2016
Merged

implement RFC 1521 #33420

merged 1 commit into from
May 8, 2016

Conversation

durka
Copy link
Contributor

@durka durka commented May 5, 2016

Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.

Fixes #33416.

I tried to use "should" and "must" as defined here.

cc @ubsan

@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@durka durka force-pushed the patch-20 branch 2 times, most recently from 6fc4220 to c576c75 Compare May 5, 2016 02:15
Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.

Fixes rust-lang#33416.
@alexcrichton
Copy link
Member

@bors: r+ c5aa879

Thanks!

@alexcrichton alexcrichton added the relnotes Marks issues that should be documented in the release notes of the next release. label May 5, 2016
Manishearth added a commit to Manishearth/rust that referenced this pull request May 8, 2016
implement RFC 1521

Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.

Fixes rust-lang#33416.

I tried to use "should" and "must" as defined [here](https://tools.ietf.org/html/rfc2119).

cc @ubsan
bors added a commit that referenced this pull request May 8, 2016
Rollup of 8 pull requests

- Successful merges: #33369, #33373, #33404, #33406, #33420, #33424, #33426, #33473
- Failed merges:
Manishearth added a commit to Manishearth/rust that referenced this pull request May 8, 2016
implement RFC 1521

Adds documentation to Clone, specifying that Copy types should have a trivial Clone impl.

Fixes rust-lang#33416.

I tried to use "should" and "must" as defined [here](https://tools.ietf.org/html/rfc2119).

cc @ubsan
bors added a commit that referenced this pull request May 8, 2016
Rollup of 8 pull requests

- Successful merges: #33369, #33373, #33404, #33406, #33420, #33424, #33426, #33473
- Failed merges:
@bors bors merged commit c5aa879 into rust-lang:master May 8, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants