-
Notifications
You must be signed in to change notification settings - Fork 98
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
Blob data gas refunds #111
Conversation
5443ca8
to
1ff97e2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I'd consider unifying the blob basefee reader contract with the data hash reader contract but that's mostly separate.
c6a61e8
to
7e7f683
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
7e7f683
to
c5b6a78
Compare
3961315
to
0e733fc
Compare
Formatting Updates for code review Remove whitespace Formatting
…ity with block basefee
c5b6a78
to
e6871c5
Compare
Sorry for the force-push spam, should be good now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As gzeon mentioned, gas refunder in addSequencerL2BatchFromOrigin
should be updated, but there is a stack too deep error with extra arguments for the modifier.
Plausibly refundsGas(x,y,z)
could be refactored to refundsGas(x, bool refundBlob)
or simply refundsGas(x)
, refundsGasWithBlob(x)
Now refunds if there were blobs on the transaction