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.

Sign up for GitHub

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

Jump to bottom

Fix scoping for shadowed variables #142

Merged

nielstron merged 2 commits into dev from fix/scoping

Apr 24, 2023

Contributor

nielstron commented Apr 24, 2023

Fixes a recently discovered issue, where shadowed variables might cause invalid variable name accesses.

nielstron added 2 commits

April 24, 2023 16:33


          Seperate rewriting varnames and saving the original name

8caaf3c


          Rewrite variable access to fix scope accesses

0631b43

nielstron added the bugfix label

nielstron changed the title ~~Fix/scoping~~ Fix scoping for shadowed variables

nielstron merged commit 72c8bf0 into dev

2 checks passed

Contributor Author

nielstron commented Apr 24, 2023

Specifically, this fixes #7 (comment) and #140

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