refactoring

Author: swiftuiux

Sources/swiftui-loop-videoplayer/protocol/helpers/PlayerDelegateProtocol.swift

@@ -19,57 +19,48 @@ public protocol PlayerDelegateProtocol: AnyObject {
     /// display errors accordingly.
     ///
     /// - Parameter error: The specific `VPErrors` instance describing what went wrong.
-    @MainActor
     func didReceiveError(_ error: VPErrors)
 
     /// A method that handles the passage of time in the player.
     /// - Parameter seconds: The amount of time, in seconds, that has passed.
-    @MainActor
     func didPassedTime(seconds: Double)
 
     /// A method that handles seeking in the player.
     /// - Parameters:
     ///   - value: A Boolean indicating whether the seek was successful.
     ///   - currentTime: The current time of the player after seeking, in seconds.
-    @MainActor
     func didSeek(value: Bool, currentTime: Double)
 
     /// Called when the player has paused playback.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.paused`.
-    @MainActor
     func didPausePlayback()
 
     /// Called when the player is waiting to play at the specified rate.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.waitingToPlayAtSpecifiedRate`.
-    @MainActor
     func isWaitingToPlay()
 
     /// Called when the player starts or resumes playing.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.playing`.
-    @MainActor
     func didStartPlaying()
 
     /// Called when the current media item in the player changes.
     ///
     /// This method is triggered when the player's `currentItem` is updated to a new `AVPlayerItem`.
     /// - Parameter newItem: The new `AVPlayerItem` that the player has switched to, if any.
-    @MainActor
     func currentItemDidChange(to newItem: AVPlayerItem?)
 
     /// Called when the current media item is removed from the player.
     ///
     /// This method is triggered when the player's `currentItem` is set to `nil`, indicating that there is no longer an active media item.
-    @MainActor
     func currentItemWasRemoved()
 
     /// Called when the volume level of the player changes.
     ///
     /// This method is triggered when the player's `volume` property changes.
     /// - Parameter newVolume: The new volume level, expressed as a float between 0.0 (muted) and 1.0 (maximum volume).
-    @MainActor
     func volumeDidChange(to newVolume: Float)
 
 }

Sources/swiftui-loop-videoplayer/protocol/player/AbstractPlayer.swift

@@ -233,8 +233,8 @@ extension AbstractPlayer{
         }
 
         player.seek(to: seekTime){ [weak self] value in
-            let currentTime = CMTimeGetSeconds(player.currentTime())
-            self?.delegate?.didSeek(value: value, currentTime: currentTime)
+                let currentTime = CMTimeGetSeconds(player.currentTime())
+                self?.delegate?.didSeek(value: value, currentTime: currentTime)
         }
     }
 

Sources/swiftui-loop-videoplayer/protocol/player/LoopingPlayerProtocol.swift

@@ -125,8 +125,9 @@ internal extension LoopingPlayerProtocol {
         if let timePublishing{
             timeObserver = player.addPeriodicTimeObserver(forInterval: timePublishing, queue: .main) { [weak self] time in
                 guard let self = self else{ return }
-                
-                self.delegate?.didPassedTime(seconds: time.seconds)
+                DispatchQueue.main.async {
+                    self.delegate?.didPassedTime(seconds: time.seconds)
+                }
             }
         }
     }

Sources/swiftui-loop-videoplayer/view/helpers/PlayerCoordinator.swift

@@ -9,7 +9,6 @@ import SwiftUI
 import Combine
 import AVFoundation
 
-@MainActor
 internal class PlayerCoordinator: NSObject, PlayerDelegateProtocol {
 
     let eventPublisher: PassthroughSubject<PlayerEvent, Never>
@@ -77,23 +76,20 @@ internal class PlayerCoordinator: NSObject, PlayerDelegateProtocol {
     /// Called when the player has paused playback.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.paused`.
-    @MainActor
     func didPausePlayback(){
         eventPublisher.send(.paused)
     }
 
     /// Called when the player is waiting to play at the specified rate.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.waitingToPlayAtSpecifiedRate`.
-    @MainActor
     func isWaitingToPlay(){
         eventPublisher.send(.waitingToPlayAtSpecifiedRate)
     }
 
     /// Called when the player starts or resumes playing.
     ///
     /// This method is triggered when the player's `timeControlStatus` changes to `.playing`.
-    @MainActor
     func didStartPlaying(){
         eventPublisher.send(.playing)
     }
@@ -102,15 +98,13 @@ internal class PlayerCoordinator: NSObject, PlayerDelegateProtocol {
     ///
     /// This method is triggered when the player's `currentItem` is updated to a new `AVPlayerItem`.
     /// - Parameter newItem: The new `AVPlayerItem` that the player has switched to, if any.
-    @MainActor
     func currentItemDidChange(to newItem: AVPlayerItem?){
         eventPublisher.send(.currentItemChanged(newItem: newItem))
     }
 
     /// Called when the current media item is removed from the player.
     ///
     /// This method is triggered when the player's `currentItem` is set to `nil`, indicating that there is no longer an active media item.
-    @MainActor
     func currentItemWasRemoved(){
         eventPublisher.send(.currentItemRemoved)
     }
@@ -119,7 +113,6 @@ internal class PlayerCoordinator: NSObject, PlayerDelegateProtocol {
     ///
     /// This method is triggered when the player's `volume` property changes.
     /// - Parameter newVolume: The new volume level, expressed as a float between 0.0 (muted) and 1.0 (maximum volume).
-    @MainActor
     func volumeDidChange(to newVolume: Float){
         eventPublisher.send(.volumeChanged(newVolume: newVolume))
     }