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 for nested subroutines and if statements in subroutines #53

Merged

PhilReinhold merged 2 commits into main from pcrein/nested-subroutine-if

Jul 10, 2023

Collaborator

PhilReinhold commented Jul 10, 2023

We encountered some errors using subroutines:

If statements without accompanying else statements were being dropped in the output
Using subroutines within a subroutine did not cause the inner subroutine definition to be added to the program
Variables used within a subroutine are not properly autodeclared within that subroutine.

PhilReinhold added 2 commits

July 10, 2023 10:37


          Fix for nested subroutines and if statements in subroutines

ca54ad3


          Also autodeclare

986d4bf

jcjaskula-aws approved these changes

View reviewed changes

Collaborator

jcjaskula-aws left a comment

LGTM

PhilReinhold merged commit 0af69d9 into main

jcjaskula-aws deleted the pcrein/nested-subroutine-if branch

June 17, 2024 15:56

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

Labels

None yet