Rename DEFAULT_BLOCK_MIN_TX_FEE for clarity
ClosedPublic
Actions

Authored by schancel on Sep 24 2018, 18:21.

Details

Reviewers

Group Reviewers

Restricted Project

Commits

rSTAGING44f7ceb5f303: Rename DEFAULT_BLOCK_MIN_TX_FEE for clarity
rABC44f7ceb5f303: Rename DEFAULT_BLOCK_MIN_TX_FEE for clarity

Summary

Rename DEFAULT_BLOCK_MIN_TX_FEE to DEFAULT_BLOCK_MIN_TX_FEE_PER_KB
for clarity.

Test Plan

make check

Diff Detail

Repository

rABC Bitcoin ABC

Branch

rename-block-min-tx-fee

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3351
Build 4785: Bitcoin ABC Buildbot (legacy)
Build 4784: arc lint + arc unit

Event Timeline

schancel created this revision.Sep 24 2018, 18:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 24 2018, 18:21

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

schancel added a child revision: D1801: Begin using billable txn size for wallet and mining.Sep 24 2018, 18:27

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing failed.

Harbormaster failed remote builds in B3351: Diff 5007!Sep 24 2018, 18:40

deadalnix accepted this revision.Sep 24 2018, 18:57

This revision is now accepted and ready to land.Sep 24 2018, 18:57

Closed by commit rABC44f7ceb5f303: Rename DEFAULT_BLOCK_MIN_TX_FEE for clarity (authored by schancel). · Explain WhySep 24 2018, 22:08

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

init.cpp

2 lines

miner.cpp

2 lines

policy/

policy.h

2 lines

test/

miner_tests.cpp

2 lines

Diff 5007

View Options

src/init.cpp

View Options

src/miner.cpp

View Options

src/policy/policy.h

View Options

Rename DEFAULT_BLOCK_MIN_TX_FEE for clarityClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 5007

src/init.cpp

src/miner.cpp

src/policy/policy.h

src/test/miner_tests.cpp

Rename DEFAULT_BLOCK_MIN_TX_FEE for clarity
ClosedPublic
Actions

Revision Contents
Changeset List