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

bug in the constants #190

Closed
Tracked by #231
MatiDragon-YT opened this issue Apr 7, 2022 · 5 comments
Closed
Tracked by #231

bug in the constants #190

MatiDragon-YT opened this issue Apr 7, 2022 · 5 comments
Labels
priority:p3 Low priority scope:compiler type:bug Something isn't working workaround-available Users can fix for themselves
Milestone

Comments

@MatiDragon-YT
Copy link
Member

Sanny does not replace variable constants that are used as arrays.

image

@MatiDragon-YT
Copy link
Member Author

It only compiles successfully, if you manually replace the constants with their values.

@MatiDragon-YT
Copy link
Member Author

It also fails this way, along with the highlighter.

image

@x87
Copy link
Collaborator

x87 commented Apr 8, 2022

this page describes that limitation and a workaround: https://docs.sannybuilder.com/coding/constants#limitations

@x87 x87 added type:bug Something isn't working priority:p3 Low priority workaround-available Users can fix for themselves scope:compiler labels Apr 8, 2022
@MatiDragon-YT
Copy link
Member Author

I see... so you could support this?
And while thinking about whether to give it or not, may I suggest documenting the case of local variables.

@x87 x87 added this to the v3.9.0 milestone Aug 4, 2023
@x87 x87 mentioned this issue Aug 4, 2023
46 tasks
@x87 x87 closed this as completed Sep 9, 2023
@x87
Copy link
Collaborator

x87 commented Sep 9, 2023

fixed in 3.9.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:p3 Low priority scope:compiler type:bug Something isn't working workaround-available Users can fix for themselves
Projects
None yet
Development

No branches or pull requests

2 participants