Ignacio Roldán Etcheverry

Results 84 comments of Ignacio Roldán Etcheverry

> ... which is why I wish we could just do `SignalName.Invoke`. ... I very much wish that was the case. I can't believe events have `add` and `remove` overloads,...

This depends on #12908 first before a fix can be added for C#.

Yes, I'll review this tonight. Property default values are delicate so I want to make sure we're not doing anything undesirable.

`window_center` is confusing, as "center" can be interpreted as a noun. Since `center_window` would go against the convention of using `window_` as a prefix, how about `window_center_to_screen` (used in [WinForms](https://learn.microsoft.com/en-us/dotnet/api/system.windows.forms.form.centertoscreen))...

This was removed on purpose. I've thought of a way to add it back, though, including for type parameters of Godot generic collections and delegate invocations. Let's leave this unmerged...

This change is only about the default value, right? If that's the case, I can change `GetGodotPropertyDefaultValues` to return `Variant`. That would be one usage less of `ConvertManagedObjectToVariant`. The last...

The `DelegateUtils` usage seems complicated to replace. It uses `System.Object` because it needs reflection to access the fields of compiler generated classes (`c__DisplayClass`) for closures (needed by the editor during...

> I understand this, but we largely prefer source scanning. It is a big plus in usability and version control. Wildcards have a problem when it comes to version control...

Not a language specific issue. Here's the same example project uploaded by @qarmin but in GDScript: [Test32470.zip](https://github.com/godotengine/godot/files/4098841/Test32470.zip)