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.

Sign up for GitHub

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

Jump to bottom

Initialize vertexSource #4284

Merged
merged 1 commit into from Sep 25, 2022
Merged

Initialize vertexSource #4284

merged 1 commit into from Sep 25, 2022

Conversation

vitalybuka
Copy link
Contributor

Default constructor of btVector3 does not initialize the object.
This fixes Memory Sanitizer reports.

@vitalybuka
Initialize vertexSource
6626ed1 
Default constructor of btVector3 does not initialize the object.
This fixes Memory Sanitizer reports.
@fuchuyuan
Copy link
Contributor

Hi Erwin, can you take a look at this PR?

@vitalybuka
Copy link
Contributor Author

vitalybuka commented Jul 21, 2022 •
edited

Do we need to cc @erwincoumans somehow?

@erwincoumans
Copy link
Member

Thanks, looks good to me.

@erwincoumans erwincoumans merged commit 2c204c4 into bulletphysics:master Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@vitalybuka @fuchuyuan @erwincoumans