Skip to content

♻️ Check extcodesize #1430

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

Closed
wants to merge 6 commits into from
Closed

♻️ Check extcodesize #1430

wants to merge 6 commits into from

Conversation

Vectorized
Copy link
Owner

@Vectorized Vectorized commented Jun 8, 2025

Description

Still considering. Footgun protection.

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

Copy link

github-actions bot commented Jun 8, 2025

Gas Snapshot Comparison Report

Generated at commit : de770d5, Compared to commit : 61af38f

Contract Name Test Name Main Gas PR Gas Diff
CallContextCheckerTest test__codesize() 6364 6370 6
DeploylessPredeployQueryerTest test__codesize() 8491 8497 6
EIP712Test test__codesize() 15141 15147 6
ERC1967FactoryTest testChangeAdmin() 266261 268991 2730
testChangeAdminUnauthorized() 257261 260019 2758
testDeploy() 257129 259858 2729
testDeployAndCallWithRevert() 211629 211859 230
testProxyFails() 258426 256156 -2270
testProxySucceeds() 255451 255681 230
testUpgrade() 266604 269306 2702
testUpgradeAndCall() 355397 357932 2535
testUpgradeAndCallWithRevert() 265384 268142 2758
testUpgradeUnauthorized() 270073 272804 2731
testUpgradeWithCorruptedProxy() 263056 265814 2758
test__codesize() 32277 32283 6
ERC4337FactoryTest testCreateAccountRepeatedDeployment() 149423 149538 115
test__codesize() 12227 12256 29
ERC4337Test testInitializer() 285285 285515 230
test__codesize() 56234 56240 6
ERC7821Test test__codesize() 12738 12744 6
LibCloneTest testClone() 97792 97907 115
testCloneDeterministic() 98708 98823 115
testCloneDeterministicWithImmutableArgs() 130890 131216 326
testCloneWithImmutableArgs() 131399 131725 326
testCloneWithImmutableArgsSlicing() 58012 58127 115
testDeployDeterministicERC1967() 133882 133997 115
testDeployDeterministicERC1967I() 129900 130015 115
testDeployDeterministicERC1967IWithImmutableArgs() 129921 130036 115
testDeployDeterministicERC1967WithImmutableArgs() 172753 173079 326
testDeployERC1967() 131045 131160 115
testDeployERC1967BeaconProxyWithImmutableArgs() 946731 4146124 3199393
testDeployERC1967I() 130117 130232 115
testDeployERC1967IBeaconProxy() 359614 359729 115
testDeployERC1967IBeaconProxyWithImmutableArgs() 960319 973957 13638
testDeployERC1967IWithImmutableArgs() 169147 169262 115
testDeployERC1967WithImmutableArgs() 147374 147700 326
testImplemenationOfGas() 631821 632946 1125
testInitialDeposit() 872427 874037 1610
test__codesize() 46694 47263 569
LibZipTest testCdFallback() 2342473 2344018 1545
testFlzCompressDecompress() 1209087 1209922 835
testFlzCompressDecompress2() 769230 718552 -50678
test__codesize() 24196 24214 18
LifebuoyTest test__codesize() 43390 43396 6
UpgradeableBeaconTest test__codesize() 8511 8517 6
UUPSUpgradeableTest test__codesize() 5428 5434 6

@Vectorized Vectorized closed this Jun 8, 2025
@Vectorized Vectorized reopened this Jun 9, 2025
@Vectorized Vectorized closed this Jun 9, 2025
@Vectorized Vectorized deleted the check-extcodesize branch June 9, 2025 09:20
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