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.

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

Improve CheckInputs() comment about sig verification #7281

Merged
merged 1 commit into from Jan 19, 2016

Conversation

Projects
None yet
3 participants
@petertodd
Contributor

petertodd commented Jan 4, 2016

More clear about what the sig verification skipping optimization does, and how it could theoretically fail.

@laanwj laanwj added the Docs label Jan 4, 2016

@laanwj

This comment has been minimized.

Show comment
Hide comment
@laanwj

laanwj Jan 5, 2016

Member

ACK

Member

laanwj commented Jan 5, 2016

ACK

@instagibbs

This comment has been minimized.

Show comment
Hide comment
@instagibbs
Member

instagibbs commented Jan 11, 2016

ACK

@laanwj laanwj merged commit fd83615 into bitcoin:master Jan 19, 2016

1 check passed

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

laanwj added a commit that referenced this pull request Jan 19, 2016

Merge pull request #7281: Improve CheckInputs() comment about sig ver…
…ification

fd83615 Improve CheckInputs() comment about sig verification (Peter Todd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment