Wladimir J. van der Laan 6859ad2936 Merge #10706: Improve wallet fee logic and fix GUI bugs ... 11590d3 Properly bound check conf_target in wallet RPC calls (Alex Morcos) fd29d3d Remove checking of mempool min fee from estimateSmartFee. (Alex Morcos) 2fffaa9 Make QT fee displays use GetMinimumFee instead of estimateSmartFee (Alex Morcos) 1983ca6 Use CoinControl to pass custom fee setting from QT. (Alex Morcos) 03ee701 Refactor to use CoinControl in GetMinimumFee and FeeBumper (Alex Morcos) ecd81df Make CoinControl a required argument to CreateTransaction (Alex Morcos) Pull request description: This builds on #10589 (first 5 commits from that PR, last 5 commits are new) The first couple commits refactor to use the CCoinControl class to pass fee calculation parameters around. This allows for fixing the buggy interaction in QT between the global payTxFee which can be modified by the RPC call settxfee or temporarily modified by the QT custom fee settings. Before these changes the GUI could sometimes send a transaction with a recently set payTxFee and not respect the settings displayed in the GUI. After these changes, using the GUI does not involve the global transaction confirm target or payTxFee. The prospective fee displays in the smart fee slider and the coin control dialog are changed to use the fee calculation from GetMinimumFee, this simplifies the code and makes them slightly more correct in edge cases. Maxing the fee calculation with the mempool min fee is move from estimateSmartFee to GetMinimumFee. This fixes a long standing bug, and should be tagged for 0.15 as it is holding up finalizing the estimatesmartfee RPC API before release. Tree-SHA512: 4d36a1bd5934aa62f3806d380fcafbef73e9fe5bdf190fc5259a3e3a13349e5ce796e50e7068c46dc630ccf56d061bce5804f0bfe2e082bb01ca725b63efd4c1		2017-07-17 09:25:33 +02:00
..
test	Make CoinControl a required argument to CreateTransaction	2017-07-14 23:07:18 -04:00
coincontrol.h	Make QT fee displays use GetMinimumFee instead of estimateSmartFee	2017-07-14 23:41:37 -04:00
crypter.cpp	Merge #10193: scripted-diff: Remove #include <boost/foreach.hpp>	2017-07-04 18:05:18 +02:00
crypter.h	Use the override specifier (C++11) where we expect to be overriding the virtual function of a base class	2017-06-28 02:12:06 +02:00
db.cpp	Merge #10193: scripted-diff: Remove #include <boost/foreach.hpp>	2017-07-04 18:05:18 +02:00
db.h	Fix uninitialized atomic variables	2017-07-13 23:25:56 +01:00
feebumper.cpp	Refactor to use CoinControl in GetMinimumFee and FeeBumper	2017-07-14 23:10:58 -04:00
feebumper.h	Refactor to use CoinControl in GetMinimumFee and FeeBumper	2017-07-14 23:10:58 -04:00
rpcdump.cpp	Merge #10193: scripted-diff: Remove #include <boost/foreach.hpp>	2017-07-04 18:05:18 +02:00
rpcwallet.cpp	Merge #10706: Improve wallet fee logic and fix GUI bugs	2017-07-17 09:25:33 +02:00
rpcwallet.h	Fix incorrect Doxygen tag (@ince → @since). Make Doxygen parameter names match actual parameter names.	2017-07-16 21:22:05 +02:00
wallet.cpp	Merge #10706: Improve wallet fee logic and fix GUI bugs	2017-07-17 09:25:33 +02:00
wallet.h	Merge #10706: Improve wallet fee logic and fix GUI bugs	2017-07-17 09:25:33 +02:00
walletdb.cpp	Merge #10760: Avoid dereference-of-casted-pointer	2017-07-15 12:22:50 -07:00
walletdb.h	wallet: Include actual backup filename in recovery warning message	2017-06-06 21:17:09 +00:00