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

Jurst can't use multi-dimensional arrays #396

Closed
Cokemonkey11 opened this issue Sep 8, 2015 · 4 comments
Closed

Jurst can't use multi-dimensional arrays #396

Cokemonkey11 opened this issue Sep 8, 2015 · 4 comments

Comments

@Cokemonkey11
Copy link
Collaborator

Jurst aims to support vJass, but multi-dimensional arrays are not included - they raise an error "Only one-dimensional arrays are supported currently." in Eclipse.

Fixing this shouldn't be too complicated I would imagine.

@Crigges
Copy link
Contributor

Crigges commented Sep 9, 2015

rel #364

@peq
Copy link
Collaborator

peq commented Sep 9, 2015

Maybe a better approach is to use a custom data type or module instead of a built-in language feature. Then you could also decide whether sizes are dynamic or static, whether to do bound-checks or not, and so on.

#297 might help to make it look very similar to native arrays.

Support from the compiler is probably only required, if you want more than 8191 entries.

So: this feature might be nice to have, but it would not give too many benefits compared to a library. Therefore I would give it a low priority.

@Cokemonkey11
Copy link
Collaborator Author

Maybe a better approach is to use a custom data type or module instead of a built-in language feature. Then you could also decide whether sizes are dynamic or static, whether to do bound-checks or not, and so on.

Isn't it true that plain JASS already has a weird static/dynamic syntax?

I think int array q and int array w[5] are both valid, and dynamic sizing will only work for the first. Am I misled?

@peq peq added the Jurst label Mar 10, 2018
@Frotty
Copy link
Member

Frotty commented Nov 29, 2023

Jurst is deprecated

@Frotty Frotty closed this as completed Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants