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

identifier Analyzer: should recognize variables of for loops #8

Open
treinen opened this issue May 30, 2018 · 3 comments
Open

identifier Analyzer: should recognize variables of for loops #8

treinen opened this issue May 30, 2018 · 3 comments
Assignees

Comments

@treinen
Copy link
Contributor

treinen commented May 30, 2018

there is a third case of identifiers that should be recognized: variables that are only assigned
by the iteration of a for loop.

@treinen treinen self-assigned this May 30, 2018
@Niols
Copy link
Member

Niols commented May 30, 2018 via email

@treinen
Copy link
Contributor Author

treinen commented May 30, 2018

assignement to loop variables does not appear as assignement word in the concrete syntax tree, so this wasn't a good idea. And the detection of for loops is already covered by the structure analyzer. However detection of special buildins concerning variables should go here, as suggested by Nicolas.

@treinen
Copy link
Contributor Author

treinen commented Mar 14, 2019

the parameter analyser (previously known as dollar analyser) keeps track of the loop variables of for loops, and filters them out when looking for occurences of non-expanded variables.

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

No branches or pull requests

2 participants