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.

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

Jump to bottom

Simplify some code in transaction execution #72

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

Simplify some code in transaction execution #72

wants to merge 1 commit into from

Commits on Aug 7, 2023

  1. Simplify some code in transaction execution 

    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.
    @metaproph3t
    metaproph3t committed Aug 7, 2023
    Configuration menu
    Copy the full SHA
    a734c9e View commit details
    Browse the repository at this point in the history