Skip to content

Fix error caused by modifying Flags.MAX_SPLASHES - #963

Open
HEIHUAa wants to merge 1 commit into
CodenameCrew:mainfrom
HEIHUAa:Flags.MAX_SPLASHES-fix
Open

Fix error caused by modifying Flags.MAX_SPLASHES#963
HEIHUAa wants to merge 1 commit into
CodenameCrew:mainfrom
HEIHUAa:Flags.MAX_SPLASHES-fix

Conversation

@HEIHUAa

@HEIHUAa HEIHUAa commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Raltyro Raltyro added the status:awaiting Awaiting response by creator of issue label Jul 26, 2026
Comment thread source/funkin/game/SplashGroup.hx
@HEIHUAa

HEIHUAa commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

If we want to change it to Math.min(Flags.MAX_SPLASHES, 7) without crashing, we'll have to dig into flixel.group to find the real issue.

@Raltyro

Raltyro commented Aug 1, 2026

Copy link
Copy Markdown
Member

Do you have the crash log? it would be helpful

@HEIHUAa

HEIHUAa commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

Do you have the crash log? it would be helpful

Wait a moment.

@HEIHUAa

HEIHUAa commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

Do you have the crash log? it would be helpful

2026-08-01-13-05-18-CodenameEngine This is the error pop-up. The crash occurs when the first splash effect is triggered.

@HEIHUAa

HEIHUAa commented Aug 1, 2026

Copy link
Copy Markdown
Contributor Author

At least judging from the code, I believe that FlxTypedGroup.recycle() fails to retrieve a splash, which leads to __splash.strum = strum; — however, the crash occurs at the point of assignment.

@Raltyro Raltyro added type:help wanted Extra attention is needed and removed status:awaiting Awaiting response by creator of issue labels Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:help wanted Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants