Differential D12086

[avalanche] Make ProofPool's for each proof logic public
ClosedPublic
Actions

Authored by sdulfari on Sep 28 2022, 18:01.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC5f75c2e74f53: [avalanche] Make ProofPool's for each proof logic public

Summary

forEachProof() can be used more generically. A follow up patch will sum immature
proof stakes in the getavalancheinfo RPC.

This patch demonstrates its usage by replacing the guts of countProofs()

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pool-foreach-proof

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20226
Build 40136: Build Diff	lint-circular-dependencies · build-debug · build-clang-tidy · build-without-wallet · build-diff · build-clang
Build 40135: arc lint + arc unit

Event Timeline

sdulfari created this revision.Sep 28 2022, 18:01

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 28 2022, 18:01

sdulfari requested review of this revision.Sep 28 2022, 18:01

Harbormaster completed remote builds in B20226: Diff 35262.Sep 28 2022, 18:16

sdulfari added a child revision: D12087: [avalanche] Add immature_stake_amount to getavalancheinfo.Sep 28 2022, 18:38

Fabien accepted this revision.Sep 28 2022, 19:03

This revision is now accepted and ready to land.Sep 28 2022, 19:03

Closed by commit rABC5f75c2e74f53: [avalanche] Make ProofPool's for each proof logic public (authored by sdulfari). · Explain WhySep 28 2022, 19:07

This revision was automatically updated to reflect the committed changes.

sdulfari added a commit: rABC5f75c2e74f53: [avalanche] Make ProofPool's for each proof logic public.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

proofpool.h

12 lines

proofpool.cpp

10 lines

Diff 35262

View Options

src/avalanche/proofpool.h