Skip to content
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

txscript: Separate code for standard scripts. #410

Merged
merged 1 commit into from
May 1, 2015

Conversation

davecgh
Copy link
Member

@davecgh davecgh commented Apr 30, 2015

Depends on #409.

This pull request moves all code related to standard scripts into a separate file named standard.go as well as the associated tests into standard_test.go. Since the code in address.go and address_test.go is only related to standard scripts, it has been combined into the new files and the old files deleted.

The intent here is to make it clear that the code in standard.go is not related to consensus.

@davecgh davecgh force-pushed the txscript_standard branch 2 times, most recently from 2d2ef37 to ae12c09 Compare May 1, 2015 18:17
@davecgh
Copy link
Member Author

davecgh commented May 1, 2015

Rebased and updated for relicensing.

@dajohi
Copy link
Member

dajohi commented May 1, 2015

OK

@davecgh davecgh force-pushed the txscript_standard branch 2 times, most recently from 3d50300 to 005b540 Compare May 1, 2015 20:20
This commit moves all code related to standard scripts into a separate
file named standard.go as well as the associated tests into
standard_test.go.  Since the code in address.go and address_test.go is
only related to standard scripts, it has been combined into the new
files and the old files deleted.

The intent here is to make it clear that the code in standard.go is not
related to consensus.
@conformal-deploy conformal-deploy merged commit 005b540 into btcsuite:master May 1, 2015
@davecgh davecgh deleted the txscript_standard branch May 1, 2015 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants