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

analyze+eval: fall back to evaluating a form with less metadata #359

Merged
merged 2 commits into from
Apr 1, 2021

Conversation

vemv
Copy link
Collaborator

@vemv vemv commented Mar 27, 2021

This metadata, likely provenant from Eastwood's processing, was bloating code size, leading to "Method code too large!" exceptions.

...Metadata is only removed as a last resource. I also checked that basically all of the proposed metadata (:arglists :file :line :column :end-line :end-column) has to be removed and not just a subset. I guess every byte counts!

Fixes #191
Fixes #344

@vemv vemv changed the title analyze+eval: fall back to evaluating a form with less metadata when needed analyze+eval: fall back to evaluating a form with less metadata Mar 27, 2021
…n needed

This metadata, partly provenant from Eastwood's processing, was bloating code size, leading to "Method code too large!" exceptions.

Metadata is only removed as a last resource.

Fixes jonase#191
@vemv
Copy link
Collaborator Author

vemv commented Mar 29, 2021

Rebased to use the latest test matrix. Green :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants