Skip to content

Fix error with "+=" in "n?.n += n" - #26

Closed
HEIHUAa wants to merge 3 commits into
CodenameCrew:codename-devfrom
HEIHUAa:fix
Closed

HEIHUAa wants to merge 3 commits into
CodenameCrew:codename-devfrom
HEIHUAa:fix

Conversation

@HEIHUAa

@HEIHUAa HEIHUAa commented Oct 22, 2026

Copy link
Copy Markdown

No description provided.

@HEIHUAa HEIHUAa changed the title Fix error with += in n?.n += n Fix error with "+=" in "n?.n += n" Oct 22, 2026
@HEIHUAa

HEIHUAa commented Oct 22, 2026

Copy link
Copy Markdown
Author

I actually wrote this fix a few days ago, but forgot to post it.

@Jamextreme140

Copy link
Copy Markdown
Collaborator

Thanks for the fix, however the refactored safe navigation feature was reverted to prevent breaking scripts that assign values while using safe navigation.

The refactored featured will be used for the next version

@HEIHUAa

HEIHUAa commented Oct 22, 2026

Copy link
Copy Markdown
Author

Wait, did they actually cancel the changes today?

@HEIHUAa

HEIHUAa commented Oct 22, 2026

Copy link
Copy Markdown
Author

Thanks for the fix, however the refactored safe navigation feature was reverted to prevent breaking scripts that assign values while using safe navigation.

The refactored featured will be used for the next version

Okay

@HEIHUAa HEIHUAa closed this Oct 22, 2026
@Jamextreme140

Jamextreme140 commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Wait, did they actually cancel the changes today?

Yeah, turns out many users had many errors on latest builds, specifically doing things like this: Framerate.instance?.visible = false;

I'm looking for keep it working since latest Haxe 5 preview allows you to assign values while using safe navigation

@HEIHUAa

HEIHUAa commented Oct 22, 2026

Copy link
Copy Markdown
Author

Wait, did they actually cancel the changes today?

Yeah, turns out many users had many errors on latest builds, specifically doing things like this: Framerate.instance?.visible = false;

I'm looking for keep it working since latest Haxe 5 preview allows you to assign values while using safe navigation

My fix actually covers the whole set of logic. Theoretically, this has been fixed as well. After checking, it just reduces the processing in this part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants