Differential D11179

[avalanche] Add helper function Proof::amountToScore().
ClosedPublic
Actions

Authored by tyler-smith on Mar 16 2022, 00:38.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABC12185af8e7b6: [avalanche] Add helper function Proof::amountToScore().

Summary

There are multiple places we want to convert an Amount of stake into a proof score, so pull that
logic into it's own helper function. Score computation logic in tested in the Avalanche unit tests,
which covers this behavior.

Test Plan

ninja check-avalanche

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tyler-smith created this revision.Mar 16 2022, 00:38

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 16 2022, 00:39

tyler-smith requested review of this revision.Mar 16 2022, 00:39

Harbormaster completed remote builds in B18498: Diff 32725.Mar 16 2022, 00:54

Fabien edited the test plan for this revision. (Show Details)Mar 16 2022, 08:02

Fabien accepted this revision.Mar 16 2022, 08:13

This revision is now accepted and ready to land.Mar 16 2022, 08:13

Closed by commit rABC12185af8e7b6: [avalanche] Add helper function Proof::amountToScore(). (authored by tyler-smith). · Explain WhyMar 16 2022, 09:22

This revision was automatically updated to reflect the committed changes.

tyler-smith added a commit: rABC12185af8e7b6: [avalanche] Add helper function Proof::amountToScore()..

Fabien added a task: Restricted Maniphest Task.Mar 18 2022, 17:19

Revision Contents
Changeset List

Path

Size

src/

avalanche/

proof.h

2 lines

proof.cpp

6 lines

Diff 32742

View Options

src/avalanche/proof.h