-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Update codec version to the 4.1 version #2948
Changes from 7 commits
3e3c17f
ed465b5
9cb959d
fcc98e2
6b48f83
d9090f2
e82ef0b
97643b1
5356b31
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,7 +30,7 @@ use client::{ | |
}; | ||
use grandpa::{ | ||
BlockNumberOps, Equivocation, Error as GrandpaError, round::State as RoundState, | ||
voter, voter_set::VoterSet, | ||
voter, voter_set::VoterSet, HistoricalVotes, | ||
}; | ||
use runtime_primitives::generic::BlockId; | ||
use runtime_primitives::traits::{ | ||
|
@@ -636,7 +636,7 @@ where | |
round: u64, | ||
state: RoundState<Block::Hash, NumberFor<Block>>, | ||
base: (Block::Hash, NumberFor<Block>), | ||
votes: Vec<SignedMessage<Block>>, | ||
votes: &HistoricalVotes<Block::Hash, NumberFor<Block>, Self::Signature, Self::Id>, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the reason for this change? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It is an implementation of the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
) -> Result<(), Self::Error> { | ||
debug!( | ||
target: "afg", "Voter {} completed round {} in set {}. Estimate = {:?}, Finalized in round = {:?}", | ||
|
@@ -655,7 +655,7 @@ where | |
number: round, | ||
state: state.clone(), | ||
base, | ||
votes, | ||
votes: votes.seen().to_owned(), | ||
}) { | ||
let msg = "Voter completed round that is older than the last completed round."; | ||
return Err(Error::Safety(msg.to_string())); | ||
|
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.
Maybe not now but:
don't we want to put some guidelines on import names? sometimes it can be confusing. I bet you can find the crate
core/sr-primitives
along the code base with names:primitive
sr-primitives
substrate-primitives
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.
I like the idea, though it will be hard to apply strict rules everywhere because we have a lot of similar names (for example, "primitives").