X Tutup
Skip to content

Use safe ObjectID for mouse over controls.#109063

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
bruvzg:safe_mouseover_ids
Nov 18, 2025
Merged

Use safe ObjectID for mouse over controls.#109063
Repiteo merged 1 commit intogodotengine:masterfrom
bruvzg:safe_mouseover_ids

Conversation

@bruvzg
Copy link
Member

@bruvzg bruvzg commented Jul 28, 2025

Fixes #109053

Use ObjectIDs instead of raw pointer for mouse over control states to ensure it is safe to use in case control was deleted in the mouse enter/exit notification callback.

@bruvzg bruvzg added this to the 4.6 milestone Jul 28, 2025
@bruvzg bruvzg requested a review from a team as a code owner July 28, 2025 21:06
@bruvzg bruvzg force-pushed the safe_mouseover_ids branch from 917f949 to d39c1d0 Compare July 29, 2025 09:16
Copy link

@jerome-ceccato jerome-ceccato left a comment

Choose a reason for hiding this comment

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

Can confirm that this fixes the issue that I reported in #109053 in both the MRP and my actual project where I encountered this crash originally.
Thanks for your work!

Copy link
Contributor

@Repiteo Repiteo left a comment

Choose a reason for hiding this comment

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

Changes make sense & implementation is solid! Just needs a rebase

@bruvzg bruvzg force-pushed the safe_mouseover_ids branch from d39c1d0 to 179d1b7 Compare November 18, 2025 18:34
@Repiteo Repiteo merged commit 5cb2bfc into godotengine:master Nov 18, 2025
39 of 40 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 18, 2025

Thanks!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[macOS] Game crashes when freeing a control under the cursor

4 participants

X Tutup