Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Decrease COIN_DUST to 0.001 BTC, Update TX miner and relay fee defaults #2100

Closed
wants to merge 3 commits into from

Conversation

jgarzik
Copy link
Contributor

@jgarzik jgarzik commented Dec 12, 2012

  1. Create COIN_DUST constant, to represent the dust spam limit used.

  2. Decrease COIN_DUST to 0.001 BTC

Rationale: With the increase in bitcoin value (US$13.67 as of this writing), it seems reasonable to reduce the value level of which we consider "dust spam."

  1. Update TX miner and relay fee defaults to 0.001 / 0.0005 BTC respectively

Rationale: Reflects growth of dust spam in unspent transaction output dataset.

Jeff Garzik added 3 commits December 12, 2012 16:12
With the increase in bitcoin value (US$13.67 as of this writing), it seems
reasonable to reduce the level of which we consider "dust spam."
…vely

Reflects growth of dust spam in unspent transaction output dataset.
@luke-jr
Copy link
Member

luke-jr commented Dec 13, 2012

$13 is a decrease from a previous $25, which used 0.01 BTC COIN_DUST-equivalent just fine.

@gmaxwell
Copy link
Contributor

Meh. Lets have some offline discussion. Having a back and forth argument in a pull request isn't going to be as productive as chatting on IRC some, when I think there is probably already agreement on this even if we don't know it yet.

@BitcoinPullTester
Copy link

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/477acd1a11e238caf30c180c623f0999de26a2ac for binaries and test log.

@luke-jr
Copy link
Member

luke-jr commented Dec 16, 2012

FWIW, I wasn't actually making an argument against the pullreq, just posting a (devil's advocate style) comment.

@BitcoinPullTester
Copy link

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/477acd1a11e238caf30c180c623f0999de26a2ac for binaries and test log.

@jgarzik
Copy link
Contributor Author

jgarzik commented Apr 26, 2013

Superceded by #2577

@jgarzik jgarzik closed this Apr 26, 2013
@jgarzik jgarzik deleted the dust branch August 24, 2014 04:24
HashUnlimited pushed a commit to chaincoin/chaincoin that referenced this pull request Jun 6, 2018
10s denoms were ignored due to the value overflow
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 6, 2018
10s denoms were ignored due to the value overflow
sidhujag added a commit to syscoin/syscoin that referenced this pull request Jul 6, 2018
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants