PausingDispatcher should always dispatch when calling yield() [168777346]

Fixed

Bug

Status Update

No update yet.

Description

[Deleted User]

created issue #1

Sep 17, 2020 06:44PM

Component used: lifecycle-runtime-ktx
Version used: 2.2.0-alpha01 up to 2.3.0-alpha07

The following example should print "1", "2" then "3" but prints "1", "3", "2":

lifecycleScope.launchWhenResumed {
  launch {
    println("1")
    yield()
    println("3")
  }
  println("2")
}

A similar problem existed in Dispatchers.Main but was fixed: https://github.com/Kotlin/kotlinx.coroutines/issues/1474

Comments

yb...@google.com <yb...@google.com> #2Sep 21, 2020 04:16PM

Project: platform/frameworks/support
Branch: androidx-main

commit 66494e2d6810347e6a3b41ec3f2665e598f108f0
Author: Ian Lake <ilake@google.com>
Date: Thu Apr 14 00:14:24 2022

Ensure that SavedStateHandle restores upon initial creation

runOnNextRecreation() only runs on the next time
the lifecycle reaches CREATED, not the first time.

By using a LifecycleObserver directly, we ensure that
each Lifecycle move causes a performRestore() to
consistently happen.

Test: existing tests pass, sample app works
BUG: 228865698
Relnote: "Fixed an issue where nesting one `NavHost` within
another `NavHost` in a non-primary bottom navigation tab
would lead to an `IllegalStateException` when using
multiple back stacks."

Change-Id: I11bd5173f035ae1da7922b4142fcedc0f3b54ff6

M lifecycle/lifecycle-viewmodel-savedstate/src/main/java/androidx/lifecycle/SavedStateHandleSupport.kt

https://android-review.googlesource.com/2064518

yb...@google.com <yb...@google.com> #3Sep 21, 2020 04:19PM

Thanks for quick fix! Works well with latest snapshot.

ap...@google.com <ap...@google.com> #4Oct 5, 2020 05:38PM

Marked as fixed.

Thanks for confirming! The fix will be in the next release of Lifecycle (2.5.0-beta01).

Issue 168777346

Description

Issue summary

Comments

yb...@google.com <yb...@google.com> #2Sep 21, 2020 04:16PM

yb...@google.com <yb...@google.com> #3Sep 21, 2020 04:19PM

ap...@google.com <ap...@google.com> #4Oct 5, 2020 05:38PM

Add comment

Issue metadata