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

Already on GitHub? Sign in to your account

bugfix: do not consider /32 addresses as network adresses #16

Merged
merged 1 commit into from Oct 6, 2014

Conversation

Projects
None yet
2 participants

I do not know how the network method should handle this case but network? should not return true.

since the repository seems alive again any chance for this to get merged ?
This is not a particular hard pull request to review and the behavior should look logical to anyone working with ipv4 addresses.

@bluemonk bluemonk added a commit that referenced this pull request Oct 6, 2014

@bluemonk bluemonk Merge pull request #16 from schmurfy/master
bugfix: do not consider /32 addresses as network adresses

Thanks Julien Ammous!
27a4d39

@bluemonk bluemonk merged commit 27a4d39 into ipaddress-gem:master Oct 6, 2014

schmurfy commented Oct 6, 2014

thanks :)

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