Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Commit

Permalink
Fix compact npos solution edge count calculation (#7021)
Browse files Browse the repository at this point in the history
This edge count is used for weighing, and it is somewhat trivial to review and verify that the current implementation was ignoring `votes16` field of the struct. As reminder, the struct is like this: 
```rust

struct Compact {
  votes1: ... ,
  votes2: ..., 
  ...,
  votes16: ...,
}
```

I already will fix this in #7007, but since it might take a while, this one can go in asap and make it to the very next runtime.
  • Loading branch information
kianenigma authored and bkchr committed Sep 18, 2020
1 parent b309241 commit 130b7d4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion primitives/npos-elections/compact/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ fn struct_def(
)
}).collect::<TokenStream2>();

let edge_count_impl = (1..count).map(|c| {
let edge_count_impl = (1..=count).map(|c| {
let field_name = field_name_for(c);
quote!(
all_edges = all_edges.saturating_add(
Expand Down

0 comments on commit 130b7d4

Please sign in to comment.