Skip to content

Conversation

zauguin
Copy link
Contributor

@zauguin zauguin commented Dec 30, 2015

Since #4365 (6259937) P2SH scripts do not have to be IsStandard scripts.

Since bitcoin#4365 (6259937) P2SH scripts do not have to be IsStandard scripts.
@petertodd
Copy link
Contributor

utACK

1 similar comment
@dcousens
Copy link
Contributor

dcousens commented Jan 1, 2016

utACK

@jonasschnelli
Copy link
Contributor

ACK (why ut?)

@fanquake
Copy link
Member

fanquake commented Jan 2, 2016

ACK zauguin@6cd198f

@dcousens
Copy link
Contributor

dcousens commented Jan 2, 2016

@jonasschnelli because I haven't tested it? Sure, it should have 0 effect, but, it may not, hence, utACK.

utACK @ 6cd198f

@jonasschnelli
Copy link
Contributor

@dcousens: Yes. Sure. But I can't see how removing a comment could change the compiled binary.

@paveljanik
Copy link
Contributor

ACK

@laanwj laanwj merged commit 6cd198f into bitcoin:master Jan 7, 2016
laanwj added a commit that referenced this pull request Jan 7, 2016
6cd198f Removed comment about IsStandard for P2SH scripts (Marcel Krüger)
@zauguin zauguin deleted the zauguin-patch-1 branch January 9, 2016 16:49
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Feb 12, 2016
Since bitcoin#4365 (6259937) P2SH scripts do not have to be IsStandard scripts.
Github-Pull: bitcoin#7266
Rebased-From: 6cd198f
codablock pushed a commit to codablock/dash that referenced this pull request Dec 9, 2017
6cd198f Removed comment about IsStandard for P2SH scripts (Marcel Krüger)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants