Removing size param (not used anymore) #1077

pull dr-orlovsky wants to merge 2 commits into bitcoin:master from dr-orlovsky:patch-6 changing 1 files +1 −2
  1. dr-orlovsky commented at 1:14 AM on March 6, 2021: contributor

    Text mentions size param and provides broken link to the explanation which is absent. It seems the parameter was removed after strict requirement to always use bitcoin denomination, but this line was left.

  2. Removing size param (not used anymore) a29c14f664
  3. luke-jr commented at 8:19 PM on March 15, 2021: member
  4. luke-jr added the label Proposed BIP modification on Mar 15, 2021
  5. TheBlueMatt commented at 9:42 PM on April 25, 2021: contributor

    Sure, but maybe remove the "/size" thing in the next header given I have no idea what "size" even means here.

  6. BIP-21 remove of all "size" mentions dd39e2234c
  7. dr-orlovsky commented at 8:56 PM on April 26, 2021: contributor

    @TheBlueMatt nice catch, did it

  8. TheBlueMatt approved
  9. TheBlueMatt commented at 11:45 PM on April 27, 2021: contributor

    LGTM

  10. dr-orlovsky commented at 10:34 AM on June 6, 2021: contributor

    Waiting for @tcatm review

  11. tcatm commented at 6:33 PM on June 6, 2021: none

    ACK

  12. kallewoof merged this on Jun 12, 2021
  13. kallewoof closed this on Jun 12, 2021


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-14 11:10 UTC

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