Differential D4790

Remove priority from CTxMemPoolEntry
ClosedPublic
Actions

Authored by deadalnix on Dec 20 2019, 17:21.

Details

Reviewers

markblundeberg

Group Reviewers

Restricted Project

Commits

rSTAGING43b4c129aac9: Remove priority from CTxMemPoolEntry
rABC43b4c129aac9: Remove priority from CTxMemPoolEntry

Summary

As per title.

Depends on D4785, D4786 and D4789

Test Plan

ninja all check-all bench-bitcoin

Diff Detail

Repository

rABC Bitcoin ABC

Branch

txmempoolprioritycleanup

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 8667
Build 15320: Default Diff Build & Tests
Build 15319: arc lint + arc unit

Event Timeline

deadalnix created this revision.Dec 20 2019, 17:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 20 2019, 17:21

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8667: Diff 15037.Dec 20 2019, 17:56

rebase

markblundeberg accepted this revision.Dec 20 2019, 23:17

This revision is now accepted and ready to land.Dec 20 2019, 23:17

Build Bitcoin-ABC / Diffs / Diff Testing started.

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B8673: Diff 15057.Dec 21 2019, 00:23

Closed by commit rABC43b4c129aac9: Remove priority from CTxMemPoolEntry (authored by deadalnix). · Explain WhyDec 21 2019, 01:56

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC43b4c129aac9: Remove priority from CTxMemPoolEntry.

markblundeberg mentioned this in D4811: finish backporting PR9602 - test_bitcoin clean up unused param.Dec 31 2019, 02:54

Mark Lundeberg <markblundeberg@users.noreply.github.com> mentioned this in rABC3b628c52ce0b: finish backporting PR9602 - test_bitcoin clean up unused param.Jan 3 2020, 01:46

deadalnix added a commit: rSTAGING43b4c129aac9: Remove priority from CTxMemPoolEntry.Apr 7 2020, 22:13

Mark Lundeberg <markblundeberg@users.noreply.github.com> mentioned this in rSTAGING3b628c52ce0b: finish backporting PR9602 - test_bitcoin clean up unused param.Apr 7 2020, 22:15

Revision Contents
Changeset List

Path

Size

src/

bench/

mempool_eviction.cpp

4 lines

rpc_mempool.cpp

9 lines

test/

mempool_tests.cpp

26 lines

miner_tests.cpp

1 line

policyestimator_tests.cpp

2 lines

8 lines

9 lines

11 lines

9 lines

7 lines

wallet/

wallet.cpp

3 lines

Diff 15037

View Options

src/bench/mempool_eviction.cpp

View Options

src/bench/rpc_mempool.cpp

View Options

src/test/mempool_tests.cpp

View Options

src/test/miner_tests.cpp

View Options

src/test/policyestimator_tests.cpp

View Options

src/test/test_bitcoin.h

View Options

src/test/test_bitcoin.cpp

View Options

src/txmempool.h

View Options

src/txmempool.cpp

View Options

src/validation.cpp

View Options

Remove priority from CTxMemPoolEntryClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 15037

src/bench/mempool_eviction.cpp

src/bench/rpc_mempool.cpp

src/test/mempool_tests.cpp

src/test/miner_tests.cpp

src/test/policyestimator_tests.cpp

src/test/test_bitcoin.h

src/test/test_bitcoin.cpp

src/txmempool.h

src/txmempool.cpp

src/validation.cpp

src/wallet/wallet.cpp

Remove priority from CTxMemPoolEntry
ClosedPublic
Actions

Revision Contents
Changeset List