Skip to content

[BUGFIX] Fix the game crashing when hot-reloading with F5#5065

Merged
AbnormalPoof merged 2 commits intoFunkinCrew:developfrom
AbnormalPoof:bugfix/playstate-instance-crash
Jun 17, 2025
Merged

[BUGFIX] Fix the game crashing when hot-reloading with F5#5065
AbnormalPoof merged 2 commits intoFunkinCrew:developfrom
AbnormalPoof:bugfix/playstate-instance-crash

Conversation

@AbnormalPoof
Copy link
Member

@AbnormalPoof AbnormalPoof commented May 11, 2025

Linked Issues

Fixes #4990

Description

instance gets set to null in performCleanup() when the game hot-reloads, this was caused by #4769.
This led to multiple errors from scripts that tried to access instance, and eventually a crash from line 1882 from DiscordClient.

This PR sets instance back to this, fixing the errors and the aforementioned crash.

@AbnormalPoof AbnormalPoof added the type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression label May 11, 2025
@github-actions github-actions bot added status: pending triage Awaiting review. size: small A small pull request with 10 or fewer changes. pr: haxe PR modifies game code. labels May 11, 2025
@AbnormalPoof AbnormalPoof changed the base branch from main to develop May 11, 2025 20:15
@Hundrec
Copy link
Member

Hundrec commented May 11, 2025

lol

…OOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO

Co-authored-by: Hundrec <hundrecard@gmail.com>
Copy link
Member

@Hundrec Hundrec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes Blazin' reloading, probably everything else too!

Copy link
Contributor

@NotHyper-474 NotHyper-474 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay I was right #4990 (comment)

@EliteMasterEric EliteMasterEric added size: tiny A tiny pull request with 4 or fewer changes. and removed size: small A small pull request with 10 or fewer changes. size: tiny A tiny pull request with 4 or fewer changes. labels May 14, 2025
@AbnormalPoof AbnormalPoof added status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. and removed status: pending triage Awaiting review. labels May 28, 2025
@AbnormalPoof AbnormalPoof added this to the 0.7.0 milestone May 28, 2025
@AbnormalPoof AbnormalPoof merged commit 9cffb3f into FunkinCrew:develop Jun 17, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: haxe PR modifies game code. size: tiny A tiny pull request with 4 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: major bug Involves a major bug, including crashes, softlocks, or issues blocking progression

Development

Successfully merging this pull request may close these issues.

Crash Report: Blazin' crashin' when hitting F5 (Hot Reload)

5 participants