From 7c7a8602c473bc160506e064097bd9ecc204425b Mon Sep 17 00:00:00 2001
From: Riku Isokoski <riksu9000@gmail.com>
Date: Wed, 22 Feb 2023 22:05:37 +0200
Subject: screens: Remove displayapp parameter from screen

The DisplayApp class isn't used in the Screen base class and most
screens, so requiring it is pointless.

In this commit, DisplayApp pointers were added to screens which use it
and the explicit Screen constructor was removed in those screens.
---
 src/displayapp/screens/ScreenList.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/displayapp/screens/ScreenList.h')

diff --git a/src/displayapp/screens/ScreenList.h b/src/displayapp/screens/ScreenList.h
index 255544b6..78f32314 100644
--- a/src/displayapp/screens/ScreenList.h
+++ b/src/displayapp/screens/ScreenList.h
@@ -19,7 +19,7 @@ namespace Pinetime {
                    uint8_t initScreen,
                    const std::array<std::function<std::unique_ptr<Screen>()>, N>&& screens,
                    ScreenListModes mode)
-          : Screen(app),
+          : app {app},
             initScreen {initScreen},
             screens {std::move(screens)},
             mode {mode},
@@ -102,6 +102,7 @@ namespace Pinetime {
         }
 
       private:
+        DisplayApp* app;
         uint8_t initScreen = 0;
         const std::array<std::function<std::unique_ptr<Screen>()>, N> screens;
         ScreenListModes mode = ScreenListModes::UpDown;
-- 
cgit v1.2.3-70-g09d2