Sigops are in witnessScript, not witness program #379

pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:patch-7 changing 1 files +1 −1

instagibbs commented at 2:34 PM on April 26, 2016: member

No description provided.
Sigops are in witnessScript, not witness program fedc71ff0e
jl2012 commented at 3:10 PM on April 26, 2016: contributor

thanks. But that's tricky as there is no witnessScript for P2WPKH. Any idea?
instagibbs commented at 3:37 PM on April 26, 2016: member

Hmm, yes you're right. Perhaps just say

"while the cost of a sigop triggered during witness evaluation is 1"
jl2012 commented at 4:34 PM on April 26, 2016: contributor

oh, this conflicts with #365
luke-jr added the label Proposed BIP modification on May 1, 2016
luke-jr commented at 3:04 PM on July 15, 2016: member

@CodeShark @jl2012 @sipa
CodeShark commented at 11:08 PM on July 15, 2016: contributor

ACK
luke-jr commented at 11:44 PM on July 15, 2016: member

I'm actually not clear on how to merge this... :|
sipa commented at 11:58 PM on July 15, 2016: member

@jl2012 Is this still relevant after #365 ?
jl2012 commented at 4:19 PM on July 18, 2016: contributor

This is fixed with #416
jl2012 commented at 7:19 AM on July 22, 2016: contributor

@instagibbs @luke-jr This should be closed as fixed with #416
luke-jr closed this on Jul 22, 2016

Contributors

Labels