HomePhabricator

[avalanche] Use proof registration time instead of "proof time"

Description

[avalanche] Use proof registration time instead of "proof time"

Summary:
The most puzzling about the existing code isn't that someone wrote it, it is that some read it and thought "yeah, this does make sense. We have compiler time, run time, and now proof time. Let's smash that accept button."

Change the type to use seconds, because this is what the network stack uses anyways and this is where it is consumed.

Test Plan: For some reason, this feature of the PeerManager is tested in the Processor test. So the test has been updated there. In any case the test suite passes.

Reviewers: #bitcoin_abc, majcosta

Reviewed By: #bitcoin_abc, majcosta

Differential Revision: https://reviews.bitcoinabc.org/D9746

Details

Provenance
deadalnixAuthored on Jul 7 2021, 18:06
deadalnixPushed on Jul 7 2021, 19:09
Reviewer
Restricted Project
Differential Revision
D9746: [avalanche] Use proof registration time instead of "proof time"
Parents
rABC40cbef2d958e: remove boost::noncopyable dependency
Branches
Unknown
Tags
Unknown