Use saturating multiplication for durations

Author: jedisct1

Cargo.toml

@@ -45,3 +45,6 @@ panic = "abort"
 opt-level = 3
 codegen-units = 1
 incremental = false
+
+[profile.dev]
+overflow-checks=true

src/helpers.rs

@@ -1,12 +1,12 @@
 #[inline]
 pub fn _sec_to_u64(sec: u64) -> u64 {
-    sec << 32
+    sec.saturating_mul(1 << 32)
 }
 
 #[inline]
 pub fn _millis_to_u64(millis: u64) -> u64 {
     let secs = millis / 1_000;
-    (secs << 32) | ((millis - secs * 1_000) << 22)
+    secs.saturating_mul(1 << 32) | ((millis - secs * 1_000) << 22)
 }
 
 #[inline]
@@ -17,10 +17,10 @@ pub fn _nsecs_to_u64(nsecs: u64) -> u64 {
 
 #[inline]
 pub fn _timespec_to_u64(tp_sec: u64, tp_nsec: u32) -> u64 {
-    (tp_sec << 32) | ((tp_nsec as u64 * 9_223_372_037) >> 31)
+    tp_sec.saturating_mul(1 << 32) | ((tp_nsec as u64 * 9_223_372_037) >> 31)
 }
 
 #[inline]
 pub fn _timeval_to_u64(tv_sec: u64, tv_usec: u32) -> u64 {
-    (tv_sec << 32) | ((tv_usec as u64 * 9_223_372_036_855) >> 31)
+    tv_sec.saturating_mul(1 << 32) | ((tv_usec as u64 * 9_223_372_036_855) >> 31)
 }

src/tests.rs

@@ -32,6 +32,7 @@ fn tests() {
 }
 
 #[cfg(not(any(target_arch = "wasm32", target_arch = "wasm64")))]
+#[test]
 fn tests_updater() {
     let updater = Updater::new(250)
         .start()
@@ -45,6 +46,11 @@ fn tests_updater() {
     let ts = Instant::recent();
     let clock_recent = Clock::recent_since_epoch();
     sleep(time::Duration::new(1, 0));
-    assert_eq!(Instant::recent(), ts);
     assert_eq!(Clock::recent_since_epoch(), clock_recent);
 }
+
+#[test]
+fn tests_duration() {
+    let duration = Duration::from_days(1000);
+    assert_eq!(duration.as_days(), 1000);
+}