[wallet] Set fLimitFree = true #8287

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:Mf1607-walletLimitFree changing 3 files +7 −7

MarcoFalke commented at 3:06 PM on June 29, 2016: member
- Free transactions is no longer a thing.
- It was removed from the GUI in 0.13 and made debug option. #7686
- I don't see a need to be susceptible to fingerprinting.
MarcoFalke added the label Wallet on Jun 29, 2016
paveljanik commented at 3:07 PM on June 29, 2016: contributor
=false?

Edit: I was too quick ;-) Under the PR name '[wallet] Set fLimitFree = true', the first hunk is
```
+    bool fLimitFree = false;
```
which is a pure refactor of the constant. Do not come confused as I was ;-)
paveljanik commented at 6:10 PM on June 29, 2016: contributor

ACK https://github.com/bitcoin/bitcoin/pull/8287/commits/fa0aa3c5b36cc7add83ab40ef9ac2a636150bc78

I think it could be more readable as two separate commits (at least I was confused ;-).
sipa commented at 12:44 PM on August 25, 2016: member

Agree, I prefer 2 commits.
[wallet] Set fLimitFree = true fa28bfa341
[rpc] rawtx: Prepare fLimitFree to make it an option fa8b02d36d
MarcoFalke force-pushed on Aug 25, 2016
paveljanik approved
paveljanik commented at 4:46 PM on September 20, 2016: contributor

ACK https://github.com/bitcoin/bitcoin/pull/8287/commits/fa8b02d36d5c75266d3c8ffb91ece761ce90e774
laanwj merged this on Oct 18, 2016
laanwj closed this on Oct 18, 2016
laanwj referenced this in commit e10af96cf4 on Oct 18, 2016
MarcoFalke deleted the branch on Oct 18, 2016
codablock referenced this in commit 4e0fb050b7 on Sep 19, 2017
codablock referenced this in commit 0dcca9b99b on Jan 12, 2018
andvgal referenced this in commit 0ce7dada69 on Jan 6, 2019
furszy referenced this in commit b554c29785 on Jul 14, 2021
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#7686 [qt] Remove 0-fee from send dialog