HomePhabricator

Clear container after move

Description

Clear container after move

Summary:
Clear parents after move in
DisconnectedBlockTransactions::addForBlock to ensure a known state
before reusing the variable (the standard enforces a valid state, but
not necessarely a known & specific one).

Signed-off-by: Andres Correa Casablanca <andres@thirdhash.com>

Test Plan: Running the current tests should be enough.

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Subscribers: teamcity, schancel

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

Details

Provenance
Andres Correa Casablanca <andres@thirdhash.com>Authored on Jan 19 2019, 11:56
jasonbcoxPushed on Jan 27 2019, 20:36
Reviewer
Restricted Owners Package
Differential Revision
D2344: Clear container after move
Parents
rSTAGINGb1c3160da758: add sigcache tests
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/6959, tag: phabricator/base/6954, tag: phabricator/base/6953

Event Timeline

Andres Correa Casablanca <andres@thirdhash.com> committed rSTAGINGff5296fa2cfb: Clear container after move (authored by Andres Correa Casablanca <andres@thirdhash.com>).Jan 27 2019, 14:31