Page MenuHomePhabricator
Differential D9603

[refactor] get ValueFromAmount via an implicit Amount::operator UniValue()
ClosedPublic
Actions

Authored by majcosta on May 28 2021, 15:22.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABCb789fff5c597: [refactor] get ValueFromAmount via an implicit Amount::operator UniValue()
Summary

the number of decimals of precision that an Amount represents is currently all over the place on the codebase, this is one step to getting it all under one roof.

this seems more correct as well, representing as a UniValue is the Amount's job, not some free function. also less verbose code.

AmountFromValue probably should be an Amount constructor next.

Test Plan
ninja check-all

Diff Detail

Event Timeline

majcosta created this revision.May 28 2021, 15:22
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 28 2021, 15:22
majcosta requested review of this revision.May 28 2021, 15:22
majcosta planned changes to this revision.May 28 2021, 15:41

fix oversight on test cases

Harbormaster completed remote builds in B15847: Diff 28666.May 28 2021, 15:43
majcosta requested review of this revision.May 28 2021, 15:50

nvm

majcosta updated this revision to Diff 28670.May 28 2021, 19:44

remove spurious include in amount_tests.cpp

PiRK accepted this revision.May 28 2021, 19:54
This revision is now accepted and ready to land.May 28 2021, 19:54
Harbormaster completed remote builds in B15850: Diff 28670.May 28 2021, 19:57
Closed by commit rABCb789fff5c597: [refactor] get ValueFromAmount via an implicit Amount::operator UniValue() (authored by majcosta). · Explain WhyMay 28 2021, 20:10
This revision was automatically updated to reflect the committed changes.
majcosta added a commit: rABCb789fff5c597: [refactor] get ValueFromAmount via an implicit Amount::operator UniValue().
PiRK mentioned this in D11563: remove unneeded includes of core_io.h and network.h.Jun 3 2022, 10:43
PiRK mentioned this in rABCc747d3f9db61: remove unneeded includes of core_io.h and network.h.Jun 7 2022, 08:25

Revision Contents
Changeset List

PathSize
src/
7 lines
10 lines
1 line
11 lines
2 lines
rpc/
2 lines
60 lines
2 lines
7 lines
8 lines
test/
2 lines
fuzz/
5 lines
2 lines
64 lines
2 lines
util/
2 lines
wallet/
2 lines
73 lines

Diff 28666

View Options

src/amount.h

Loading...
View Options

src/amount.cpp

Loading...
View Options

src/core_io.h

Loading...
View Options

src/core_write.cpp

Loading...
View Options

src/rest.cpp

Loading...
View Options

src/rpc/avalanche.cpp

Loading...
View Options

src/rpc/blockchain.cpp

Loading...
View Options

src/rpc/mining.cpp

Loading...
View Options

src/rpc/net.cpp

Loading...
View Options

src/rpc/rawtransaction.cpp

Loading...
View Options

src/test/amount_tests.cpp

Loading...
View Options

src/test/fuzz/integer.cpp

Loading...
View Options

src/test/fuzz/transaction.cpp

Loading...
View Options

src/test/rpc_tests.cpp

Loading...
View Options

src/test/script_tests.cpp

Loading...
View Options

src/util/moneystr.h

Loading...
View Options

src/wallet/rpcdump.cpp

Loading...
View Options

src/wallet/rpcwallet.cpp

Loading...