Shapes using a Dictionary to store data leak memory

[jbromberg]

Certain collision shapes have memory leaks originating from the Dictionary that is used to store their data. The simple script below illustrates the problem:

extends Node

# Leaks
func _process(delta):
	for i in 1000:
		var cap := CapsuleShape3D.new()

# Doesn't leak
func _process(delta):
	for i in 1000:
		var box := BoxShape3D.new()

[mihe]

I don't have the ability to test this for myself right at this moment, but do you see the same behavior if you use Godot Physics?

[jbromberg]

There is no leak using Godot Physics

[mihe]

It turns out this was a bug in the godot-cpp GDExtension bindings, which was fixed a few months back.

I am not able to reproduce this on latest master of this extension, which I guess more or less confirms this, since I haven't really changed much about how these particular Dictionary are used.

This also means that this issue has been fixed in 0.13.0-beta1, and will similarly be fixed in the upcoming 0.13.0-stable, which should hopefully be out shortly after Godot 4.3-stable is released.

[mihe]

0.13.0-stable has just been released, which includes the fix for this.

Note that the release in the Godot Asset Library is still pending approval as of typing this, so you might need to download it from the link above.