Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Dec 16, 2016

This applies the floor after rounding, so we don't miss out on transactions that pay our min relay fee (or slightly more) in transaction fees.

@morcos
Copy link
Contributor

morcos commented Dec 16, 2016

Thanks for taking care of this.
utACK

@maflcko maflcko force-pushed the Mf1612-feeFilterAlways2 branch from 1f233d3 to fa16b8f Compare December 16, 2016 13:30
@maflcko maflcko added this to the 0.14.0 milestone Dec 16, 2016
@laanwj
Copy link
Member

laanwj commented Dec 16, 2016

This should fix #9364?

@laanwj
Copy link
Member

laanwj commented Dec 16, 2016

utACK fa16b8f

Merging, as it seems to make Travis happy.

@laanwj laanwj merged commit fa16b8f into bitcoin:master Dec 16, 2016
laanwj added a commit that referenced this pull request Dec 16, 2016
…ke 2)

fa16b8f If we don't allow free txs, always send a fee filter (take 2) (MarcoFalke)
@maflcko maflcko deleted the Mf1612-feeFilterAlways2 branch December 17, 2016 12:32
codablock pushed a commit to codablock/dash that referenced this pull request Jan 18, 2018
…ter (take 2)

fa16b8f If we don't allow free txs, always send a fee filter (take 2) (MarcoFalke)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
…ter (take 2)

fa16b8f If we don't allow free txs, always send a fee filter (take 2) (MarcoFalke)
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Feb 26, 2019
…ter (take 2)

fa16b8f If we don't allow free txs, always send a fee filter (take 2) (MarcoFalke)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants