Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GDScript: enum with item referencing another item causes editor crash with no error #71588

Closed
mrtripie opened this issue Jan 18, 2023 · 2 comments · Fixed by #71624
Closed

GDScript: enum with item referencing another item causes editor crash with no error #71588

mrtripie opened this issue Jan 18, 2023 · 2 comments · Fixed by #71624

Comments

@mrtripie
Copy link

Godot version

4.0 Beta 11 to 13

System information

Windows 10

Issue description

When trying to upgrade from Beta 10 one of my projects would immediately crash when opening the editor, with no error in the console which also crashed. I managed to isolate it to this enum:

enum { # Directions, clockwise, powers of 100, each direction gets 2 digits, allowing 100 terrains
	CENTER = 01,
	NORTH = 0100, # -Z
	NORTH_EAST = 010000,
	EAST = 01000000, # +X
	SOUTH_EAST = 0100000000,
	SOUTH = 010000000000, # +Z
	SOUTH_WEST = 01000000000000,
	WEST = 0100000000000000, # -X
	NORTH_WEST = 010000000000000000,
	ALL = 010101010101010101,
	ALL_CORNERS = NORTH_EAST + SOUTH_EAST + SOUTH_WEST + NORTH_WEST,
}

With this last line causing the crash:
ALL_CORNERS = NORTH_EAST + SOUTH_EAST + SOUTH_WEST + NORTH_WEST,

This worked in all the betas I've used up to and including Beta 10

Steps to reproduce

Copy and paste that enum into a script, and save. Simply having a script with this enum will cause the editor to crash, the script doesn't need to be used anywhere.

Minimal reproduction project

N/A

@akien-mga
Copy link
Member

CC @anvilfolk

@anvilfolk
Copy link
Contributor

Found the cause, definitely a regression from #69590. Super simple & easy fix in #71624.

Thank you for reporting!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants