Small improvements in main::CheckInputs() #5402

pull jtimon wants to merge 1 commits into bitcoin:master from jtimon:checkinputs changing 1 files +5 −10
  1. jtimon commented at 11:46 PM on December 1, 2014: contributor
    1. There are 2 variables nFees and nTxFee for no reason.
    2. "nValueIn < tx.GetValueOut()" is equivalent to "nValueIn - tx.GetValueOut() < 0".
    3. "!MoneyRange(nFees)" repeats "nFees < 0" check: replace with "nFees > MAX_MONEY".
    4. Don't call tx.GetValueOut() in 3 places.
  2. Small improvements in main::CheckInputs() 433e23b480
  3. jtimon closed this on Dec 2, 2014

  4. 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-04-17 15:15 UTC

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