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 force-pushed on Jul 15, 2015
  3. dexX7 force-pushed on Jul 16, 2015
  4. 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.

  5. jgarzik commented at 12:08 pm on July 17, 2015: contributor
    ACK
  6. laanwj added the label TX fees and policy on Jul 17, 2015
  7. sipa commented at 6:31 pm on July 17, 2015: member
    Seems reasonable.
  8. paveljanik commented at 6:57 am on July 18, 2015: contributor
    ACK minor nit: one line if, please remove { and }
  9. 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
  10. dexX7 force-pushed on Jul 18, 2015
  11. dexX7 commented at 3:35 pm on July 18, 2015: contributor
    @paveljanik: sure, updated.
  12. btcdrak commented at 4:39 pm on July 18, 2015: contributor
    ACK
  13. 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.

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

  16. laanwj referenced this in commit 24ce77d775 on Jul 21, 2015
  17. zkbot referenced this in commit 9af55822fb on Feb 15, 2017
  18. zkbot referenced this in commit a7cf698873 on Mar 4, 2017
  19. furszy referenced this in commit 0724bbbad2 on Jun 28, 2020
  20. MarcoFalke 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: 2024-11-17 12:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me