From 8c6dd39bd7f1dfb4c6d22e1b7ddce19f45ae09df Mon Sep 17 00:00:00 2001 From: anschrammh Date: Fri, 22 Apr 2022 08:03:14 +0200 Subject: [PATCH] Added return value to displayNextView(); and displayPreviousView(); telling if the action was a success or not --- src/app/ScreenManager.cpp | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/app/ScreenManager.cpp b/src/app/ScreenManager.cpp index b3b9ce4..6b6960d 100644 --- a/src/app/ScreenManager.cpp +++ b/src/app/ScreenManager.cpp @@ -295,13 +295,13 @@ boolean ScreenManager::displayView(const uint8_t UID) return _error == OK; } -void ScreenManager::displayNextView() +boolean ScreenManager::displayNextView() { _autoOFFDelayRef = millis(); - if(!_enabled) return; + if(!_enabled) return false; _forceRefresh = true; _error = OK; - if(isListEmpty(_viewLinkedList))return; + if(isListEmpty(_viewLinkedList))return false; if(_currentView == NO_CURRENT_VIEW) { @@ -328,15 +328,16 @@ void ScreenManager::displayNextView() } run(); + return _error == OK; } -void ScreenManager::displayPreviousView() +boolean ScreenManager::displayPreviousView() { _autoOFFDelayRef = millis(); - if(!_enabled) return; + if(!_enabled) return false; _forceRefresh = true; _error = OK; - if(isListEmpty(_tail))return; + if(isListEmpty(_tail))return false; if(_currentView == NO_CURRENT_VIEW) { @@ -363,6 +364,7 @@ void ScreenManager::displayPreviousView() } run(); + return _error == OK; } void ScreenManager::invertDisplayColor(const boolean inverted)