Skip to content

Add Clone to std::time::Duration #16519

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

Closed
wants to merge 1 commit into from
Closed

Conversation

apoelstra
Copy link
Contributor

This is needed to derive Clone on structures containing durations.

@japaric
Copy link
Member

japaric commented Aug 15, 2014

Duration already implements Show. So, it shouldn't need the #[deriving(Show)], rigth?

@apoelstra
Copy link
Contributor Author

Ugh, yes. Sorry, that's what I get for using the github editor to make PRs..

@apoelstra apoelstra changed the title Add Clone and Show to std::time::Duration Add Cloneto std::time::Duration Aug 15, 2014
@apoelstra apoelstra changed the title Add Cloneto std::time::Duration Add Clone to std::time::Duration Aug 15, 2014
This is needed to derive Clone for types containing Durations.
bors added a commit that referenced this pull request Aug 16, 2014
This is needed to derive Clone on structures containing durations.
@bors bors closed this Aug 16, 2014
matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this pull request Feb 11, 2024
Enable some minor lints that we should tackles (take2)

This enables these lints:

- borrowed_box
- derived_hash_with_manual_eq
- forget_non_drop
- needless_doctest_main
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.

4 participants