don't allocate entire app on heap

src/backend/Core.cpp

@@ -2,22 +2,22 @@
 #include <Scheduler.hpp>
 
 namespace n64 {
-Core::Core() {
+Core::Core() : cpu{new Interpreter} {
   if(SDL_GameControllerAddMappingsFromFile("resources/gamecontrollerdb.txt") < 0) {
     Util::warn("Failed to load game controller DB");
   }
 }
 
 void Core::Stop() {
-  cpu.Reset();
+  cpu->Reset();
-  cpu.mem.Reset();
+  cpu->mem.Reset();
   pause = true;
   romLoaded = false;
 }
 
 void Core::LoadROM(const std::string& rom_) {
   rom = rom_;
-  cpu.Reset();
+  cpu->Reset();
   pause = false;
   romLoaded = true;
 
@@ -27,21 +27,21 @@ void Core::LoadROM(const std::string& rom_) {
     return extension == i;
   });
 
-  cpu.mem.LoadROM(isArchive, rom);
+  cpu->mem.LoadROM(isArchive, rom);
-  GameDB::match(cpu.mem);
+  GameDB::match(cpu->mem);
-  cpu.mem.mmio.vi.isPal = cpu.mem.IsROMPAL();
+  cpu->mem.mmio.vi.isPal = cpu->mem.IsROMPAL();
-  cpu.mem.mmio.si.pif.InitDevices(cpu.mem.saveType);
+  cpu->mem.mmio.si.pif.InitDevices(cpu->mem.saveType);
-  cpu.mem.mmio.si.pif.LoadMempak(rom);
+  cpu->mem.mmio.si.pif.LoadMempak(rom);
-  cpu.mem.mmio.si.pif.LoadEeprom(cpu.mem.saveType, rom);
+  cpu->mem.mmio.si.pif.LoadEeprom(cpu->mem.saveType, rom);
-  cpu.mem.flash.Load(cpu.mem.saveType, rom);
+  cpu->mem.flash.Load(cpu->mem.saveType, rom);
-  cpu.mem.LoadSRAM(cpu.mem.saveType, rom);
+  cpu->mem.LoadSRAM(cpu->mem.saveType, rom);
-  cpu.mem.mmio.si.pif.ExecutePIF(cpu.mem, cpu.regs);
+  PIF::ExecutePIF(cpu->mem, cpu->regs);
 }
 
 void Core::Run(float volumeL, float volumeR) {
-  Mem& mem = cpu.mem;
+  Mem& mem = cpu->mem;
   MMIO& mmio = mem.mmio;
-  Registers& regs = cpu.regs;
+  Registers& regs = cpu->regs;
 
   for (int field = 0; field < mmio.vi.numFields; field++) {
     int frameCycles = 0;
@@ -53,7 +53,7 @@ void Core::Run(float volumeL, float volumeR) {
       }
 
       for(; cycles < mem.mmio.vi.cyclesPerHalfline; cycles++, frameCycles++) {
-        int taken = cpu.Step();
+        int taken = cpu->Step();
         static int cpuSteps = 0;
         cpuSteps += taken;
         if(mmio.rsp.spStatus.halt) {
@@ -83,7 +83,7 @@ void Core::Run(float volumeL, float volumeR) {
       InterruptRaise(mmio.mi, regs, Interrupt::VI);
     }
 
-    mmio.ai.Step(cpu.mem, regs, frameCycles, volumeL, volumeR);
+    mmio.ai.Step(cpu->mem, regs, frameCycles, volumeL, volumeR);
     scheduler.tick(frameCycles, mem, regs);
   }
 }

src/backend/Core.hpp

@@ -8,7 +8,7 @@ struct Window;
 
 namespace n64 {
 struct Core {
-  ~Core() { Stop(); }
+  ~Core() { Stop(); delete cpu; }
   Core();
   void Stop();
   void LoadROM(const std::string&);
@@ -23,6 +23,6 @@ struct Core {
   bool romLoaded = false;
   bool done = false;
   std::string rom;
-  Interpreter cpu;
+  Interpreter* cpu = nullptr;
 };
 }

src/frontend/imgui/Window.cpp

@@ -11,7 +11,7 @@ namespace fs = std::filesystem;
 
 Window::Window(n64::Core& core) : settings() {
   InitSDL();
-  InitParallelRDP(core.cpu.mem.GetRDRAM(), window);
+  InitParallelRDP(core.cpu->mem.GetRDRAM(), window);
   InitImgui();
   NFD::Init();
 }
@@ -152,7 +152,7 @@ ImDrawData* Window::Present(n64::Core& core) {
 void Window::LoadROM(n64::Core& core, const std::string &path) {
   if(!path.empty()) {
     core.LoadROM(path);
-    gameName = core.cpu.mem.rom.gameNameDB;
+    gameName = core.cpu->mem.rom.gameNameDB;
 
     if(gameName.empty()) {
       gameName = fs::path(path).stem().string();
@@ -181,13 +181,13 @@ void Window::RenderMainMenuBar(n64::Core &core) {
       }
     }
     if (ImGui::MenuItem("Dump RDRAM")) {
-      core.cpu.mem.DumpRDRAM();
+      core.cpu->mem.DumpRDRAM();
     }
     if (ImGui::MenuItem("Dump IMEM")) {
-      core.cpu.mem.DumpIMEM();
+      core.cpu->mem.DumpIMEM();
     }
     if (ImGui::MenuItem("Dump DMEM")) {
-      core.cpu.mem.DumpDMEM();
+      core.cpu->mem.DumpDMEM();
     }
     if (ImGui::MenuItem("Exit")) {
       core.done = true;
@@ -232,8 +232,8 @@ void Window::Render(n64::Core& core) {
   static u32 lastFrame = 0;
   if(!core.pause && lastFrame < ticks - 1000) {
     lastFrame = ticks;
-    windowTitle += fmt::format("  | {:02d} VI/s", core.cpu.mem.mmio.vi.swaps);
+    windowTitle += fmt::format("  | {:02d} VI/s", core.cpu->mem.mmio.vi.swaps);
-    core.cpu.mem.mmio.vi.swaps = 0;
+    core.cpu->mem.mmio.vi.swaps = 0;
     SDL_SetWindowTitle(window, windowTitle.c_str());
     windowTitle = shadowWindowTitle;
   }

src/main.cpp

@@ -2,16 +2,16 @@
 #include <MupenMovie.hpp>
 
 int main(int argc, char** argv) {
-  App* app = new App;
+  App app;
+
   if(argc > 1) {
     if(argc > 2) {
       LoadTAS(argv[2]);
     }
-    app->LoadROM(argv[1]);
+    app.LoadROM(argv[1]);
   }
 
-  app->Run();
+  app.Run();
 
-  delete app;
   return 0;
 }