From e76e7780d4f634478fd0573f67ce5a6a66ee684c Mon Sep 17 00:00:00 2001 From: Isira Seneviratne Date: Mon, 28 Aug 2023 06:13:44 +0530 Subject: [PATCH] refactor: Use listOfNotNull --- .../libretube/ui/activities/OfflinePlayerActivity.kt | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt b/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt index 89cec426b..78f5b4b32 100644 --- a/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt +++ b/app/src/main/java/com/github/libretube/ui/activities/OfflinePlayerActivity.kt @@ -168,9 +168,7 @@ class OfflinePlayerActivity : BaseActivity() { videoUri != null && audioUri != null -> { val videoItem = MediaItem.Builder() .setUri(videoUri) - .apply { - if (subtitle != null) setSubtitleConfigurations(listOf(subtitle)) - } + .setSubtitleConfigurations(listOfNotNull(subtitle)) .build() val videoSource = ProgressiveMediaSource.Factory(FileDataSource.Factory()) @@ -193,18 +191,14 @@ class OfflinePlayerActivity : BaseActivity() { videoUri != null -> player.setMediaItem( MediaItem.Builder() .setUri(videoUri) - .apply { - if (subtitle != null) setSubtitleConfigurations(listOf(subtitle)) - } + .setSubtitleConfigurations(listOfNotNull(subtitle)) .build() ) audioUri != null -> player.setMediaItem( MediaItem.Builder() .setUri(audioUri) - .apply { - if (subtitle != null) setSubtitleConfigurations(listOf(subtitle)) - } + .setSubtitleConfigurations(listOfNotNull(subtitle)) .build() ) }