Skip to content
This repository was archived by the owner on Jun 17, 2021. It is now read-only.

Zero address #100

Merged
holgerd77 merged 5 commits into from
Oct 24, 2017
Merged

Zero address #100

holgerd77 merged 5 commits into from
Oct 24, 2017

Conversation

@subramanianv
Copy link
Contributor

@subramanianv subramanianv commented Oct 24, 2017

closes #95

@subramanianv subramanianv mentioned this pull request Oct 24, 2017
Closed
@subramanianv
Copy link
Contributor Author

subramanianv commented Oct 24, 2017

@holgerd77 See #100

@coveralls
Copy link

coveralls commented Oct 24, 2017

Coverage Status

Coverage increased (+0.03%) to 99.16% when pulling aecb58a on subramanianv:zeroAddress into 05f0cef on ethereumjs:master.

@subramanianv
Copy link
Contributor Author

subramanianv commented Oct 24, 2017

follow up from #99

@subramanianv subramanianv mentioned this pull request Oct 24, 2017
Closed
holgerd77
holgerd77 approved these changes Oct 24, 2017
View reviewed changes
Copy link
Member

@holgerd77 holgerd77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, great, thanks!

@holgerd77 holgerd77 merged commit 926f2d7 into ethereumjs:master Oct 24, 2017
@subramanianv subramanianv deleted the zeroAddress branch October 24, 2017 20:06
@subramanianv subramanianv restored the zeroAddress branch October 24, 2017 20:06
@subramanianv subramanianv deleted the zeroAddress branch October 24, 2017 20:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@holgerd77 holgerd77 holgerd77 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Function to check if the address is a Zero Address

3 participants

@subramanianv @coveralls @holgerd77