Skip to content

Conversation

@petertodd
Copy link
Contributor

Keeps the UTXO set from being bloated by the creation of outputs that will never be spent because doing so would cost more in fees than they are worth.

Includes changes to the UI to ensure the user can't create such outputs, as well as the transaction creation code to round off change if the change txout would itself be unspendable.

Keeps the UTXO set from being bloated by the creation of outputs that
will never be spent because doing so would cost more in fees than they
are worth.

Includes changes to the UI to ensure the user can't create such outputs,
as well as the transaction creation code to round off change if the
change txout would itself be unspendable.
@BitcoinPullTester
Copy link

@gavinandresen
Copy link
Contributor

Needs unit tests and a thorough test plan. If you want this to be pulled, you'll also need a user-friendly discussion of what this changes and how it affects SatoshiDice, how it affects users who might already have coin dust in their wallets, etc.

Or, in other words: writing the code is the easy part...

@petertodd
Copy link
Contributor Author

Oh, frankly I'm surprised you think the issue is settled enough to be writing that stuff at that stage; I wasn't expecting this to be anywhere close to being pulled yet.

@gavinandresen
Copy link
Contributor

If it isn't anywhere close to being pulled, why did you submit it? I don't like "Pull Request As A Forum For Discussion" ...

@gavinandresen
Copy link
Contributor

(oh, WHY I don't like pull-request-as-a-forum-for-discussion: because it make my job harder, and wastes people time reviewing code that may never have a chance of being pulled)

@petertodd
Copy link
Contributor Author

Oh, I see you just want pull-reqs saved for code that has already had the initial discussion about it specifically done elsewhere; the pros and cons of the basic idea and varients have been discussed endlessly elsewhere.

@gavinandresen
Copy link
Contributor

Closing; please re-open when this is ready for pulling into mainline.

@jgarzik
Copy link
Contributor

jgarzik commented Apr 9, 2013

Recent data spam seems to make this worth revisiting, and re-opening.

@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.

4 participants