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

Add checks for risky goto statements. #757

Closed
wants to merge 7 commits into from
Closed

Add checks for risky goto statements. #757

wants to merge 7 commits into from

Conversation

laubzega
Copy link
Contributor

Emit a warning in a situation described in #756

src/cc65/function.c Outdated Show resolved Hide resolved
src/cc65/function.c Outdated Show resolved Hide resolved
src/cc65/function.h Outdated Show resolved Hide resolved
src/cc65/locals.c Outdated Show resolved Hide resolved
src/cc65/stmt.c Outdated Show resolved Hide resolved
src/cc65/symentry.h Outdated Show resolved Hide resolved
src/cc65/symentry.h Outdated Show resolved Hide resolved
@laubzega
Copy link
Contributor Author

Hey greg-king5, thanks for the quick review. I take it you guys are ok with the way I am using Collection as a stack of numbers?

@laubzega
Copy link
Contributor Author

Superseded by PR #759 .

@laubzega laubzega closed this Sep 30, 2018
oliverschmidt pushed a commit that referenced this pull request Oct 2, 2018
polluks pushed a commit to polluks/cc65 that referenced this pull request Dec 11, 2018
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 this pull request may close these issues.

None yet

2 participants