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

Pre-pack asset configs (again) #234

Closed
wants to merge 2 commits into from
Closed

Pre-pack asset configs (again) #234

wants to merge 2 commits into from

Conversation

jflatow
Copy link
Contributor

@jflatow jflatow commented Mar 11, 2022

Saves ~1.5KB contract size, which we're likely going to need

@jflatow jflatow force-pushed the jflatow/pre-pack branch 5 times, most recently from deba145 to 0804a90 Compare March 17, 2022 21:31
@jflatow
Copy link
Contributor Author

jflatow commented Mar 21, 2022

If we get more space by merging this, we will likely crank up compiler gas optimizations again (see also #183)

@jflatow jflatow added the contingency plan Keeping open as an alternative option label Apr 26, 2022
@jflatow
Copy link
Contributor Author

jflatow commented Jan 31, 2023

I think we're past this, so closing, but we can always find it in history if we want to come back

@jflatow jflatow closed this Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contingency plan Keeping open as an alternative option
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant