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

std: Derive Default for Duration. #37699

Merged
merged 1 commit into from
Nov 12, 2016

Conversation

alexcrichton
Copy link
Member

Discussed in #37546 the libs team reached the conclusion that a default zero
duration seems like a reasonable implementation of the Default trait.

Closes #37546

Discussed in rust-lang#37546 the libs team reached the conclusion that a default zero
duration seems like a reasonable implementation of the `Default` trait.

Closes rust-lang#37546
@rust-highfive
Copy link
Contributor

r? @brson

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

@brson
Copy link
Contributor

brson commented Nov 11, 2016

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 11, 2016

📌 Commit 30502b8 has been approved by brson

@brson brson added the relnotes Marks issues that should be documented in the release notes of the next release. label Nov 11, 2016
eddyb added a commit to eddyb/rust that referenced this pull request Nov 11, 2016
…r=brson

std: Derive `Default` for `Duration`.

Discussed in rust-lang#37546 the libs team reached the conclusion that a default zero
duration seems like a reasonable implementation of the `Default` trait.

Closes rust-lang#37546
eddyb added a commit to eddyb/rust that referenced this pull request Nov 12, 2016
…r=brson

std: Derive `Default` for `Duration`.

Discussed in rust-lang#37546 the libs team reached the conclusion that a default zero
duration seems like a reasonable implementation of the `Default` trait.

Closes rust-lang#37546
bors added a commit that referenced this pull request Nov 12, 2016
@bors bors merged commit 30502b8 into rust-lang:master Nov 12, 2016
@alexcrichton alexcrichton deleted the default-for-duration branch December 19, 2016 20:45
# 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.

4 participants