Skip to content

Commit 8cc78cf

Browse files
committed
Update free_property_list callback to take count
1 parent e23b117 commit 8cc78cf

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

gdextension/gdextension_interface.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -256,6 +256,7 @@ typedef struct {
256256

257257
typedef const GDExtensionPropertyInfo *(*GDExtensionClassGetPropertyList)(GDExtensionClassInstancePtr p_instance, uint32_t *r_count);
258258
typedef void (*GDExtensionClassFreePropertyList)(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list);
259+
typedef void (*GDExtensionClassFreePropertyList2)(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list, uint32_t p_count);
259260
typedef GDExtensionBool (*GDExtensionClassPropertyCanRevert)(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name);
260261
typedef GDExtensionBool (*GDExtensionClassPropertyGetRevert)(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name, GDExtensionVariantPtr r_ret);
261262
typedef GDExtensionBool (*GDExtensionClassValidateProperty)(GDExtensionClassInstancePtr p_instance, GDExtensionPropertyInfo *p_property);
@@ -333,7 +334,7 @@ typedef struct {
333334
GDExtensionClassSet set_func;
334335
GDExtensionClassGet get_func;
335336
GDExtensionClassGetPropertyList get_property_list_func;
336-
GDExtensionClassFreePropertyList free_property_list_func;
337+
GDExtensionClassFreePropertyList2 free_property_list_func;
337338
GDExtensionClassPropertyCanRevert property_can_revert_func;
338339
GDExtensionClassPropertyGetRevert property_get_revert_func;
339340
GDExtensionClassValidateProperty validate_property_func;

include/godot_cpp/classes/wrapped.hpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ class Wrapped {
7878
static GDExtensionBool set_bind(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name, GDExtensionConstVariantPtr p_value) { return false; }
7979
static GDExtensionBool get_bind(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name, GDExtensionVariantPtr r_ret) { return false; }
8080
static const GDExtensionPropertyInfo *get_property_list_bind(GDExtensionClassInstancePtr p_instance, uint32_t *r_count) { return nullptr; }
81-
static void free_property_list_bind(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list) {}
81+
static void free_property_list_bind(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list, uint32_t p_count) {}
8282
static GDExtensionBool property_can_revert_bind(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name) { return false; }
8383
static GDExtensionBool property_get_revert_bind(GDExtensionClassInstancePtr p_instance, GDExtensionConstStringNamePtr p_name, GDExtensionVariantPtr r_ret) { return false; }
8484
static GDExtensionBool validate_property_bind(GDExtensionClassInstancePtr p_instance, GDExtensionPropertyInfo *p_property) { return false; }
@@ -291,11 +291,10 @@ public:
291291
return ::godot::internal::create_c_property_list(plist_cpp, r_count); \
292292
} \
293293
\
294-
static void free_property_list_bind(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list) { \
294+
static void free_property_list_bind(GDExtensionClassInstancePtr p_instance, const GDExtensionPropertyInfo *p_list, uint32_t p_count) { \
295295
if (p_instance) { \
296296
m_class *cls = reinterpret_cast<m_class *>(p_instance); \
297297
cls->plist_owned.clear(); \
298-
/* TODO `GDExtensionClassFreePropertyList` is ill-defined, we need a non-const pointer to free this. */ \
299298
::godot::internal::free_c_property_list(const_cast<GDExtensionPropertyInfo *>(p_list)); \
300299
} \
301300
} \

include/godot_cpp/core/class_db.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ void ClassDB::_register_class(bool p_virtual, bool p_exposed, bool p_runtime) {
228228
T::set_bind, // GDExtensionClassSet set_func;
229229
T::get_bind, // GDExtensionClassGet get_func;
230230
T::has_get_property_list() ? T::get_property_list_bind : nullptr, // GDExtensionClassGetPropertyList get_property_list_func;
231-
T::free_property_list_bind, // GDExtensionClassFreePropertyList free_property_list_func;
231+
T::free_property_list_bind, // GDExtensionClassFreePropertyList2 free_property_list_func;
232232
T::property_can_revert_bind, // GDExtensionClassPropertyCanRevert property_can_revert_func;
233233
T::property_get_revert_bind, // GDExtensionClassPropertyGetRevert property_get_revert_func;
234234
T::validate_property_bind, // GDExtensionClassValidateProperty validate_property_func;

0 commit comments

Comments
 (0)