UpsilonNumworks/Upsilon
1
0
Fork 0
mirror of https://github.com/UpsilonNumworks/Upsilon.git synced 2026-01-19 00:37:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

[apps/settings] Fix broken assert

Browse Source 
Pressing OK or EXE with the brightness setting selected would cause a
crash because of a mishandling of the events for this cell.

Change-Id: I4978b4f749b5f19bc1e49ec05b60a32044d86638
This commit is contained in:
Gabriel Ozouf
2020-08-27 11:20:14 +02:00
committed by Émilie Feral
parent d6c7b2b1ac
commit 9be5e7671c
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
apps/settings/main_controller.cpp
View File

@@ -65,7 +65,11 @@ bool MainController::handleEvent(Ion::Events::Event event) {
}
if (event == Ion::Events::OK || event == Ion::Events::EXE || event == Ion::Events::Right) {
assert(rowIndex != k_indexOfBrightnessCell);
if (rowIndex == k_indexOfBrightnessCell) {
/* Nothing is supposed to happen when OK or EXE are pressed on the
* brightness cell. The case of pressing Right has been handled above. */
return true;
}
if (rowIndex == k_indexOfLanguageCell) {
m_localizationController.setMode(LocalizationController::Mode::Language);