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

Simplify some code in transaction execution #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

metaproph3t
Copy link

Hello, if anyone's out there 👋

I'm not sure if anyone uses this repo in production but it still seems like a repo that people use to learn Anchor, judging by the 100 forks and my own personal experience.

Case in point, I'm working on a timelock program & ended up re-using a lot of the code from this repo. When I was looking at this part of it though, I realized that you could accomplish this in a way that was more simple and efficient. I've measured it and it removes 285 CUs in these tests.

I think this provides pedagogical value even if this repo isn't being used in prod.

Hello, if anyone's out there 👋

I'm not sure if anyone uses this repo in production but it still seems
like a repo that people use to learn Anchor, judging by the 100 forks
and my own personal experience.

Case in point, I'm working on a timelock program & ended up re-using a
lot of the code from this repo. When I was looking at this part of it
though, I realized that you could accomplish this in a way that was more
simple and efficient. I've measured it and it removes 285 CUs in these
tests.

I think this provides pedagogical value even if this repo isn't being
used in prod.
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.

1 participant