sipa / bitcoin

Bitcoin integration/staging tree
http://www.bitcoin.org
MIT License
88 stars 21 forks source link

Fix witness sigop counting #45

Closed afk11 closed 8 years ago

afk11 commented 8 years ago

WitnessSigOps is likely using the wrong sigop counting scheme, since there aren't version 1 witness programs any more.

This PR should be in line with the existing specification. It may not be safe to shortcut the v0/20byte special case, but can adjust this if it's a concern.

sipa commented 8 years ago

ACK, will merge when I get home

afk11 commented 8 years ago

Let me do a few more tests on this first :)

sipa commented 8 years ago

Merging.