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

Pretty printer as a backend #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ManasJayanth
Copy link

@ManasJayanth ManasJayanth commented Oct 6, 2019

Inspired by -dcmm option. Sample output

(program
    (((a = (dot (var joo_global_object) jsoo_runtime)))
        (call
            (0
                (etag 0 ((funexpr ((a) ((return (toint (+ 7 (var a)))))))))
                SomeFile))))

This can definitely be improved, but it's a start 🙂

PS: I wasn't sure which branch to raise the PR against

@jordwalke
Copy link
Owner

This is really great! I think it needs to be rebased on top of master though? Something is up with the commits.

@robz
Copy link
Collaborator

robz commented Jul 13, 2020

@prometheansacrifice this is really cool, and would be very useful for debugging! Would you like to rebase?

@ManasJayanth
Copy link
Author

@robz Would you like to take it over? I can see that a lot of interesting changes have landed on master and it could take a while for me to resolve the conflicts

@ManasJayanth
Copy link
Author

If not, I'll try in a few weeks

@robz
Copy link
Collaborator

robz commented Jul 24, 2020

I probably won't get to it in the next couple weeks, unless I encounter another difficult bug to fix. If I start I'll comment here

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.

3 participants