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

Variable-sized Coin serialization #2373

Draft
wants to merge 5 commits into
base: albatross
Choose a base branch
from
Draft

Variable-sized Coin serialization #2373

wants to merge 5 commits into from
+156 −77

Commits on Apr 17, 2024

  1. Serialize Coin as a variable size int to save space

    @sisou
    sisou committed Apr 17, 2024
    Configuration menu
    Copy the full SHA
    fdd55f8 View commit details
    Browse the repository at this point in the history

  2. Add a newtype around Coin that serializes to big-endian fixint 

    To de/serialize Coins to big-endian fixint for backwards-compatible vesting contract creation data.
    @sisou
    sisou committed Apr 17, 2024
    Configuration menu
    Copy the full SHA
    8e3af6d View commit details
    Browse the repository at this point in the history

  3. Fix mempool test to check for fee_per_byte order, instead of absolute… 

    … fee
    @sisou
    sisou committed Apr 17, 2024
    Configuration menu
    Copy the full SHA
    f7e9a85 View commit details
    Browse the repository at this point in the history

  4. Ensure that transaction hashes stay the same

    @sisou
    sisou committed Apr 17, 2024
    Configuration menu
    Copy the full SHA
    b3178ea View commit details
    Browse the repository at this point in the history

  5. Update test vectors

    @sisou
    sisou committed Apr 17, 2024
    Configuration menu
    Copy the full SHA
    fc78f99 View commit details
    Browse the repository at this point in the history