diff options
3 files changed, 11 insertions, 6 deletions
diff --git a/app/src/main/kotlin/com/pitchedapps/frost/activities/MainActivity.kt b/app/src/main/kotlin/com/pitchedapps/frost/activities/MainActivity.kt index 230c1a55..044a1f37 100644 --- a/app/src/main/kotlin/com/pitchedapps/frost/activities/MainActivity.kt +++ b/app/src/main/kotlin/com/pitchedapps/frost/activities/MainActivity.kt @@ -34,7 +34,7 @@ import org.jsoup.Jsoup class MainActivity : BaseMainActivity() { override val fragmentChannel = BroadcastChannel<Int>(10) - override val headerBadgeChannel = Channel<String>(Channel.RENDEZVOUS) + override val headerBadgeChannel = Channel<String>(Channel.CONFLATED) var lastPosition = -1 override fun onNestedCreate(savedInstanceState: Bundle?) { diff --git a/app/src/main/kotlin/com/pitchedapps/frost/views/FrostContentView.kt b/app/src/main/kotlin/com/pitchedapps/frost/views/FrostContentView.kt index 72d8803c..575e5a4d 100644 --- a/app/src/main/kotlin/com/pitchedapps/frost/views/FrostContentView.kt +++ b/app/src/main/kotlin/com/pitchedapps/frost/views/FrostContentView.kt @@ -43,6 +43,7 @@ import com.pitchedapps.frost.utils.Prefs import kotlinx.coroutines.CoroutineScope import kotlinx.coroutines.ExperimentalCoroutinesApi import kotlinx.coroutines.channels.BroadcastChannel +import kotlinx.coroutines.channels.ConflatedBroadcastChannel import kotlinx.coroutines.channels.ReceiveChannel import kotlinx.coroutines.launch @@ -82,9 +83,13 @@ abstract class FrostContentView<out T> @JvmOverloads constructor( override val core: FrostContentCore get() = coreView - override val refreshChannel: BroadcastChannel<Boolean> = BroadcastChannel(100) - override val progressChannel: BroadcastChannel<Int> = BroadcastChannel(100) - override val titleChannel: BroadcastChannel<String> = BroadcastChannel(100) + /** + * While this can be conflated, there exist situations where we wish to watch refresh cycles. + * Here, we'd need to make sure we don't skip events + */ + override val refreshChannel: BroadcastChannel<Boolean> = BroadcastChannel(10) + override val progressChannel: BroadcastChannel<Int> = ConflatedBroadcastChannel() + override val titleChannel: BroadcastChannel<String> = ConflatedBroadcastChannel() override lateinit var scope: CoroutineScope diff --git a/app/src/test/kotlin/com/pitchedapps/frost/utils/CoroutineTest.kt b/app/src/test/kotlin/com/pitchedapps/frost/utils/CoroutineTest.kt index e7520794..fb302648 100644 --- a/app/src/test/kotlin/com/pitchedapps/frost/utils/CoroutineTest.kt +++ b/app/src/test/kotlin/com/pitchedapps/frost/utils/CoroutineTest.kt @@ -150,12 +150,12 @@ class CoroutineTest { /** * Not a true throttle, but for things like fetching header badges, we want to avoid simultaneous fetches. - * As a result, I want to test that the usage of offer along with a rendezvous channel will work as I expect. + * As a result, I want to test that the usage of offer along with a conflated channel will work as I expect. * Events should be consumed when there is no pending consumer on previous elements. */ @Test fun throttledChannel() { - val channel = Channel<Int>(Channel.RENDEZVOUS) + val channel = Channel<Int>(Channel.CONFLATED) runBlocking { val deferred = async { listen(channel) { |