initial menu rework part 1

Author: Yanis002

include/common.hpp

@@ -0,0 +1,5 @@
+#pragma once
+
+#include <types.h>
+
+typedef void (*GZAction)(u32 params);

include/gz_commands.hpp

@@ -1,25 +1,27 @@
 #pragma once
 
+#include "common.hpp"
 #include "gz_controls.hpp"
 #include "gz_menu.hpp"
 
 #include <nitro/math.h>
 
 typedef void (*GZCmdInit)(void);
-typedef void (*GZCmdAction)(void);
 
 struct GZCmdItem {
     ButtonCombo btnCombo;
-    GZCmdAction actionCallback;
+    GZAction actionCallback;
 };
 
 class GZCommandManager {
   public:
     Input* mpButtons;
     GZCmdItem* mpCommands;
+    GZMenu mMenu;
 
     GZCommandManager();
 
+    void CreateMenuItems();
     void Update();
     void Draw(Vec2b* pPos);
 };

include/gz_menu.hpp

@@ -1,5 +1,6 @@
 #pragma once
 
+#include "common.hpp"
 #include "gz_controls.hpp"
 
 #include <Item/ItemManager.hpp>
@@ -8,7 +9,8 @@
 #include <nitro/math.h>
 #include <types.h>
 
-typedef void (*GZMenuAction)(u32 params);
+#define DRAW_TO_TOP_SCREEN 1
+
 struct GZMenu;
 
 typedef enum InventoryAmountType {
@@ -24,18 +26,18 @@ typedef enum InventoryAmountType {
 } InventoryAmountType;
 
 struct GZMenuItem {
-    const char* mName; // menu item name
-    GZMenuAction mActionCallback; // associated action
-    u32 params; // parameters for the action callback
-    GZMenu* mSubMenu; // tied submenu
-    bool needSaveFile; // does it require the save data
-    s32 value; // misc value for internal use
+    const char* name;
+    GZAction action;
+    u32 params;
+    GZMenu* submenu;
+    int value;
 };
 
 struct GZMenu {
-    GZMenuItem* mpItems;
+    GZMenuItem* entries;
     s32 mCount;
-    GZMenu* mPrev;
+    bool needSaveFile;
+    s16 itemIndex;
 };
 
 struct GZMenuState {
@@ -78,6 +80,7 @@ class GZMenuManager {
   public:
     GZMenuState mState;
     GZMenu* mpActiveMenu;
+    GZMenu* mpPrevMenu;
     GZMenuControls mControls;
     Input* mpButtons;
 
@@ -90,7 +93,7 @@ class GZMenuManager {
         this->mState.isOpened = false;
     }
 
-    GZMenuItem* GetActiveMenuItem() { return &this->mpActiveMenu->mpItems[this->mState.itemIndex]; }
+    GZMenuItem* GetActiveMenuItem() { return &this->mpActiveMenu->entries[this->mState.itemIndex]; }
 
     bool IsInventoryMenuActive();
     bool IsAmountsMenuActive();

include/gz_settings.hpp

@@ -16,16 +16,13 @@ extern "C" void func_02030d58(u16 lockID); // CARD_UnlockBackup
 #define MAX_POS_SLOTS 8
 #define MAX_SAVE_PROFILES 8
 
-//! TODO: make it one setting set per save file
-
 extern "C" {
 typedef struct GZProfileHeader {
     u8 curProfileIndex;
 } GZProfileHeader __attribute__((aligned(32)));
 
 // the save file is large so we don't care about packing stuff to save space
 typedef struct GZProfile {
-    u8 profileIndex;
     bool mFasterTitleScreen; // goes into the "touch or start to exit" state immediately
     bool mSkipTitleScreen; // same as above except it jump straight into the file select without requiring any input
     s8 mPositionIndex;

src/gz_commands.cpp

@@ -12,17 +12,17 @@
 extern "C" void DisplayDebugText(int, void*, int, int, const char*);
 extern "C" void DisplayDebugTextF(int, void*, int, int, const char*, ...);
 
-static void ExecuteLevitate();
-static void ExecutePause();
-static void ExecuteFrameAdvance();
-static void ExecutePrevPosition();
-static void ExecuteNextPosition();
-static void ExecuteSavePosition();
-static void ExecuteLoadPosition();
-static void ExecuteVoidOut();
-static void ExecuteTurbo();
-
-// commands with default button combos
+static void ExecuteLevitate(u32 params);
+static void ExecutePause(u32 params);
+static void ExecuteFrameAdvance(u32 params);
+static void ExecutePrevPosition(u32 params);
+static void ExecuteNextPosition(u32 params);
+static void ExecuteSavePosition(u32 params);
+static void ExecuteLoadPosition(u32 params);
+static void ExecuteVoidOut(u32 params);
+static void ExecuteTurbo(u32 params);
+
+// commands with default button combos, assigning the held btn to none means it's not required (and vice versa)
 static GZCmdItem sCommands[] = {
     {ButtonCombo("Levitate", BTN_X, BTN_NONE), ExecuteLevitate},
     {ButtonCombo("Pause/Unpause", BTN_NONE, BTN_DDOWN), ExecutePause},
@@ -35,19 +35,19 @@ static GZCmdItem sCommands[] = {
     {ButtonCombo("Turbo", BTN_R, BTN_B), ExecuteTurbo},
 };
 
-static void ExecuteLevitate() { data_027e0478.mPlayer.mVel.y = FLOAT_TO_Q20(0.334375f); }
+static void ExecuteLevitate(u32 params) { data_027e0478.mPlayer.mVel.y = FLOAT_TO_Q20(0.334375f); }
 
-static void ExecutePause() {
+static void ExecutePause(u32 params) {
     if (!gGZ.mState.isPaused) {
         gGZ.mState.isPaused = true;
     } else {
         gGZ.mState.isPaused = false;
     }
 }
 
-static void ExecuteFrameAdvance() { gGZ.mState.requestedFrames++; }
+static void ExecuteFrameAdvance(u32 params) { gGZ.mState.requestedFrames++; }
 
-static void ExecutePrevPosition() {
+static void ExecutePrevPosition(u32 params) {
     GZProfile* pProfile = gSettings.GetProfile();
 
     pProfile->mPositionIndex--;
@@ -57,7 +57,7 @@ static void ExecutePrevPosition() {
     }
 }
 
-static void ExecuteNextPosition() {
+static void ExecuteNextPosition(u32 params) {
     GZProfile* pProfile = gSettings.GetProfile();
 
     pProfile->mPositionIndex++;
@@ -67,31 +67,42 @@ static void ExecuteNextPosition() {
     }
 }
 
-static void ExecuteSavePosition() {
+static void ExecuteSavePosition(u32 params) {
     GZProfile* pProfile = gSettings.GetProfile();
     Vec3p* pPosArray = gSettings.GetPosArray();
     pPosArray[pProfile->mPositionIndex] = data_027e0478.mPlayer.mPos;
 }
 
-static void ExecuteLoadPosition() {
+static void ExecuteLoadPosition(u32 params) {
     GZProfile* pProfile = gSettings.GetProfile();
     Vec3p* pPosArray = gSettings.GetPosArray();
     data_027e0478.mPlayer.mPos = pPosArray[pProfile->mPositionIndex];
 }
 
-static void ExecuteVoidOut() {
+static void ExecuteVoidOut(u32 params) {
     if (gGZ.IsOnLand()) {
         data_027e0478.mPlayer.mPos.y = FLOAT_TO_Q20(-4000.0f);
     }
 }
 
-static void ExecuteTurbo() {}
+static void ExecuteTurbo(u32 params) {}
 
 GZCommandManager gCommandManager;
 
 GZCommandManager::GZCommandManager() {
     this->mpButtons = &gGZ.mButtons;
     this->mpCommands = sCommands;
+    this->CreateMenuItems();
+}
+
+void GZCommandManager::CreateMenuItems() {
+    this->mMenu.entries = new GZMenuItem[ARRAY_LEN(sCommands)];
+
+    for (int i = 0; i < ARRAY_LEN(sCommands); i++) {
+        this->mMenu.entries[i].name = sCommands[i].btnCombo.name;
+        this->mMenu.entries[i].action = sCommands[i].actionCallback;
+        this->mMenu.entries[i].submenu = NULL;
+    }
 }
 
 void GZCommandManager::Update() {
@@ -104,7 +115,7 @@ void GZCommandManager::Update() {
 
         if (pCmd->btnCombo.Executed(this->mpButtons)) {
             if (pCmd->actionCallback != NULL) {
-                pCmd->actionCallback();
+                pCmd->actionCallback(0);
             }
         }
     }
@@ -118,7 +129,7 @@ void GZCommandManager::Draw(Vec2b* pPos) {
         bool selected = i + 1 == gMenuManager.mState.itemIndex;
 
         pCmd->btnCombo.SetComboString();
-        DisplayDebugText(0, &elemPos, 0, selected, pCmd->btnCombo.fullName);
+        DisplayDebugText(DRAW_TO_TOP_SCREEN, &elemPos, 0, selected, pCmd->btnCombo.fullName);
         elemPos.y++;
     }
 }

src/gz_game.cpp

@@ -62,14 +62,7 @@ void CustomGame::Run() {
 
     do {
         gGZ.Update();
-
-        {
-            gMenuManager.Update();
-
-            if (gMenuManager.IsActive()) {
-                gMenuManager.Draw();
-            }
-        }
+        gMenuManager.Update();
 
         // stgz: pause and frame advance block
         {