Skip to content

required sigs #137

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

required sigs #137

wants to merge 3 commits into from

Conversation

@juliustayl
Copy link
Contributor

@juliustayl juliustayl commented Apr 17, 2014

wrong chunk array index used for number of required sigs
@juliustayl
required sigs
5f2b711 
wrong chunk array index used for number of required sigs
@dcousens
Copy link
Contributor

dcousens commented Apr 17, 2014

Could you please lookup the OP_x opcode instead of just doing -80.
Also, can this please be put under test.
@juliustayl
Copy link
Contributor Author

juliustayl commented Apr 17, 2014

Sure, I will submit another branch
@juliustayl juliustayl closed this Apr 17, 2014
@juliustayl juliustayl deleted the MultiSigInputScript branch April 17, 2014 05:59
@dcousens
Copy link
Contributor

dcousens commented Apr 17, 2014

@juliustayl you can just push to the existing branch and it will add the commits here.

Also, please provide meaningful commit messages in the present tense.
@juliustayl juliustayl restored the MultiSigInputScript branch April 17, 2014 06:03
@dcousens dcousens reopened this Apr 17, 2014
@dcousens
Copy link
Contributor

dcousens commented Apr 17, 2014

Please re-open when ready.
@dcousens dcousens closed this Apr 17, 2014
@dcousens dcousens mentioned this pull request Apr 17, 2014
Merged
@juliustayl juliustayl deleted the MultiSigInputScript branch April 19, 2014 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@juliustayl @dcousens