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

import: Fix uv2 by avoiding premature ImporterMesh::get_mesh() #73814

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

lyuma
Copy link
Contributor

@lyuma lyuma commented Feb 23, 2023

Implements create_convex_shape in ImpoterMesh.
Note: ImporterMeshInstance3D::get_mesh() is safe.
The only dangerous function with side effects is ImpoterMesh::get_mesh()

Thanks to @fire for pointing out that the create_convex_shape function in Mesh could be easily replicated in ImporterMesh.

Fixes #64746
Fixes #60257
Fixes #70937

image

Implements create_convex_shape in ImpoterMesh.
Note: ImporterMeshInstance3D::get_mesh() is safe.
The only dangerous function with side effects is ImpoterMesh::get_mesh()
@lyuma lyuma requested a review from a team as a code owner February 23, 2023 10:57
@lyuma lyuma added this to the 4.0 milestone Feb 23, 2023
@akien-mga akien-mga merged commit 13382a8 into godotengine:master Feb 23, 2023
@akien-mga
Copy link
Member

Thanks!

@fire fire deleted the importer_mesh_convex branch February 23, 2023 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants