Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix CSGShape3D's _mesh_changed to be thread safe #86538

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

jsjtxietian
Copy link
Contributor

@jsjtxietian jsjtxietian commented Dec 27, 2023

Fixes #86381

If in a node safe thread, just call update_gizmos dierctly, otherwise use call_deferred

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

Might need a dedicated approach to queue a change, but think this is a good approach, will leave it for evaluation by others as well before I'm approving 🙂

@jsjtxietian jsjtxietian force-pushed the fix-shape-thread-error branch from 73aa7d9 to 8814ba5 Compare December 28, 2023 04:35
@KoBeWi
Copy link
Member

KoBeWi commented Jul 29, 2024

The call could always be deferred, no? I don't see why a condition would be useful.

@KoBeWi KoBeWi modified the milestones: 4.3, 4.4 Jul 29, 2024
@jsjtxietian jsjtxietian force-pushed the fix-shape-thread-error branch from 8814ba5 to 5e4dafb Compare July 30, 2024 02:57
@akien-mga akien-mga merged commit 40f938c into godotengine:master Aug 19, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks!

@jsjtxietian jsjtxietian deleted the fix-shape-thread-error branch August 19, 2024 13:07
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.

Create a Mesh for a CSGMesh3D will trigger caller thread can't call this function in this node
4 participants