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

Revert: KeyedAccount refactoings in builtin programs #23649

Conversation

Lichtso
Copy link
Contributor

@Lichtso Lichtso commented Mar 14, 2022

Problem

@jackcmay and me came to the conclusion that the recent refactoings in builtin programs are too hard to review.
So we decided to revert them so that we can reapply them in a structured and clean way.

Summary of Changes

Reverts #23375, #23348, #23302, #23214, #23217, #23056.
Some conflicts occurred and had to be resolved because of changes in the meantime (such as checked integer arithmetic and checked array accesses which were added).

Fixes #

@Lichtso Lichtso requested a review from jackcmay March 14, 2022 16:22
@Lichtso Lichtso force-pushed the revert/keyed_account_refactoings_in_builtin_programs branch 4 times, most recently from ce83ddf to 9cd9fd9 Compare March 15, 2022 07:07
@Lichtso Lichtso force-pushed the revert/keyed_account_refactoings_in_builtin_programs branch from 9cd9fd9 to a69207c Compare March 15, 2022 14:55
@codecov
Copy link

codecov bot commented Mar 15, 2022

Codecov Report

Merging #23649 (a69207c) into master (11be3fb) will decrease coverage by 0.0%.
The diff coverage is 84.8%.

@@            Coverage Diff            @@
##           master   #23649     +/-   ##
=========================================
- Coverage    81.7%    81.7%   -0.1%     
=========================================
  Files         580      580             
  Lines      158184   158151     -33     
=========================================
- Hits       129341   129271     -70     
- Misses      28843    28880     +37     

@jackcmay
Copy link
Contributor

Looks like #23672 was reverted here, @Lichtso can you confirm?

@jackcmay
Copy link
Contributor

Compared this to v1.9, lgtm

@Lichtso Lichtso merged commit 584ac80 into solana-labs:master Mar 16, 2022
@Lichtso Lichtso deleted the revert/keyed_account_refactoings_in_builtin_programs branch March 16, 2022 10:30
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