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

Refactor the blockstore #1694

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Refactor the blockstore #1694

wants to merge 3 commits into from

Commits on Mar 3, 2023

  1. Refactor the blockstore 

    First, remove the multihash "code" from the arguments and use a plain
u64. This will let us compile the actors without importing hashing
libraries.

fixes #1238
fixes #504

Second, replace the concrete `Block` type with a trait. We can then
implement this trait for, e.g., `fvm_ipld_encoding::IpldBlock`.

Finally, make the blockstore object safe. This will let us work with the
blockstore as `&dyn Blockstore` in most cases.
    @Stebalien
    Stebalien committed Mar 3, 2023
    Configuration menu
    Copy the full SHA
    c333c02 View commit details
    Browse the repository at this point in the history

  2. Adapt encoding for blockstore changes

    @Stebalien
    Stebalien committed Mar 3, 2023
    Configuration menu
    Copy the full SHA
    f4f0f8c View commit details
    Browse the repository at this point in the history

  3. Adapt fvm to blockstore changes

    @Stebalien
    Stebalien committed Mar 3, 2023
    Configuration menu
    Copy the full SHA
    c74826b View commit details
    Browse the repository at this point in the history