playback: remove needless seeking

Remove needless seek calls when setMediaItem will do.

app/build.gradle

@@ -101,6 +101,7 @@ dependencies {
     // WARNING: THE EXOPLAYER VERSION MUST BE KEPT IN LOCK-STEP WITH THE PRE-BUILD SCRIPT.
     // IF NOT, VERY UNFRIENDLY BUILD FAILURES AND CRASHES MAY ENSUE.
     implementation "com.google.android.exoplayer:exoplayer-core:2.18.1"
+
     implementation fileTree(dir: "libs", include: ["extension-*.aar"])
 
     // Image loading

app/src/main/java/org/oxycblt/auxio/playback/AnimatedMaterialButton.kt

@@ -25,6 +25,7 @@ import com.google.android.material.button.MaterialButton
 /**
  * A [MaterialButton] that automatically morphs from a circle to a squircle shape appearance when it
  * is activated.
+ * @author OxygenCobalt
  */
 class AnimatedMaterialButton
 @JvmOverloads

app/src/main/java/org/oxycblt/auxio/playback/replaygain/ReplayGainAudioProcessor.kt

@@ -188,14 +188,13 @@ class ReplayGainAudioProcessor(context: Context) : BaseAudioProcessor() {
         }
     }
 
-    private fun parseReplayGainFloat(raw: String): Float {
+    private fun parseReplayGainFloat(raw: String) =
         // Grok a float from a ReplayGain tag by removing everything that is not 0-9, , or -.
-        return try {
+        try {
-            raw.replace(Regex("[^0-9.-]"), "").toFloat()
+            raw.replace(Regex("[^\\d.-]"), "").toFloat()
         } catch (e: Exception) {
             0f
         }
-    }
 
     // --- AUDIO PROCESSOR IMPLEMENTATION ---
 
@@ -244,9 +243,8 @@ class ReplayGainAudioProcessor(context: Context) : BaseAudioProcessor() {
     // little-endian value.
 
     /** Always get a little-endian short value from a [ByteBuffer] */
-    private fun ByteBuffer.getLeShort(at: Int): Short {
+    private fun ByteBuffer.getLeShort(at: Int) =
-        return get(at + 1).toInt().shl(8).or(get(at).toInt().and(0xFF)).toShort()
+        get(at + 1).toInt().shl(8).or(get(at).toInt().and(0xFF)).toShort()
-    }
 
     /** Always place a little-endian short value into a [ByteBuffer]. */
     private fun ByteBuffer.putLeShort(short: Short) {

app/src/main/java/org/oxycblt/auxio/playback/state/PlaybackStateManager.kt

@@ -163,7 +163,6 @@ class PlaybackStateManager private constructor() {
             }
 
         applyNewQueue(library, settings, settings.keepShuffle && isShuffled, song)
-        seekTo(0)
         notifyNewPlayback()
         notifyShuffledChanged()
         isPlaying = true
@@ -176,7 +175,6 @@ class PlaybackStateManager private constructor() {
         val library = musicStore.library ?: return
         this.parent = parent
         applyNewQueue(library, settings, shuffled, null)
-        seekTo(0)
         notifyNewPlayback()
         notifyShuffledChanged()
         isPlaying = true
@@ -189,7 +187,6 @@ class PlaybackStateManager private constructor() {
         val library = musicStore.library ?: return
         parent = null
         applyNewQueue(library, settings, true, null)
-        seekTo(0)
         notifyNewPlayback()
         notifyShuffledChanged()
         isPlaying = true
@@ -229,7 +226,6 @@ class PlaybackStateManager private constructor() {
 
     private fun gotoImpl(idx: Int, play: Boolean) {
         index = idx
-        seekTo(0)
         notifyIndexMoved()
         isPlaying = play
     }

app/src/main/java/org/oxycblt/auxio/playback/system/PlaybackService.kt

@@ -249,9 +249,7 @@ class PlaybackService :
         newPosition: Player.PositionInfo,
         reason: Int
     ) {
-        if (reason == Player.DISCONTINUITY_REASON_SEEK) {
+        playbackManager.synchronizePosition(this, player.currentPosition)
-            playbackManager.synchronizePosition(this, player.currentPosition)
-        }
     }
 
     override fun onTracksChanged(tracks: Tracks) {