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.

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

final chain serialization bits #199

Merged
merged 4 commits into from
Sep 17, 2019
Merged

final chain serialization bits #199

merged 4 commits into from
Sep 17, 2019

Conversation

whyrusleeping
Copy link
Member

This should be the last of the bits required to have 'correct' chain seialization, Actors and Message receipts got the codegen treatment, and we switched to using the AMT instead of the sharray everywhere.

chain/sync.go Outdated
@@ -16,15 +16,15 @@ import (
"github.com/filecoin-project/go-lotus/chain/types"
"github.com/filecoin-project/go-lotus/chain/vm"
"github.com/filecoin-project/go-lotus/lib/vdf"
cbg "github.com/whyrusleeping/cbor-gen"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: import grouping

@Kubuxu Kubuxu mentioned this pull request Sep 17, 2019
whyrusleeping and others added 2 commits September 18, 2019 04:08
Co-Authored-By: Łukasz Magiera <magik6k@users.noreply.github.com>
@whyrusleeping whyrusleeping merged commit 0f92df7 into master Sep 17, 2019
@whyrusleeping whyrusleeping deleted the feat/serialization-3 branch September 17, 2019 18:23
nonsense pushed a commit that referenced this pull request Nov 6, 2020
…dd missing mocked verification calls (#199)

* warn when extracting a message from a tipset that contains
  more leading messages from the same sender, as this leads to
  nonce misalignment.
* fix mocked randomness being too short (must be 32 bytes).
* mock some further verification syscalls.
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.

2 participants