Skip to content

Implement EXTCODE* changes for pdn-5#13106

Merged
somnergy merged 34 commits intomainfrom
som/7720_extcode
Jan 14, 2025
Merged

Implement EXTCODE* changes for pdn-5#13106
somnergy merged 34 commits intomainfrom
som/7720_extcode

Conversation

@somnergy
Copy link
Copy Markdown
Member

@somnergy somnergy commented Dec 13, 2024

See ethereum/EIPs#8969
Issue board: #12401

@somnergy somnergy marked this pull request as draft December 16, 2024 10:00
@somnergy somnergy changed the title Implement EXTCODE* changes for pdn-5 [Do-Not-Merge] Implement EXTCODE* changes for pdn-5 Dec 16, 2024
@somnergy somnergy requested a review from yperbasis December 16, 2024 14:21
if err != nil {
return nil, fmt.Errorf("%w: %w", ErrIntraBlockStateFailed, err)
}
if codeSize == types.DelegateDesignationCodeSize {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The size check is not enough. The prefix of the code should be checked as well (see ParseDelegation).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Got it.

Copy link
Copy Markdown
Member

@yperbasis yperbasis left a comment

Choose a reason for hiding this comment

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

EXTCODECOPY should be changed as well

@somnergy somnergy changed the title [Do-Not-Merge] Implement EXTCODE* changes for pdn-5 Implement EXTCODE* changes for pdn-5 Jan 9, 2025
@somnergy somnergy changed the base branch from main to som/eip7685 January 9, 2025 10:16
@somnergy
Copy link
Copy Markdown
Member Author

somnergy commented Jan 9, 2025

This is 5/6 in the chain
Next: #13346

@somnergy somnergy marked this pull request as ready for review January 13, 2025 10:26
Base automatically changed from som/eip7685 to main January 14, 2025 14:14
@somnergy somnergy enabled auto-merge (squash) January 14, 2025 15:59
@somnergy somnergy disabled auto-merge January 14, 2025 17:52
@somnergy somnergy enabled auto-merge (squash) January 14, 2025 17:52
@somnergy somnergy merged commit d1d1894 into main Jan 14, 2025
@somnergy somnergy deleted the som/7720_extcode branch January 14, 2025 18:23
somnergy added a commit that referenced this pull request Jan 15, 2025
somnergy added a commit that referenced this pull request Jan 16, 2025
@VBulikov VBulikov mentioned this pull request Feb 5, 2025
revitteth pushed a commit to 0xPolygon/cdk-erigon that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants