From e07857533aeb2baaaa9d3bc1a3955141024ea33a Mon Sep 17 00:00:00 2001
From: fvanroie <15969459+fvanroie@users.noreply.github.com>
Date: Tue, 23 Mar 2021 11:33:19 +0100
Subject: [PATCH] Tidy up code

---
 src/hasp/hasp_object.cpp | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/hasp/hasp_object.cpp b/src/hasp/hasp_object.cpp
index 75d628fa..ddd12286 100644
--- a/src/hasp/hasp_object.cpp
+++ b/src/hasp/hasp_object.cpp
@@ -24,6 +24,10 @@
 
 #include "hasplib.h"
 
+#define HASP_NUM_PAGE_PREV (HASP_NUM_PAGES + 1)
+#define HASP_NUM_PAGE_BACK (HASP_NUM_PAGES + 2)
+#define HASP_NUM_PAGE_NEXT (HASP_NUM_PAGES + 3)
+
 const char** btnmatrix_default_map; // memory pointer to lvgl default btnmatrix map
 // static unsigned long last_change_event = 0;
 static bool last_press_was_short = false; // Avoid SHORT + UP double events
@@ -370,14 +374,18 @@ void generic_event_handler(lv_obj_t* obj, lv_event_t event)
     if(obj->user_data.actionid) {
         if(eventid == HASP_EVENT_UP || eventid == HASP_EVENT_SHORT) {
             lv_scr_load_anim_t transitionid = (lv_scr_load_anim_t)obj->user_data.transitionid;
-            if(obj->user_data.actionid == HASP_NUM_PAGES + 1) {
-                haspPages.prev(transitionid);
-            } else if(obj->user_data.actionid == HASP_NUM_PAGES + 2) {
-                haspPages.back(transitionid);
-            } else if(obj->user_data.actionid == HASP_NUM_PAGES + 3) {
-                haspPages.next(transitionid);
-            } else {
-                haspPages.set(obj->user_data.actionid, transitionid);
+            switch(obj->user_data.actionid) {
+                case HASP_NUM_PAGE_PREV:
+                    haspPages.prev(transitionid);
+                    break;
+                case HASP_NUM_PAGE_BACK:
+                    haspPages.back(transitionid);
+                    break;
+                case HASP_NUM_PAGE_NEXT:
+                    haspPages.next(transitionid);
+                    break;
+                default:
+                    haspPages.set(obj->user_data.actionid, transitionid);
             }
             dispatch_output_current_page();
         }