diff options
| author | Samuel Dobson <dobsonsa68@gmail.com> | 2020-07-12 14:31:46 +1200 |
|---|---|---|
| committer | Samuel Dobson <dobsonsa68@gmail.com> | 2020-07-12 14:42:35 +1200 |
| commit | 4db44acf2d5d3b40943c94b3960f42838255b7ad (patch) | |
| tree | 7bc20aa15a8878c2ad0ee82b8ec2e08c31098b7a /doc | |
| parent | 32302e5c88ef990b2eca0561bb6be38cdbc11ec3 (diff) | |
| parent | 08fc6f6cfc3b06fd170452a766696d7b833113fa (diff) | |
| download | bitcoin-4db44acf2d5d3b40943c94b3960f42838255b7ad.tar.xz | |
Merge #18202: refactor: consolidate sendmany and sendtoaddress code
08fc6f6cfc3b06fd170452a766696d7b833113fa [rpc] refactor: consolidate sendmany and sendtoaddress code (Sjors Provoost)
Pull request description:
I consolidated code between these two RPC calls, since `sendtoaddress` is essentially `sendmany` with 1 destination.
Unless I overlooked something, the only behaviour change is that some `sendtoaddress` error codes changed from `-4` to `-6`. The release note mentions this.
Salvaged from #18201.
ACKs for top commit:
fjahr:
Code review ACK 08fc6f6cfc3b06fd170452a766696d7b833113fa
jonatack:
ACK 08fc6f6cfc3b06fd170452a766696d7b833113fa
meshcollider:
Code review & functional test run ACK 08fc6f6cfc3b06fd170452a766696d7b833113fa
Tree-SHA512: 7b66c52fa0444a4d02fc3f81d9c2a386794d447616026a30111eda35fb46510475eea6506a9ceda00bb4e0230ebb758da5d236b3ac05c954c044fa68a1e3e909
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/release-notes-18202.md | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/doc/release-notes-18202.md b/doc/release-notes-18202.md new file mode 100644 index 0000000000..f2bd870ac4 --- /dev/null +++ b/doc/release-notes-18202.md @@ -0,0 +1,8 @@ +Low-level RPC Changes +--------------------- + +- To make RPC `sendtoaddress` more consistent with `sendmany` the following error + `sendtoaddress` codes were changed from `-4` to `-6`: + - Insufficient funds + - Fee estimation failed + - Transaction has too long of a mempool chain |