godot icon indicating copy to clipboard operation
godot copied to clipboard

Acknowledge that a CLASS kind of a DataType might not have an identifier

Open red1939 opened this issue 2 years ago • 1 comments

In the case of a preload()ed Resource that will be later used for a @export var typed_array : Array[MyRes] identifier is nullptr which leads to a quick death by a segfault when trying to set the hint. Looking at the to_string() method it seems for a very long time we assumed that this might be the case, but we don't do it know.

Other explanation is that when set_container_element_type() is called a non-fully initialized DataType for a class is passed, for some reason.

red1939 avatar Nov 27 '22 22:11 red1939

CC @rune-scape

adamscott avatar Nov 27 '22 23:11 adamscott

Looks good! Could you squash the commits? See PR workflow for instructions.

akien-mga avatar Nov 30 '22 07:11 akien-mga

I wanted to ask it, to be honest, but sure ;).

On Wed, 30 Nov 2022 at 08:13, Rémi Verschelde @.***> wrote:

Looks good! Could you squash the commits? See PR workflow https://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html#the-interactive-rebase for instructions.

— Reply to this email directly, view it on GitHub https://github.com/godotengine/godot/pull/69269#issuecomment-1331731216, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNME2BJHDDZFDPOOASYHN3WK35AHANCNFSM6AAAAAASMXEKPM . You are receiving this because you authored the thread.Message ID: @.***>

red1939 avatar Nov 30 '22 08:11 red1939

Thanks! And congrats for your first merged Godot contribution :tada:

akien-mga avatar Nov 30 '22 09:11 akien-mga

Not the last one I guess ;).

On Wed, 30 Nov 2022 at 10:53, Rémi Verschelde @.***> wrote:

Thanks! And congrats for your first merged Godot contribution 🎉

— Reply to this email directly, view it on GitHub https://github.com/godotengine/godot/pull/69269#issuecomment-1331895907, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNME2B57QRDOKD6XZW7NODWK4PZJANCNFSM6AAAAAASMXEKPM . You are receiving this because you authored the thread.Message ID: @.***>

red1939 avatar Nov 30 '22 10:11 red1939