Skip to content

Commit

Permalink
Rollup merge of #79884 - Digital-Chaos:replace-magic, r=m-ou-se
Browse files Browse the repository at this point in the history
Replace magic numbers with existing constants

Replaced magic numbers in `library/core/src/time.rs` with predefined constants.
  • Loading branch information
jonas-schievink authored Jan 24, 2021
2 parents 13b88c2 + c387e29 commit 3ed8a37
Showing 1 changed file with 15 additions and 5 deletions.
20 changes: 15 additions & 5 deletions library/core/src/time.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1067,13 +1067,23 @@ impl fmt::Debug for Duration {
}

if self.secs > 0 {
fmt_decimal(f, self.secs, self.nanos, 100_000_000)?;
fmt_decimal(f, self.secs, self.nanos, NANOS_PER_SEC / 10)?;
f.write_str("s")
} else if self.nanos >= 1_000_000 {
fmt_decimal(f, self.nanos as u64 / 1_000_000, self.nanos % 1_000_000, 100_000)?;
} else if self.nanos >= NANOS_PER_MILLI {
fmt_decimal(
f,
(self.nanos / NANOS_PER_MILLI) as u64,
self.nanos % NANOS_PER_MILLI,
NANOS_PER_MILLI / 10,
)?;
f.write_str("ms")
} else if self.nanos >= 1_000 {
fmt_decimal(f, self.nanos as u64 / 1_000, self.nanos % 1_000, 100)?;
} else if self.nanos >= NANOS_PER_MICRO {
fmt_decimal(
f,
(self.nanos / NANOS_PER_MICRO) as u64,
self.nanos % NANOS_PER_MICRO,
NANOS_PER_MICRO / 10,
)?;
f.write_str("µs")
} else {
fmt_decimal(f, self.nanos as u64, 0, 1)?;
Expand Down

0 comments on commit 3ed8a37

Please sign in to comment.