Skip to content

Commit 6e5521b

Browse files
committed
Revert "Allow disabling gapless offload"
This reverts commit 91aad35.
1 parent fde6266 commit 6e5521b

5 files changed

Lines changed: 3 additions & 36 deletions

File tree

app/src/main/java/org/akanework/gramophone/logic/GramophonePlaybackService.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -345,8 +345,7 @@ class GramophonePlaybackService : MediaLibraryService(), MediaSessionService.Lis
345345
this,
346346
GramophoneRenderFactory(
347347
this, rgAp, this::onAudioSinkInputFormatChanged,
348-
afFormatTracker::setAudioSink,
349-
Flags.OFFLOAD && prefs.getStringStrict("offload", "0")?.toIntOrNull() == 3,
348+
afFormatTracker::setAudioSink
350349
)
351350
.setPcmEncodingRestrictionLifted(true)
352351
.setEnableDecoderFallback(true)

app/src/main/java/org/akanework/gramophone/logic/utils/exoplayer/GramophoneRenderFactory.kt

Lines changed: 1 addition & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -3,17 +3,14 @@ package org.akanework.gramophone.logic.utils.exoplayer
33
import android.content.Context
44
import android.os.Handler
55
import android.os.Looper
6-
import androidx.media3.common.AudioAttributes
76
import androidx.media3.common.Format
87
import androidx.media3.common.PlaybackParameters
98
import androidx.media3.common.audio.AudioProcessor
109
import androidx.media3.common.audio.AudioProcessorChain
1110
import androidx.media3.exoplayer.DefaultRenderersFactory
1211
import androidx.media3.exoplayer.Renderer
13-
import androidx.media3.exoplayer.audio.AudioOffloadSupport
1412
import androidx.media3.exoplayer.audio.AudioRendererEventListener
1513
import androidx.media3.exoplayer.audio.AudioSink
16-
import androidx.media3.exoplayer.audio.DefaultAudioOffloadSupportProvider
1714
import androidx.media3.exoplayer.audio.DefaultAudioSink
1815
import androidx.media3.exoplayer.audio.ForwardingAudioSink
1916
import androidx.media3.exoplayer.mediacodec.MediaCodecSelector
@@ -27,8 +24,7 @@ class GramophoneRenderFactory(
2724
context: Context,
2825
private val rgAp: ReplayGainAudioProcessor,
2926
private val configurationListener: (Format?) -> Unit,
30-
private val audioSinkListener: (DefaultAudioSink) -> Unit,
31-
private val disableGaplessOffload: Boolean
27+
private val audioSinkListener: (DefaultAudioSink) -> Unit
3228
) :
3329
DefaultRenderersFactory(context) {
3430
override fun buildTextRenderers(
@@ -124,11 +120,6 @@ class GramophoneRenderFactory(
124120
return 0
125121
}
126122
})
127-
128-
builder.setAudioOffloadSupportProvider(
129-
MyAudioOffloadSupportProvider(DefaultAudioOffloadSupportProvider(context), disableGaplessOffload)
130-
)
131-
132123
var postAmpAudioSink: PostAmpAudioSink? = null
133124
val root = builder.setEnableAudioTrackPlaybackParams(enableAudioTrackPlaybackParams)
134125
.setCanReuse { postAmpAudioSink!!.canReuse() }.build()
@@ -157,22 +148,4 @@ class GramophoneRenderFactory(
157148
configurationListener(null)
158149
}
159150
}
160-
161-
class MyAudioOffloadSupportProvider(
162-
private val default: DefaultAudioOffloadSupportProvider,
163-
private val disableGaplessOffload: Boolean
164-
) : DefaultAudioSink.AudioOffloadSupportProvider by default {
165-
override fun getAudioOffloadSupport(
166-
format: Format,
167-
audioAttributes: AudioAttributes
168-
): AudioOffloadSupport {
169-
val defaultResult = default.getAudioOffloadSupport(format, audioAttributes)
170-
val audioOffloadSupport = AudioOffloadSupport.Builder()
171-
return audioOffloadSupport
172-
.setIsFormatSupported(defaultResult.isFormatSupported)
173-
.setIsGaplessSupported(defaultResult.isGaplessSupported && !disableGaplessOffload)
174-
.setIsSpeedChangeSupported(defaultResult.isSpeedChangeSupported)
175-
.build()
176-
}
177-
}
178151
}

app/src/main/java/org/akanework/gramophone/ui/AudioPreviewActivity.kt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,9 +165,7 @@ class AudioPreviewActivity : BaseActivity(), View.OnClickListener {
165165
this,
166166
GramophoneRenderFactory(
167167
this,
168-
rgAp, {}, {},
169-
Flags.OFFLOAD && prefs.getStringStrict("offload", "0")?.toIntOrNull() == 3
170-
)
168+
rgAp, {}, {})
171169
.setPcmEncodingRestrictionLifted(true)
172170
.setEnableDecoderFallback(true)
173171
.setEnableAudioTrackPlaybackParams(true)

app/src/main/res/values/arrays.xml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,12 @@
3030
<item>@string/disabled</item>
3131
<item>@string/offload_enabled_if_gapless</item>
3232
<item>@string/offload_enabled_if_supported</item>
33-
<item>@string/offload_enabled_no_gapless</item>
3433
</string-array>
3534

3635
<string-array name="offload_switch_val" translatable="false">
3736
<item>0</item>
3837
<item>2</item>
3938
<item>1</item>
40-
<item>3</item>
4139
</string-array>
4240

4341
<string-array name="shuffle_all_queries" translatable="false">

app/src/main/res/values/strings.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -458,7 +458,6 @@
458458
<string name="disabled">Disabled</string>
459459
<string name="offload_enabled_if_supported">Always enabled</string>
460460
<string name="offload_enabled_if_gapless">Enabled if gapless playback possible</string>
461-
<string name="offload_enabled_no_gapless">Always enabled, but without gapless offload</string>
462461
<string name="settings_audio_offload">Audio offload (saves battery)</string>
463462
<string name="settings_telegram">Telegram</string>
464463
<string name="spk_encoding_pcm_20bit">20-bit PCM</string>

0 commit comments

Comments
 (0)