diff options
| author | t895 <clombardo169@gmail.com> | 2024-02-21 08:17:30 -0500 | 
|---|---|---|
| committer | t895 <clombardo169@gmail.com> | 2024-02-21 08:17:30 -0500 | 
| commit | 45f450fca5c7fabbb30e0d193d1f4a1be6bd287b (patch) | |
| tree | a1c12ef885a07cc0a405e9fd655810d2d8760857 | |
| parent | 9a3fd76b25f0db3804789891a10c51568121bf0c (diff) | |
android: Add additional check for hasMapping
Controls can have no mapping if they are either "[empty]" or and empty string. This was causing an issue if you reset mapping on all controllers and then tried to play a game. The check to determine whether auto mapping was required would fail and leave you will no mapped controllers. This feels a bit like user error but it smooths things out if you forget so I see it as necessary.
| -rw-r--r-- | src/android/app/src/main/java/org/yuzu/yuzu_emu/features/input/model/PlayerInput.kt | 6 | 
1 files changed, 3 insertions, 3 deletions
diff --git a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/input/model/PlayerInput.kt b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/input/model/PlayerInput.kt index d35de80c4..a84ac77a2 100644 --- a/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/input/model/PlayerInput.kt +++ b/src/android/app/src/main/java/org/yuzu/yuzu_emu/features/input/model/PlayerInput.kt @@ -64,17 +64,17 @@ data class PlayerInput(      fun hasMapping(): Boolean {          var hasMapping = false          buttons.forEach { -            if (it != "[empty]") { +            if (it != "[empty]" && it.isNotEmpty()) {                  hasMapping = true              }          }          analogs.forEach { -            if (it != "[empty]") { +            if (it != "[empty]" && it.isNotEmpty()) {                  hasMapping = true              }          }          motions.forEach { -            if (it != "[empty]") { +            if (it != "[empty]" && it.isNotEmpty()) {                  hasMapping = true              }          }  | 
