diff options
| author | xbzk <xbzk@eden-emu.dev> | 2025-12-31 17:11:28 +0100 |
|---|---|---|
| committer | crueter <crueter@eden-emu.dev> | 2025-12-31 17:11:28 +0100 |
| commit | 80327312dd981c8ec94588402dfe6ae410d679a5 (patch) | |
| tree | 48f4d04f68ed40dc6254d74be5b1ff6cc666a521 /src | |
| parent | 742622a98df67d00c5d753bd4cd5aa6741dae577 (diff) | |
[android] added barrier for joypad overlay autohide after emulationfragment is destroyed (#3243)
This PR fixes the real culprit for the crash mentioned in #3212 (which has its value coz it fixes other layout's minor navigation issues)
The reason was not the carousel, but the joypad overlay autohide instead.
To reproduce the crash, just enable autohide, launch the game, and leave before overlay gets hidden. eden should crash in few seconds.
The fix was adding barriers to ensure fragment is currently attached to its activity and the binding is not null.
Reviewed-on: https://git.eden-emu.dev/eden-emu/eden/pulls/3243
Reviewed-by: Maufeat <sahyno1996@gmail.com>
Reviewed-by: DraVee <dravee@eden-emu.dev>
Co-authored-by: xbzk <xbzk@eden-emu.dev>
Co-committed-by: xbzk <xbzk@eden-emu.dev>
Diffstat (limited to 'src')
| -rw-r--r-- | src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/EmulationFragment.kt | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/EmulationFragment.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/EmulationFragment.kt index 05134cffb1..d92ca12e6d 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/EmulationFragment.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/fragments/EmulationFragment.kt @@ -1989,7 +1989,7 @@ class EmulationFragment : Fragment(), SurfaceHolder.Callback { handler.removeCallbacksAndMessages(null) handler.postDelayed({ - if (isOverlayVisible) { + if (isOverlayVisible && isAdded && _binding != null) { hideOverlay() } }, seconds * 1000L) |