David Snopek
David Snopek
> I cant see any comments with a reason why this cant be merged. To me this feels like something that should be an iOS plugin. Or, at the very...
@aaronfranke I think the point that @Bromeon is trying to make, is that we don't necessarily need to bind `Array`'s `get()` and `set()` functions in order for godot-cpp to have...
@jpxaraujo: > Even with this, it still does not work for me. Maybe i'm missing something? That error is usually caused by not having `HOT_RELOAD_ENABLED` defined, which I think is...
Cherry-picked for 4.2 in PR https://github.com/godotengine/godot-cpp/pull/1631
Cherry-picked for 4.3 in PR https://github.com/godotengine/godot-cpp/pull/1632
We discussed this PR at the last GDExtension meetup, and decided that it'd be best for `ScriptInstanceExtension` to exactly match the method signatures from Godot, even if that means allocating...
> Do you think it is possible to add the same for ScriptInstancePlaceholder in this PR ? Yes, I think we could do a similar thing with `PlaceHolderScriptInstance`, but I...
I've made some updates (a few of which were discussed on RocketChat): - Renamed from `ScriptInstanceExtension` to `ScriptInstance` to match the class name within Godot - Added a default implementation...
I've added an implementation of `PlaceHolderScriptInstance` now too, but this is all untested code
Thanks! I've only skimmed the changes, but at a glance this seems good :-) However, I think we should have the discussion I [mentioned](https://github.com/godotengine/godot/pull/95492#issuecomment-2305816212) on the Godot PR before moving...