From 5f1c1dcd2376c8a9bbef1bbbf96efd81269e5170 Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Fri, 26 Apr 2024 17:28:54 +0200 Subject: [PATCH] No need to have a MutableStateFlow for the live timeline. --- .../features/messages/impl/timeline/TimelineController.kt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelineController.kt b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelineController.kt index bc12524045..1352124454 100644 --- a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelineController.kt +++ b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelineController.kt @@ -35,6 +35,7 @@ import kotlinx.coroutines.flow.SharingStarted import kotlinx.coroutines.flow.StateFlow import kotlinx.coroutines.flow.combine import kotlinx.coroutines.flow.flatMapLatest +import kotlinx.coroutines.flow.flowOf import kotlinx.coroutines.flow.getAndUpdate import kotlinx.coroutines.flow.map import kotlinx.coroutines.flow.stateIn @@ -53,7 +54,7 @@ class TimelineController @Inject constructor( ) : Closeable, TimelineProvider { private val coroutineScope = CoroutineScope(SupervisorJob()) - private val liveTimeline = MutableStateFlow(room.liveTimeline) + private val liveTimeline = flowOf(room.liveTimeline) private val detachedTimeline = MutableStateFlow>(Optional.empty()) @OptIn(ExperimentalCoroutinesApi::class)