Exempt unspendable transaction outputs from dust checks #6444

pull dexX7 wants to merge 1 commits into bitcoin:master from dexX7:btc-rpc-setmintxrelayfee changing 2 files +21 −2
  1. dexX7 commented at 8:15 PM on July 15, 2015: contributor

    Since unspendable outputs can't be spent, there is no threshold at which it would be uneconomic to spend them.

    This primarily targets transaction outputs with OP_RETURN, and allows funding of data-only transactions with "fundrawtransaction".

  2. dexX7 cross-referenced this on Jul 15, 2015 from issue Add OP_RETURN support in createrawtransaction RPC call, add tests. by paveljanik
  3. dexX7 force-pushed on Jul 15, 2015
  4. dexX7 force-pushed on Jul 16, 2015
  5. petertodd commented at 5:34 AM on July 17, 2015: contributor

    utACK

    Pretty obvious fix, regardless of what we do with the IsDust() code in the future re: refactoring.

  6. jgarzik commented at 12:08 PM on July 17, 2015: contributor

    ACK

  7. laanwj added the label TX fees and policy on Jul 17, 2015
  8. sipa commented at 6:31 PM on July 17, 2015: member

    Seems reasonable.

  9. paveljanik commented at 6:57 AM on July 18, 2015: contributor

    ACK minor nit: one line if, please remove { and }

  10. Exempt unspendable transaction outputs from dust checks
    Since unspendable outputs can't be spent, there is no threshold at which it would be uneconomic to spend them.
    
    This primarily targets transaction outputs with `OP_RETURN`.
    
    ---
    
    Initially based on:
    
    commit 9cf0ae26350033d43d5dd3c95054c0d1b1641eda
    Author: zathras-crypto <zathrasc@gmail.com>
    Date:   Wed Mar 25 02:04:02 2015 -0700
    
    Changes:
    
    - cherry-picked on top of bitcoin:master
    - added RPC test for fundrawtransaction
    0aad1f13b2
  11. dexX7 force-pushed on Jul 18, 2015
  12. dexX7 commented at 3:35 PM on July 18, 2015: contributor

    @paveljanik: sure, updated.

  13. btcdrak commented at 4:39 PM on July 18, 2015: contributor

    ACK

  14. laanwj commented at 9:46 AM on July 21, 2015: member

    utACK

    minor nit: one line if, please remove { and }

    We don't actually have a coding style rule that says that.

  15. laanwj merged this on Jul 21, 2015
  16. laanwj closed this on Jul 21, 2015

  17. laanwj referenced this in commit 24ce77d775 on Jul 21, 2015
  18. str4d cross-referenced this on Feb 15, 2017 from issue Bitcoin 0.12 RPC PRs 1 by str4d
  19. zkbot referenced this in commit 9af55822fb on Feb 15, 2017
  20. daira cross-referenced this on Feb 26, 2017 from issue No dust by daira
  21. zkbot referenced this in commit a7cf698873 on Mar 4, 2017
  22. dagurval cross-referenced this on Dec 27, 2017 from issue Add fundrawtransaction by dagurval
  23. random-zebra cross-referenced this on Jun 4, 2020 from issue [Wallet][RPC] FundTransaction - fundrawtransaction by random-zebra
  24. furszy referenced this in commit 0724bbbad2 on Jun 28, 2020
  25. bitcoin locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-05-19 06:55 UTC