chore(ios): remove deprecated lowQualityZeroLatency (#2130)

dcvz · web-flow · commit 264ecc841102 · 2023-09-21T16:13:40.000+02:00
diff --git a/docs/docs/v4-migration.md b/docs/docs/v4-migration.md
@@ -57,6 +57,9 @@ instead of `null`
 
 1. `clearMetadata()` - Instead use [`reset()`](./api/functions/player.md#reset) - which stops playback, clears the queue and clears the notification.
 
+### General Changes
+1. on iOS pitch algorithm defaults to `timeDomain` instead of `lowQualityZeroLatency`. It has been deprecated by Apple and has a few bugs on iOS 17.
+
 ### Typescript Imports
 
 1. If you were using deep imports from RNTP, the `src` has been completely
diff --git a/ios/RNTrackPlayer/Models/Track.swift b/ios/RNTrackPlayer/Models/Track.swift
@@ -113,14 +113,12 @@ class Track: AudioItem, TimePitching, AssetOptionsProviding {
                 return .varispeed
             case PitchAlgorithm.music.rawValue:
                 return .spectral
-            case PitchAlgorithm.voice.rawValue:
+            default: // voice
                 return .timeDomain
-            default:
-                return .lowQualityZeroLatency
             }
         }
 
-        return .lowQualityZeroLatency
+        return .timeDomain
     }
 
     // MARK: - Authorizing Protocol

Original file line number	Diff line number	Diff line change
`@@ -113,14 +113,12 @@ class Track: AudioItem, TimePitching, AssetOptionsProviding {`
`113`	`113`	`return .varispeed`
`114`	`114`	`case PitchAlgorithm.music.rawValue:`
`115`	`115`	`return .spectral`
`116`		`- case PitchAlgorithm.voice.rawValue:`
	`116`	`+ default: // voice`
`117`	`117`	`return .timeDomain`
`118`		`- default:`
`119`		`- return .lowQualityZeroLatency`
`120`	`118`	`}`
`121`	`119`	`}`
`122`	`120`
`123`		`- return .lowQualityZeroLatency`
	`121`	`+ return .timeDomain`
`124`	`122`	`}`
`125`	`123`
`126`	`124`	`// MARK: - Authorizing Protocol`