refactor: perform player seek async

godly-devotion · Mar 23, 2024 · a240c8c · a240c8c
1 parent c3978a3
commit a240c8c
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/Front Row/Main Menu/PlaybackCommands.swift b/Front Row/Main Menu/PlaybackCommands.swift
@@ -26,7 +26,7 @@ struct PlaybackCommands: Commands {
                 .disabled(!playEngine.isLoaded)
 
                 Button {
-                    playEngine.goToTime(0.0)
+                    Task { await playEngine.goToTime(0.0) }
                 } label: {
                     Text(
                         "Restart",
@@ -38,7 +38,7 @@ struct PlaybackCommands: Commands {
             }
             Section {
                 Button {
-                    playEngine.goForwards()
+                    Task { await playEngine.goForwards() }
                 } label: {
                     Text("Go Forward 5s")
                 }
@@ -47,7 +47,7 @@ struct PlaybackCommands: Commands {
                     !playEngine.isLoaded || presentedViewManager.isPresenting)
 
                 Button {
-                    playEngine.goBackwards()
+                    Task { await playEngine.goBackwards() }
                 } label: {
                     Text("Go Backward 5s")
                 }

diff --git a/Front Row/Support/PlayEngine.swift b/Front Row/Support/PlayEngine.swift
@@ -143,31 +143,31 @@ import SwiftUI
         }
     }
 
-    func goForwards(_ duration: Double = 5.0) {
+    func goForwards(_ duration: Double = 5.0) async {
         guard isLoaded else { return }
         let time = CMTimeAdd(
             player.currentTime(),
             CMTimeMakeWithSeconds(duration, preferredTimescale: 1)
         )
-        player.seek(to: time, toleranceBefore: CMTime.zero, toleranceAfter: CMTime.zero)
+        await player.seek(to: time, toleranceBefore: .zero, toleranceAfter: .zero)
     }
 
-    func goBackwards(_ duration: Double = 5.0) {
+    func goBackwards(_ duration: Double = 5.0) async {
         guard isLoaded else { return }
         let time = CMTimeSubtract(
             player.currentTime(),
             CMTimeMakeWithSeconds(duration, preferredTimescale: 1)
         )
-        player.seek(to: time, toleranceBefore: CMTime.zero, toleranceAfter: CMTime.zero)
+        await player.seek(to: time, toleranceBefore: .zero, toleranceAfter: .zero)
     }
 
-    func goToTime(_ timecode: Double) {
+    func goToTime(_ timecode: Double) async {
         guard isLoaded else { return }
         let time = CMTimeMakeWithSeconds(timecode, preferredTimescale: 1)
-        player.seek(to: time, toleranceBefore: CMTime.zero, toleranceAfter: CMTime.zero)
+        await player.seek(to: time, toleranceBefore: .zero, toleranceAfter: .zero)
     }
 
-    func goToTime(_ timecode: String) {
+    func goToTime(_ timecode: String) async {
         guard let item = player.currentItem else { return }
 
         let split = Array(timecode.split(separator: ":").reversed())
@@ -186,9 +186,9 @@ import SwiftUI
         let time = CMTimeMakeWithSeconds(
             Double(hour * 3600 + minute * 60) + second, preferredTimescale: 1)
 
-        let validRange = CMTimeRange(start: CMTime.zero, end: item.duration)
+        let validRange = CMTimeRange(start: .zero, end: item.duration)
         guard validRange.containsTime(time) else { return }
-        player.seek(to: time, toleranceBefore: CMTime.zero, toleranceAfter: CMTime.zero)
+        await player.seek(to: time, toleranceBefore: .zero, toleranceAfter: .zero)
     }
 
     func fitToVideoSize() {

diff --git a/Front Row/Views/GoToTimeView.swift b/Front Row/Views/GoToTimeView.swift
@@ -16,7 +16,7 @@ struct GoToTimeView: View {
         VStack {
             TextField(text: $timecode, prompt: Text(verbatim: "0:00:00")) {}
                 .onSubmit {
-                    PlayEngine.shared.goToTime(timecode)
+                    Task { await PlayEngine.shared.goToTime(timecode) }
                     dismiss()
                 }
                 .autocorrectionDisabled()