Remove some of the UAHF activation logic.
ClosedPublic
Actions

Authored by deadalnix on Aug 4 2017, 22:08.

Details

Reviewers

freetrader
CCulianu

Group Reviewers

Restricted Project

Commits

rSTAGING658c6282afc1: Remove some of the UAHF activation logic.
rABC658c6282afc1: Remove some of the UAHF activation logic.

Summary

It is not needed anymore and it will simplify the code.

Test Plan

Run the integration tests with the various fixes enabled ( D413, D411 and D409 ). Checks that they pass.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

deadalnix created this revision.Aug 4 2017, 22:08

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 4 2017, 22:08

Looks good. Tested and works.

This revision is now accepted and ready to land.Aug 5 2017, 06:34

CCulianu mentioned this in D414: autopep8 various files.Aug 5 2017, 06:56

CCulianu mentioned this in D413: Fix various RPC tests from the extended test suite.

CCulianu mentioned this in D411: Also add the uahfstarttime to the rpc test framework.

CCulianu mentioned this in D409: Fix a bunch of rpc tests by putting uahf start time in the future.

Closed by commit rABC658c6282afc1: Remove some of the UAHF activation logic. (authored by deadalnix, committed by deadalnix). · Explain WhyAug 5 2017, 10:00

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

qa/

rpc-tests/

abc-p2p-activation.py

26 lines

src/

validation.cpp

38 lines

Diff 1043

View Options

qa/rpc-tests/abc-p2p-activation.py