Skip to content

Update to use mem_alloc2, mem_realloc2 and mem_free2#1821

Merged
dsnopek merged 1 commit into
godotengine:masterfrom
dsnopek:mem-alloc-pad-align
Oct 3, 2025
Merged

Update to use mem_alloc2, mem_realloc2 and mem_free2#1821
dsnopek merged 1 commit into
godotengine:masterfrom
dsnopek:mem-alloc-pad-align

Conversation

@dsnopek

@dsnopek dsnopek commented Jul 17, 2025

Copy link
Copy Markdown
Collaborator

Fixes #1820

Marking as a DRAFT because this depends on godotengine/godot#108725

UPDATE: Godot PR has been merged!

@dsnopek dsnopek added this to the 4.x milestone Jul 17, 2025
@dsnopek dsnopek added the bug This has been identified as a bug label Jul 17, 2025
@dsnopek
dsnopek requested a review from a team as a code owner July 17, 2025 21:29
@dsnopek
dsnopek marked this pull request as draft July 17, 2025 21:29
@dsnopek dsnopek added the waiting for Godot PRs that can't be merged until an engine PR is merged first label Jul 17, 2025
@dsnopek
dsnopek force-pushed the mem-alloc-pad-align branch from 80e5346 to 477f64a Compare July 17, 2025 21:58
@Repiteo Repiteo removed the waiting for Godot PRs that can't be merged until an engine PR is merged first label Sep 30, 2025
@Repiteo
Repiteo marked this pull request as ready for review September 30, 2025 23:42
@dsnopek dsnopek changed the title [DRAFT] Update to use mem_alloc2, mem_realloc2 and mem_free2 Update to use mem_alloc2, mem_realloc2 and mem_free2 Oct 1, 2025
@dsnopek
dsnopek force-pushed the mem-alloc-pad-align branch from 477f64a to cfdc058 Compare October 1, 2025 11:46
Comment thread src/core/memory.cpp Outdated
@dsnopek
dsnopek force-pushed the mem-alloc-pad-align branch from cfdc058 to 48d02db Compare October 1, 2025 13:13

@Ivorforce Ivorforce left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me!

@dsnopek
dsnopek merged commit 4218f93 into godotengine:master Oct 3, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug This has been identified as a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using a release build of a GDExtension (godot-cpp) with a debug build of Godot will lead to heap corruption

3 participants