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.

Sign up for GitHub

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

Jump to bottom

Fix some regolith bugs #43

Merged
merged 3 commits into from
Sep 15, 2023
Merged

Fix some regolith bugs #43

merged 3 commits into from
Sep 15, 2023

Commits on Sep 14, 2023

  1. Use computed nonce for contract creation 

    It's possible that Deposit Txes will be contract creations.  But, the
deposit tx nonce is always zero and inferred based on the order.  To
prevent collisions we properly utilize the computed nonce.
    @jyellick
    jyellick committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    1d0a410 View commit details
    Browse the repository at this point in the history

  2. Report actual Deposit Tx gas usage for Regolith 

    Previously, the deposit tx was always assumed to use all of the GasLimit
in its processing.  Instead, Regolith requires that the actual gas usage
be reported.  Gas price is always 0 so the refund logic is unaffected.
    @jyellick
    jyellick committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    36b61bc View commit details
    Browse the repository at this point in the history

  3. Properly compute RollupDataGas for Regolith 

    The Bedrock and Regolith computation for rollup data gas differs.  This
change adds the chain rules as a parameter to the computation so that it
can be appropriately switched, depending on the chain state.
    @jyellick
    jyellick committed Sep 14, 2023
    Configuration menu
    Copy the full SHA
    78f32c0 View commit details
    Browse the repository at this point in the history