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

Refactor expansion interpreter #33

Open

SabrinaJewson wants to merge 3 commits into meh:master from SabrinaJewson:refactor-expand

Collaborator

SabrinaJewson commented Mar 30, 2023

This function is kind of a mess, it’s absolutely huge and uses macros all over the place. This PR contains a couple refactors of it.

SabrinaJewson added 3 commits

March 30, 2023 19:24


Replace next! with a next_item function

8037bc6


Remove use of loop labels

92da167


Extract binary and unary evaluation to a function

SabrinaJewson changed the title ~~Refactor expand~~ Refactor expansion interpreter

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