From 87a1b797266e3c036ab7fdc1f8ac0c36c46b4057 Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Fri, 17 Oct 2025 11:42:21 +0200 Subject: [PATCH] Perform toImmutableList() less often. --- .../impl/developer/DeveloperSettingsPresenter.kt | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/developer/DeveloperSettingsPresenter.kt b/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/developer/DeveloperSettingsPresenter.kt index 56111fcdca..85aadbb06b 100644 --- a/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/developer/DeveloperSettingsPresenter.kt +++ b/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/developer/DeveloperSettingsPresenter.kt @@ -39,6 +39,7 @@ import io.element.android.libraries.featureflag.api.FeatureFlagService import io.element.android.libraries.featureflag.api.FeatureFlags import io.element.android.libraries.featureflag.ui.model.FeatureUiModel import io.element.android.libraries.preferences.api.store.AppPreferencesStore +import kotlinx.collections.immutable.ImmutableList import kotlinx.collections.immutable.persistentListOf import kotlinx.collections.immutable.toImmutableList import kotlinx.coroutines.CoroutineScope @@ -146,7 +147,7 @@ class DeveloperSettingsPresenter( } return DeveloperSettingsState( - features = featureUiModels.toImmutableList(), + features = featureUiModels, cacheSize = cacheSize.value, clearCacheAction = clearCacheAction.value, rageshakeState = rageshakeState, @@ -165,7 +166,7 @@ class DeveloperSettingsPresenter( @Composable private fun createUiModels( enabledFeatures: SnapshotStateList, - ): List { + ): ImmutableList { return enabledFeatures.map { enabledFeature -> key(enabledFeature.feature.key) { remember(enabledFeature) { @@ -178,7 +179,7 @@ class DeveloperSettingsPresenter( ) } } - } + }.toImmutableList() } private fun CoroutineScope.updateEnabledFeature(