From 287ee6b2eeb2a6b24dbef0a5e5da038840b68aac Mon Sep 17 00:00:00 2001 From: SuperDude88 <82904174+SuperDude88@users.noreply.github.com> Date: Fri, 10 Jul 2026 22:00:52 -0400 Subject: [PATCH] Small Speedrun Mode Cleanups (#2179) * Deduplicate Speedrun Overrides Avoids issues where only one is updated + makes sense organizationally * Move restore_from_speedrun_mode I think it makes more sense to have this in the same place the overrides are set * Camelcase nit --------- Co-authored-by: Irastris --- include/dusk/speedrun.h | 1 + src/dusk/speedrun.cpp | 12 ++++++++++ src/dusk/ui/settings.cpp | 51 +++------------------------------------- 3 files changed, 16 insertions(+), 48 deletions(-) diff --git a/include/dusk/speedrun.h b/include/dusk/speedrun.h index c1a92e4a50..d9e43c3ed7 100644 --- a/include/dusk/speedrun.h +++ b/include/dusk/speedrun.h @@ -37,5 +37,6 @@ struct SpeedrunInfo { extern SpeedrunInfo m_speedrunInfo; void resetForSpeedrunMode(); +void restoreFromSpeedrunMode(); } // namespace dusk diff --git a/src/dusk/speedrun.cpp b/src/dusk/speedrun.cpp index e396197040..275a8de900 100644 --- a/src/dusk/speedrun.cpp +++ b/src/dusk/speedrun.cpp @@ -1,5 +1,6 @@ #include "dusk/speedrun.h" #include "dusk/settings.h" +#include "dusk/config.hpp" #include "m_Do/m_Do_main.h" #include @@ -43,4 +44,15 @@ void resetForSpeedrunMode() { getSettings().game.debugFlyCam.setSpeedrunValue(false); } +static void clearSpeedrunOverrides() { + config::EnumerateRegistered([](config::ConfigVarBase& cvar) { + cvar.clearSpeedrunOverride(); + }); +} + +void restoreFromSpeedrunMode() { + clearSpeedrunOverrides(); + aurora_set_pause_on_focus_lost(getSettings().game.pauseOnFocusLost.getValue()); +} + } // namespace dusk diff --git a/src/dusk/ui/settings.cpp b/src/dusk/ui/settings.cpp index bcd0d3cfe4..e2ee60729a 100644 --- a/src/dusk/ui/settings.cpp +++ b/src/dusk/ui/settings.cpp @@ -15,6 +15,7 @@ #include "dusk/io.hpp" #include "dusk/livesplit.h" #include "dusk/discord_presence.hpp" +#include "dusk/speedrun.h" #include "graphics_tuner.hpp" #include "m_Do/m_Do_main.h" #include "menu_bar.hpp" @@ -218,52 +219,6 @@ AuroraBackend configured_backend() { return configuredBackend; } -void reset_for_speedrun_mode() { - mDoMain::developmentMode = -1; - - getSettings().game.enableTurboKeybind.setSpeedrunValue(false); - - getSettings().game.damageMultiplier.setSpeedrunValue(1); - getSettings().game.instantDeath.setSpeedrunValue(false); - getSettings().game.noHeartDrops.setSpeedrunValue(false); - getSettings().game.autoSave.setSpeedrunValue(false); - getSettings().game.sunsSong.setSpeedrunValue(false); - - getSettings().game.infiniteHearts.setSpeedrunValue(false); - getSettings().game.infiniteArrows.setSpeedrunValue(false); - getSettings().game.infiniteSeeds.setSpeedrunValue(false); - getSettings().game.infiniteBombs.setSpeedrunValue(false); - getSettings().game.infiniteOil.setSpeedrunValue(false); - getSettings().game.infiniteOxygen.setSpeedrunValue(false); - getSettings().game.infiniteRupees.setSpeedrunValue(false); - getSettings().game.enableIndefiniteItemDrops.setSpeedrunValue(false); - getSettings().game.moonJump.setSpeedrunValue(false); - getSettings().game.superClawshot.setSpeedrunValue(false); - getSettings().game.alwaysGreatspin.setSpeedrunValue(false); - getSettings().game.enableFastIronBoots.setSpeedrunValue(false); - getSettings().game.canTransformAnywhere.setSpeedrunValue(false); - getSettings().game.fastRoll.setSpeedrunValue(false); - getSettings().game.fastSpinner.setSpeedrunValue(false); - getSettings().game.armorRupeeDrain.setSpeedrunValue(MagicArmorMode::NORMAL); - getSettings().game.invincibleEnemies.setSpeedrunValue(false); - - getSettings().game.pauseOnFocusLost.setSpeedrunValue(false); - - getSettings().backend.enableAdvancedSettings.setSpeedrunValue(false); - getSettings().game.recordingMode.setSpeedrunValue(false); - getSettings().game.debugFlyCam.setSpeedrunValue(false); -} - -void clear_speedrun_overrides() { - config::EnumerateRegistered([](config::ConfigVarBase& cvar) { - cvar.clearSpeedrunOverride(); - }); -} - -void restore_from_speedrun_mode() { - clear_speedrun_overrides(); -} - std::filesystem::path normalized_display_path(const std::filesystem::path& path) { std::error_code ec; auto normalized = std::filesystem::weakly_canonical(path, ec); @@ -1318,9 +1273,9 @@ SettingsWindow::SettingsWindow(bool prelaunch) : mPrelaunch(prelaunch) { .onChange = [](bool enabled) { if (enabled) { - reset_for_speedrun_mode(); + resetForSpeedrunMode(); } else { - restore_from_speedrun_mode(); + restoreFromSpeedrunMode(); if (getSettings().game.liveSplitEnabled) { speedrun::disconnectLiveSplit(); }