HomePhabricator

Remove redundant stopExecutor() signal

Description

Remove redundant stopExecutor() signal

Summary:
PR description:

The QThread::finished signal do this work.

This concludes backport of Core PR14250

Commit 24313fb

Depends on D7844

Test Plan:
ninja && ninja check

Run src/qt/bitcoin-qt -regtest, open the console and run some RPC commands,
make sure the shutdown works without issues.

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, Fabien

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, Fabien

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

Details

Provenance
Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>Authored on Oct 21 2018, 12:10
PiRKCommitted on Oct 9 2020, 11:53
PiRKPushed on Oct 9 2020, 11:53
Reviewer
Restricted Owners Package
Differential Revision
D7845: Remove redundant stopExecutor() signal
Parents
rABC60f09cd3c326: Use size_t in CWallet::MarkDestinationsDirty
Branches
Unknown
Tags
Unknown

Event Timeline

PiRK committed rABCb8a022d6d69d: Remove redundant stopExecutor() signal (authored by Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>).Oct 9 2020, 11:53