Remove P2SH coinbase flag, no longer interesting #6203

Merged
merged 1 commit into from Jun 2, 2015

Conversation

Projects
None yet
5 participants
@luke-jr
Member

luke-jr commented May 31, 2015

No description provided.

@paveljanik

This comment has been minimized.

Show comment
Hide comment

utACK

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Jun 1, 2015

Member

utACK. It should be safe to remove now.

Member

laanwj commented Jun 1, 2015

utACK. It should be safe to remove now.

@laanwj laanwj added the Mining label Jun 1, 2015

@maaku

This comment has been minimized.

Show comment
Hide comment
@maaku

maaku Jun 1, 2015

Contributor

Tested ACK. That's a needlessly wasted six bytes we can recover from future coinbases.

Contributor

maaku commented Jun 1, 2015

Tested ACK. That's a needlessly wasted six bytes we can recover from future coinbases.

@gavinandresen

This comment has been minimized.

Show comment
Hide comment
@gavinandresen

gavinandresen Jun 1, 2015

Contributor

utACK. +1 for getting rid of code....

Contributor

gavinandresen commented Jun 1, 2015

utACK. +1 for getting rid of code....

@laanwj laanwj merged commit d449772 into bitcoin:master Jun 2, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request Jun 2, 2015

Merge pull request #6203
d449772 Remove P2SH coinbase flag, no longer interesting (Luke Dashjr)

laanwj added a commit that referenced this pull request Jun 2, 2015

laanwj added a commit that referenced this pull request Jun 2, 2015

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Jun 2, 2015

Member

Backported to 0.11 as c00ae64, to 0.10 as ecc96f5

Member

laanwj commented Jun 2, 2015

Backported to 0.11 as c00ae64, to 0.10 as ecc96f5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment