Use eventSink method reference in TimelinePresenter.kt (#1428)

Marco Romano · web-flow · commit 7464cbb7691d · 2023-09-25T21:56:41.000+02:00
Has been changed in #1172 but in general method references should always be preferred to lambdas in composable functions (because they have higher stability guarantees).
diff --git a/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelinePresenter.kt b/features/messages/impl/src/main/kotlin/io/element/android/features/messages/impl/timeline/TimelinePresenter.kt
@@ -123,7 +123,7 @@ class TimelinePresenter @Inject constructor(
             paginationState = paginationState,
             timelineItems = timelineItems,
             hasNewItems = hasNewItems.value,
-            eventSink = { handleEvents(it) }
+            eventSink = ::handleEvents
         )
     }
 

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ class TimelinePresenter @Inject constructor(`
`123`	`123`	`paginationState = paginationState,`
`124`	`124`	`timelineItems = timelineItems,`
`125`	`125`	`hasNewItems = hasNewItems.value,`
`126`		`- eventSink = { handleEvents(it) }`
	`126`	`+ eventSink = ::handleEvents`
`127`	`127`	`)`
`128`	`128`	`}`
`129`	`129`