policy, refactor: CFeeRate::FromSatB/FromBtcKb named constructors #20790

1. 
   jonatack commented at 9:38 pm on December 28, 2020: member

   This simple patch is the start of a series of CFeeRate refactoring changes and improvements. It contains changes from both #20391 and #20546 as well as a future pull to create estimatefeerate in sat/vB. It also resolves feedback in #20305 and #20546 to have a simpler fee rate constructor interface for most needs (BTC/kvB, sat/vB) without exposing CFeeRate internals.
2. policy: create CFeeRate::FromSatB and FromBtcKb named ctors 3abb8b7ba6
3. test: add CFeeRate::FromSatB/FromBtcKb unit test coverage 6db85fbf45
4. refactor: update rpcwallet to CFeeRate named constructors 510cc9c477
5. DrahtBot added the label RPC/REST/ZMQ on Dec 28, 2020
6. DrahtBot added the label TX fees and policy on Dec 28, 2020
7. DrahtBot added the label Wallet on Dec 28, 2020
8. 
   DrahtBot commented at 0:23 am on December 29, 2020: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #20546 (policy, wallet, refactor: check for non-representable CFeeRates by jonatack)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

9. jonatack renamed this:
   policy: create CFeeRate::FromSatB and FromBtcKb named ctors
   policy, refactor: CFeeRate::FromSatB/FromBtcKb named constructors
   on Jan 1, 2021
10. 
    jonatack commented at 1:41 pm on January 1, 2021: member

    If anyone is wondering, this “Named Constructor Idiom” is from https://isocpp.org/wiki/faq/ctors#named-ctor-idiom by Stroustrup, Cline, Sutter and Alexandrescu.
11. 
    jonatack commented at 12:16 pm on January 3, 2021: member

    Seeing no concept ACKs or other interest despite this being requested several times in review feedback, closing for now.
12. jonatack closed this on Jan 3, 2021

    ---

13. 
    michaelfolkson commented at 12:28 pm on January 3, 2021: contributor

    Seems like a clear Concept ACK to me especially if requested several times. I was unclear what was covered by the series of the PRs and what was covered by this individual PR from your initial PR description. Is “It” this PR or the series of PRs?
14. 
    jonatack commented at 3:54 pm on January 3, 2021: member

    Thanks. I’d like to work on things that people are excited about, e.g. “Strong Concept ACK” or at least a spontaneous “Concept ACK” or two. This proposal didn’t make that minimal bar. The CFeeRate class could really use some love and I have a series of good changes written for it. They just don’t seem to be compelling for reviewers at this time.
15. DrahtBot locked this on Aug 16, 2022

Contributors
jonatack DrahtBot michaelfolkson

Labels
Wallet TX fees and policy RPC/REST/ZMQ