diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-02-22 11:19:32 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-02-22 11:19:38 +0100 |
| commit | e200919cbe243133fd3b6db516263592549175ae (patch) | |
| tree | 348db8c67bc58a20e549a835ce4154ef028b5fe7 /src/Makefile.test.include | |
| parent | bc49650b7c1240b076849bd805ac10e403764c33 (diff) | |
| parent | e50a9be1540c769a99fcdc1f7a109a6bf1c7516b (diff) | |
Merge bitcoin/bitcoin#24305: Docs: [policy] Remove outdated confusing comment
e50a9be1540c769a99fcdc1f7a109a6bf1c7516b Remove outdated comment on CFeeRate (Murch)
Pull request description:
This comment described how the constructor of CFeeRate was previously indirectly used to parse fee rate arguments from RPCs. The command line input was actually in sat/vB but due to the use of AmountFromValue() it got converted to BTC/vB which then got rectified in the constructor by creating a CFeeRate from that given value and COIN as the transaction size. Since this usage pattern was removed from the codebase some months ago, the comment is now obsolete.
ACKs for top commit:
michaelfolkson:
ACK e50a9be1540c769a99fcdc1f7a109a6bf1c7516b
jonatack:
ACK e50a9be1540c769a99fcdc1f7a109a6bf1c7516b
Tree-SHA512: f17bf0baeeca85a5c7883edadd407da845f6e3af1c949e93116bd67c02e601682a5f7f1ab2497172472e3acf1c4e3c234b01161a77e7d7f028e3551da34777f0
Diffstat (limited to 'src/Makefile.test.include')
0 files changed, 0 insertions, 0 deletions