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 support for "foreach" and "define" to masm #157

Closed
wants to merge 2 commits into from

Conversation

pjuftring
Copy link
Contributor

This is an implementation for "foreach" and "define" as announced in #155 (Still not sure, if this is the right way to contribute to open PRs)
It works great, the only problem I encountered is, that in some cases (e.g. nesting) artifacts from comments mess up the generated code, but apart from that, everything works as intended.

@pjuftring pjuftring changed the title Add support for foreach and define to masm Add support for "foreach" and "define" to masm Oct 31, 2015
@sunfishcode
Copy link
Member

I think the idea of a foreach macro makes sense. I'll take a closer look at this once we know what's happening with #155.

@pjuftring
Copy link
Contributor Author

Closing this PR since #155 is closed, too.

@pjuftring pjuftring closed this Dec 2, 2015
rossberg added a commit that referenced this pull request Feb 11, 2021
ngzhian pushed a commit to ngzhian/spec that referenced this pull request Nov 4, 2021
dhil pushed a commit to dhil/webassembly-spec that referenced this pull request Mar 2, 2023
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