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.

Sign up for GitHub

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

Jump to bottom

Duration::zero() -> Duration::ZERO #78216

Merged
merged 4 commits into from
Nov 12, 2020
Merged

Duration::zero() -> Duration::ZERO #78216

merged 4 commits into from
Nov 12, 2020

Commits on Oct 22, 2020

  1. Dogfood Duration API in std::time tests 

    This expands time's test suite to use more and in more places the
range of methods and constants added to Duration in recent
proposals for the sake of testing more API surface area and
improving legibility.
    @workingjubilee
    workingjubilee committed Oct 22, 2020
    Configuration menu
    Copy the full SHA
    d72d5f4 View commit details
    Browse the repository at this point in the history

  2. Duration::zero() -> Duration::ZERO 

    Duration::ZERO composes better with match and various other things,
at the cost of an occasional parens, and results in less work for the
optimizer, so let's use that instead.
    @workingjubilee
    workingjubilee committed Oct 22, 2020
    Configuration menu
    Copy the full SHA
    ef027a1 View commit details
    Browse the repository at this point in the history

Commits on Oct 27, 2020

  1. Fixup tests: Duration::MIN -> ::ZERO

    @workingjubilee
    workingjubilee committed Oct 27, 2020
    Configuration menu
    Copy the full SHA
    af4d178 View commit details
    Browse the repository at this point in the history

  2. Remove Duration::MIN entirely 

    Duration::ZERO supercedes it in effect.
    @workingjubilee
    workingjubilee committed Oct 27, 2020
    Configuration menu
    Copy the full SHA
    82f3a23 View commit details
    Browse the repository at this point in the history