From 6854f014b0445ee085f8405fec2a618262412be3 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Fri, 17 Oct 2025 10:52:41 +0200
Subject: [PATCH] Fix warning and also fix bug!

---
 .../libraries/featureflag/impl/DefaultFeatureFlagService.kt | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/libraries/featureflag/impl/src/main/kotlin/io/element/android/libraries/featureflag/impl/DefaultFeatureFlagService.kt b/libraries/featureflag/impl/src/main/kotlin/io/element/android/libraries/featureflag/impl/DefaultFeatureFlagService.kt
index 01c1e8a258..a7ff8c09dd 100644
--- a/libraries/featureflag/impl/src/main/kotlin/io/element/android/libraries/featureflag/impl/DefaultFeatureFlagService.kt
+++ b/libraries/featureflag/impl/src/main/kotlin/io/element/android/libraries/featureflag/impl/DefaultFeatureFlagService.kt
@@ -27,16 +27,14 @@ class DefaultFeatureFlagService(
 ) : FeatureFlagService {
     override fun isFeatureEnabledFlow(feature: Feature): Flow<Boolean> {
         return providers.filter { it.hasFeature(feature) }
-            .sortedByDescending(FeatureFlagProvider::priority)
-            .firstOrNull()
+            .maxByOrNull(FeatureFlagProvider::priority)
             ?.isFeatureEnabledFlow(feature)
             ?: flowOf(feature.defaultValue(buildMeta))
     }
 
     override suspend fun setFeatureEnabled(feature: Feature, enabled: Boolean): Boolean {
         return providers.filterIsInstance<MutableFeatureFlagProvider>()
-            .sortedBy(FeatureFlagProvider::priority)
-            .firstOrNull()
+            .maxByOrNull(FeatureFlagProvider::priority)
             ?.setFeatureEnabled(feature, enabled)
             ?.let { true }
             ?: false