chore(histogram): Simplify code by using std lib method (#547)

* chore(histogram): Simplify code by using std lib method

Leverage existing standard library method to simplify implementation. No changes to public API or MSRV.

Signed-off-by: Matheus <[email protected]>

* refactor(histogram): Remove `duration_to_seconds` function

Signed-off-by: Matheus <[email protected]>

* tests(histogram): Remove `duration_to_second` test

After removing the aforementioned method in a previous commit, this test became obsolete.

Signed-off-by: Matheus <[email protected]>

---------

Signed-off-by: Matheus <[email protected]>
Co-authored-by: Luca Bruno <[email protected]>

Author: Cardosaum

src/histogram.rs

@@ -542,7 +542,7 @@ impl Instant {
 
     #[inline]
     pub fn elapsed_sec(&self) -> f64 {
-        duration_to_seconds(self.elapsed())
+        self.elapsed().as_secs_f64()
     }
 }
 
@@ -881,13 +881,6 @@ pub fn exponential_buckets(start: f64, factor: f64, count: usize) -> Result<Vec<
     Ok(buckets)
 }
 
-/// `duration_to_seconds` converts Duration to seconds.
-#[inline]
-pub fn duration_to_seconds(d: Duration) -> f64 {
-    let nanos = f64::from(d.subsec_nanos()) / 1e9;
-    d.as_secs() as f64 + nanos
-}
-
 #[derive(Clone, Debug)]
 pub struct LocalHistogramCore {
     histogram: Histogram,
@@ -1363,16 +1356,6 @@ mod tests {
         }
     }
 
-    #[test]
-    fn test_duration_to_seconds() {
-        let tbls = vec![(1000, 1.0), (1100, 1.1), (100_111, 100.111)];
-        for (millis, seconds) in tbls {
-            let d = Duration::from_millis(millis);
-            let v = duration_to_seconds(d);
-            assert!((v - seconds).abs() < f64::EPSILON);
-        }
-    }
-
     #[test]
     fn test_histogram_vec_with_label_values() {
         let vec = HistogramVec::new(