Paths

Table of Contentst

Differential D6330

[backport#16129] Make reasoning about dependencies easier by not including unused dependencies
ClosedPublic
Actions

Authored by majcosta on Jun 2 2020, 19:42.

Tags

None

Subscribers

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rABCa66c80017899: [backport#16129] Make reasoning about dependencies easier by not including…

Summary

https://github.com/bitcoin/bitcoin/pull/16129/commits/eca97676737174ac6ff775c60e3798c69fb67f4a

Backport of Core PR16129

Test Plan

ninja
ninja check-all
cd ..
./test/lint/lint-circular-dependencies.sh

Diff Detail

Repository

rABC Bitcoin ABC

Branch

pr15365

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 11145
Build 20837: Default Diff Build & Tests
Build 20836: Build without Wallet
Build 20835: Build with clang-10
Build 20834: arc lint + arc unit

Event Timeline

majcosta created this revision.Jun 2 2020, 19:42

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 2 2020, 19:42

majcosta requested review of this revision.Jun 2 2020, 19:42

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

Harbormaster completed remote builds in B11145: Diff 20806.Jun 2 2020, 19:53

You're missing a change in txindex_tests.cpp
Please run the circular dependency linter as part of your test plan to check there is no circular dependency that remains from the lines removed.

src/test/coins_tests.cpp
20	?
src/test/torcontrol_tests.cpp
5	That's unexpected :)

This revision now requires changes to proceed.Jun 2 2020, 20:44

majcosta edited the test plan for this revision. (Show Details)Jun 2 2020, 20:57

majcosta edited the test plan for this revision. (Show Details)Jun 2 2020, 21:01

majcosta added inline comments.Jun 2 2020, 21:03

src/test/coins_tests.cpp
20	that was removed in D364, probably cleanup at the time.

Fabien accepted this revision.Jun 2 2020, 21:09

This revision is now accepted and ready to land.Jun 2 2020, 21:09

majcosta requested review of this revision.Jun 2 2020, 21:10

Fabien accepted this revision.Jun 2 2020, 21:11

This revision is now accepted and ready to land.Jun 2 2020, 21:11

Closed by commit rABCa66c80017899: [backport#16129] Make reasoning about dependencies easier by not including… (authored by practicalswift <practicalswift@users.noreply.github.com>, committed by majcosta). · Explain WhyJun 2 2020, 21:12

This revision was automatically updated to reflect the committed changes.

majcosta added a commit: rABCa66c80017899: [backport#16129] Make reasoning about dependencies easier by not including….

jasonbcox added a subscriber: jasonbcox.Jun 2 2020, 21:16

jasonbcox added inline comments.

src/core_read.cpp
5	This doesn't look right.

Revision Contents
Changeset List

Path

Size

src/

1 line

arith_uint256.cpp

1 line

bench/

1 line

bench_bitcoin.cpp

2 lines

1 line

crypto_hash.cpp

2 lines

duplicate_inputs.cpp

3 lines

1 line

rpc_mempool.cpp

1 line

wallet_balance.cpp

1 line

bitcoin-wallet.cpp

1 line

3 lines

consensus/

1 line

3 lines

1 line

crypto/

1 line

sha256_avx2.cpp

1 line

sha256_sse41.cpp

1 line

1 line

1 line

interfaces/

1 line

9 lines

1 line

merkleblock.cpp

1 line

2 lines

net_processing.cpp

2 lines

3 lines

policy/

3 lines

primitives/

1 line

3 lines

qt/

addressbookpage.cpp

1 line

addresstablemodel.cpp

1 line

bantablemodel.cpp

2 lines

3 lines

bitcoinunits.cpp

2 lines

clientmodel.cpp

6 lines

coincontroldialog.cpp

2 lines

peertablemodel.cpp

1 line

platformstyle.cpp

2 lines

receivecoinsdialog.cpp

2 lines

test/

addressbooktests.cpp

2 lines

3 lines

rpcnestedtests.cpp

4 lines

wallettests.cpp

1 line

transactiondesc.cpp

2 lines

transactionrecord.cpp

3 lines

transactiontablemodel.cpp

4 lines

utilitydialog.cpp

5 lines

walletmodeltransaction.cpp

1 line

rpc/

1 line

9 lines

2 lines

1 line

rawtransaction.cpp

6 lines

rawtransaction_util.cpp

1 line

2 lines

script/

1 line

1 line

2 lines

test/

coins_tests.cpp

5 lines

denialofservice_tests.cpp

1 line

multisig_tests.cpp

1 line

2 lines

script_p2sh_tests.cpp

1 line

torcontrol_tests.cpp

1 line

txindex_tests.cpp

1 line

1 line

2 lines

2 lines

ui_interface.cpp

2 lines

util/

2 lines

validationinterface.cpp

2 lines

wallet/

4 lines

2 lines

4 lines

1 line

7 lines

test/

3 lines

wallet_test_fixture.cpp

2 lines

5 lines

1 line

zmq/

zmqabstractnotifier.cpp

2 lines

zmqnotificationinterface.cpp

1 line

test/

lint/

lint-circular-dependencies.sh

6 lines

Diff 20806

src/addrman.cpp

Loading...

src/arith_uint256.cpp

Loading...

src/bench/base58.cpp

Loading...

src/bench/bench_bitcoin.cpp

Loading...

src/bench/checkqueue.cpp

Loading...

src/bench/crypto_hash.cpp

Loading...

src/bench/duplicate_inputs.cpp

Loading...

src/bench/examples.cpp

Loading...

src/bench/rpc_mempool.cpp

Loading...

src/bench/wallet_balance.cpp

Loading...

src/bitcoin-wallet.cpp

Loading...

src/bitcoind.cpp

Loading...

src/consensus/merkle.cpp

Loading...

src/core_read.cpp

Loading...

src/core_write.cpp

Loading...

src/crypto/aes.cpp

Loading...

src/crypto/sha256_avx2.cpp

Loading...

src/crypto/sha256_sse41.cpp

Loading...

src/httprpc.cpp

Loading...

src/init.cpp

Loading...

src/interfaces/node.cpp

Loading...

src/interfaces/wallet.cpp

Loading...

src/key.cpp

Loading...

src/merkleblock.cpp

Loading...

src/miner.cpp

Loading...

src/net_processing.cpp

Loading...

src/netbase.cpp

Loading...

src/policy/policy.cpp

Loading...

src/primitives/block.cpp

Loading...

src/psbt.cpp

Loading...

src/qt/addressbookpage.cpp

Loading...

src/qt/addresstablemodel.cpp

Loading...

src/qt/bantablemodel.cpp

Loading...

src/qt/bitcoin.cpp

Loading...

src/qt/bitcoinunits.cpp

Loading...

src/qt/clientmodel.cpp

Loading...

src/qt/coincontroldialog.cpp

Loading...

src/qt/peertablemodel.cpp

Loading...

src/qt/platformstyle.cpp

Loading...

src/qt/receivecoinsdialog.cpp

Loading...

src/qt/test/addressbooktests.cpp

Loading...

src/qt/test/apptests.cpp

Loading...

src/qt/test/rpcnestedtests.cpp

Loading...

src/qt/test/wallettests.cpp

Loading...

src/qt/transactiondesc.cpp

Loading...

src/qt/transactionrecord.cpp

Loading...

src/qt/transactiontablemodel.cpp

Loading...

src/qt/utilitydialog.cpp

Loading...

src/qt/walletmodeltransaction.cpp

Loading...

src/rpc/blockchain.cpp

Loading...

src/rpc/misc.cpp

Loading...

src/rpc/net.cpp

Loading...

src/rpc/protocol.cpp

Loading...

src/rpc/rawtransaction.cpp

Loading...

src/rpc/rawtransaction_util.cpp

Loading...

src/rpc/server.cpp

Loading...

src/script/script.cpp

Loading...

src/script/sigcache.cpp

Loading...

src/script/standard.cpp

Loading...

src/test/coins_tests.cpp

Loading...

src/test/denialofservice_tests.cpp

Loading...

src/test/multisig_tests.cpp

Loading...

src/test/rpc_tests.cpp

Loading...

src/test/script_p2sh_tests.cpp

Loading...

src/test/torcontrol_tests.cpp

Loading...

src/test/txindex_tests.cpp

Loading...

src/timedata.cpp

Loading...

src/txdb.cpp

Loading...

src/txmempool.cpp

Loading...

src/ui_interface.cpp

Loading...

src/util/system.cpp

Loading...

src/validationinterface.cpp

Loading...

src/wallet/db.cpp

Loading...

src/wallet/fees.cpp

Loading...

src/wallet/init.cpp

Loading...

src/wallet/rpcdump.cpp

Loading...

src/wallet/rpcwallet.cpp

Loading...

src/wallet/test/init_tests.cpp

Loading...

src/wallet/test/wallet_test_fixture.cpp

Loading...

src/wallet/wallet.cpp

Loading...

src/wallet/wallettool.cpp

Loading...

src/zmq/zmqabstractnotifier.cpp

Loading...

src/zmq/zmqnotificationinterface.cpp

Loading...

test/lint/lint-circular-dependencies.sh

Loading...