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

Report an error when an array length is defined with a backpatch value #40

Closed
erkyrath opened this issue Oct 27, 2020 · 0 comments · Fixed by #41
Closed

Report an error when an array length is defined with a backpatch value #40

erkyrath opened this issue Oct 27, 2020 · 0 comments · Fixed by #41

Comments

@erkyrath
Copy link
Contributor

Currently you can write

Array arrname --> 'test';

This may compile with an unexpected length because the value of 'test' is not yet determined.

erkyrath pushed a commit to erkyrath/Inform6 that referenced this issue Oct 27, 2020
I also fixed a comment and added an error check for zero-length arrays.
It was already an error to write

    Array foo --> 0;

We now catch this case, which previously slipped through:

    Array foo --> [];

Fixes DavidKinder#40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant