Skip to content

improv : modify checks and optimize ixs#13

Merged
kespinola merged 3 commits intomainfrom
shld-27-add-checks-and-optimize-ixs
Jun 19, 2025
Merged

improv : modify checks and optimize ixs#13
kespinola merged 3 commits intomainfrom
shld-27-add-checks-and-optimize-ixs

Conversation

@Nagaprasadvr
Copy link
Contributor

@Nagaprasadvr Nagaprasadvr commented Jun 17, 2025

  • handle accs array destructring to avoid index out of bounds err
  • remove redundant checks
  • avoid find_program_address when we already have access to bump stored on account
  • cleanup

@Nagaprasadvr Nagaprasadvr changed the base branch from main to shld-26-reference-the-token-extensions-mint-on-the-policy June 17, 2025 12:45
@Nagaprasadvr Nagaprasadvr force-pushed the shld-27-add-checks-and-optimize-ixs branch from 5b7c625 to b3a08d3 Compare June 17, 2025 12:54
@Nagaprasadvr Nagaprasadvr requested a review from kespinola June 17, 2025 13:02
@Nagaprasadvr Nagaprasadvr changed the title Shld 27 add checks and optimize ixs improv : add checks and optimize ixs Jun 17, 2025
@Nagaprasadvr Nagaprasadvr changed the title improv : add checks and optimize ixs improv : modify checks and optimize ixs Jun 17, 2025
Copy link
Collaborator

@kespinola kespinola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only question is about the removal of the payer. Everything else looks good. Ty.

@Nagaprasadvr Nagaprasadvr force-pushed the shld-26-reference-the-token-extensions-mint-on-the-policy branch 2 times, most recently from 973808a to c08e6b8 Compare June 18, 2025 07:22
@Nagaprasadvr Nagaprasadvr changed the base branch from shld-26-reference-the-token-extensions-mint-on-the-policy to main June 18, 2025 09:36
@Nagaprasadvr Nagaprasadvr force-pushed the shld-27-add-checks-and-optimize-ixs branch from b3a08d3 to d516f59 Compare June 18, 2025 09:42
@kespinola kespinola force-pushed the shld-27-add-checks-and-optimize-ixs branch from 9ae2373 to 8dc466f Compare June 19, 2025 11:35
@kespinola kespinola force-pushed the shld-27-add-checks-and-optimize-ixs branch from 8dc466f to 4182697 Compare June 19, 2025 11:50
@kespinola kespinola merged commit ca5b5c7 into main Jun 19, 2025
9 checks passed
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.

2 participants