Added a command line option -checkscripts #7074

pull CodeShark wants to merge 1 commits into bitcoin:master from CodeShark:disable_script_checks changing 3 files +5 −2
  1. CodeShark commented at 2:27 AM on November 22, 2015: contributor

    For validation benchmarking it is often convenient to disable script checks to isolate script evaluation performance (and in particular signature verification) from other issues.

    This PR adds a command line option -checkscripts that defaults to true (preserving the original behavior) but can be set to false to shortcircuit script evaluation.

  2. CodeShark force-pushed on Nov 22, 2015
  3. Added a command line option -checkscripts which defaults to true but can be set to false for benchmarking. 065442b7b3
  4. gmaxwell commented at 2:33 AM on November 22, 2015: contributor

    NAK; sorry. Not a supported configuration, and I believe it should not be one: it has high risk of being effectively an attractive nuisance where most of the time things will be fine, but then bite you (and others). Better to just adjust the bench instrumentation to provide accounting for it, which is enough for 99% of bench purposes.

    This also bypasses all the soft-verify check stuff, like nop-non-standardness; which is a pretty dangerous thing to do; and could have bad effect network wide if some hypermiler decides to turn it on in their mining setup.

  5. CodeShark renamed this:
    Added a command line option -scriptchecks
    Added a command line option -checkscripts
    on Nov 22, 2015
  6. CodeShark closed this on Nov 22, 2015

  7. MarcoFalke locked this on Sep 8, 2021
Contributors

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-02 21:15 UTC

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