misc: impl TryFrom<&Value> for Time

SeaDve · Feb 14, 2024 · 4675926 · 4675926
1 parent 4732d4a
commit 4675926
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/metadata.rs b/src/metadata.rs
@@ -136,10 +136,7 @@ impl Metadata {
 
     /// The duration of the track.
     pub fn length(&self) -> Option<Time> {
-        self.get_value("mpris:length")?
-            .try_clone()
-            .ok()
-            .and_then(|v| v.downcast().ok())
+        self.get_value("mpris:length")?.downcast_ref().ok()
     }
 
     /// Sets the duration of the track.

diff --git a/src/time.rs b/src/time.rs
@@ -353,6 +353,14 @@ impl TryFrom<Value<'_>> for Time {
     }
 }
 
+impl TryFrom<&Value<'_>> for Time {
+    type Error = Error;
+
+    fn try_from(value: &Value<'_>) -> Result<Self, Self::Error> {
+        i64::try_from(value).map(Self)
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;