Clean up parsing of bool command line args #6125

pull morcos wants to merge 1 commits into bitcoin:master from morcos:fix-arg-parsing changing 1 files +5 −5
  1. morcos commented at 9:11 PM on May 12, 2015: member

    Some command line args (-spendzeroconfchange, -sendfreetransactions, -permitbaremultisig, and -proxyrandomize) were parsed using GetArg instead of GetBoolArg. This was causing '-spendzeroconfchange' to be interpreted as turning off the option unless you called '-spendzeroconfchange=1'.

  2. Clean up parsing of bool command line args 472b4c5dda
  3. jgarzik commented at 9:12 PM on May 12, 2015: contributor

    ut ACK

  4. laanwj added the label Improvement on May 13, 2015
  5. laanwj commented at 2:00 PM on May 13, 2015: member

    Makes sense, uACK

  6. fanquake commented at 2:04 PM on May 13, 2015: member

    utACK

  7. gavinandresen merged this on May 13, 2015
  8. gavinandresen closed this on May 13, 2015

  9. gavinandresen referenced this in commit 351f73ecd5 on May 13, 2015
  10. gavinandresen commented at 7:26 PM on May 13, 2015: contributor

    ACK (merged)

  11. laanwj commented at 9:25 AM on May 14, 2015: member

    Thanks for merging @gavinandresen

  12. 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: 2026-04-22 06:15 UTC

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