Age | Commit message (Collapse) | Author |
|
This change aims to reduce the number of places that need to be changed
when adding or editing a Godot type to the bindings.
Since the addition of `Variant.From<T>/As<T>` and
`VariantUtils.CreateFrom<T>/ConvertTo<T>`, we can now replace a lot of
the previous code in the bindings generator and the source generators
that specify these conversions for each type manually.
The only exceptions are the generic Godot collections (`Array<T>` and
`Dictionary<TKey, TValue>`) which still use the old version, as that
one cannot be matched by our new conversion methods (limitation in the
language with generics, forcing us to use delegate pointers).
The cleanup applies to:
- Bindings generator:
- `TypeInterface.cs_variant_to_managed`
- `TypeInterface.cs_managed_to_variant`
- Source generators:
- `MarshalUtils.AppendNativeVariantToManagedExpr`
- `MarshalUtils.AppendManagedToNativeVariantExpr`
- `MarshalUtils.AppendVariantToManagedExpr`
- `MarshalUtils.AppendManagedToVariantExpr`
|
|
This commit replaces most usages of `ConvertManagedObjectToVariant` and
`ConvertVariantToManagedObjectOfType`, by using the `Godot.Variant`
struct instead of `System.Object`.
The most notable change is to the `GetGodotPropertyDefaultValues` method
that's generated for scripts. The dictionary it returns now stores
`Godot.Variant` values.
Remaining usages are:
- The `DelegateUtils` class, for the serialization of closure display
classes during assembly reloading by the editor. These display classes
are compiler generated classes to store values captured by a closure.
Since it's generated by the compiler, the only way we have to access
the fields is through reflection. This leads to using `System.Object`.
- Converting parameters when invoking constructors from the engine.
This will be replaced with source generators in the future.
- Legacy support for old `GetGodotPropertyDefaultValues` return values.
We need to keep supporting the old version of this generated method
for some time. Otherwise, if loading a project built with the previous
version, it could lead to the loss of exported property values.
Ideally, we should remove this legacy support before a stable release.
|
|
Its two usages were:
- The Array `ICollection.CopyTo` implementation.
It's possible that this class shouldn't be implementing the
non-generic `ICollection`, but this commit doesn't change that.
The new implementation stores the elements as boxed `Variant` values.
- The `Variant.Obj` property.
I'm not sure if this property's existence is justified, but for now
I rewrote it as a simpler version of `ConvertVariantToManagedObject`.
|
|
C#: Fix signature of generated signal callbacks
|
|
Update docs to clarify `@icon` only works with global script classes
|
|
Fix child nodes still being drawn when the visible area of a control with `clip_contents` enabled is `0`
|
|
Update thorvg to 0.8.3
|
|
Add Skeletons and Blend Shapes to the OpenGL renderer
|
|
Update the docs to clarify that the `@icon` annotation does not work
when only attaching a script to a node, and clarify that the script itself
must be a globally accessible script type.
|
|
Add navigation control to the node3d editor viewport
|
|
Remove unneeded calc in root motion
|
|
Improve validation when setting `max`/`min`/`page` in `Range`
|
|
Restore metadata to script create dialog
|
|
Improve parent signature error
|
|
touilleMan/gdextension-skip-undefined-field-in-properties
[GDExtension] Skip unset getter/setter/index fields in class property
|
|
touilleMan/gdextension-correct-rect2-members-offsets-declaration
[GDExtension] Correct Rect2 members offsets declaration in extension_api.json generator
|
|
Rename Tree theme cache separation names
|
|
Simplify GDVIRTUAL_REQUIRED_CALL calls
|
|
Fix routing of InputEventScreenDrag events to Control nodes
|
|
GDScript 2.0: Fix shift due to skip of non-constant default argument values
|
|
|
|
This reverts commit bb5198a5ac92a6fe699751df9bb9c09109a2955a.
|
|
|
|
`clip_contents` enabled is `0`
|
|
Reset unassigned local variables to null in the loops
|
|
|
|
|
|
Bump the minimum Android target api to 21 (Android Lollipop)
|
|
|
|
Emphasize the warning about special case blend for animation
|
|
Fix OpenXR on the Meta Quest: XR_KHR_loader_init_android is not reported as available (but it works anyway)
|
|
Make remote inspector groups not foldable
|
|
Display transform properties of contained controls
|
|
Update miniupnpc to 2.2.4
|
|
|
|
Remove unnecessary namespaces in generated PropertyDefVal files.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Adjust position and size for dropping texture in canvas editor.
|
|
Acknowledge that a CLASS kind of a DataType might not have an identifier
|
|
[macOS] Fix export button incorrectly disabled when using login/app password for notarization.
|
|
Fix heap-use-after-free when changing alternative tile ID
|
|
Move z_index, z_as_relative and y_sort_enabled from `Node2D` to `CanvasItem`
|
|
Print each migrated project path instead of always printing "migrating" when there are none
|
|
for notarization.
|
|
|