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

IP address values can be unquoted #69

Closed
wants to merge 1 commit into from
Closed

IP address values can be unquoted #69

wants to merge 1 commit into from

Conversation

ocxo
Copy link

@ocxo ocxo commented Feb 24, 2013

Is this violating the rule of "one way to do something"? IP addresses seem to be standalone data types in that they have meaning on their own so why should we treat them like strings? I can see some potential complexity in implementation. Thoughts on this?

Is this violating the rule of "one way to do something"? IP addresses seem to be standalone data types in that they have meaning on their own so why should we treat them like strings? I can see some potential complexity in implementation. Thoughts on this?
@ocxo
Copy link
Author

ocxo commented Feb 24, 2013

Related discussion in #62. Difference here being the IP address has semantic meaning rather than just being an unquoted string.

@tef
Copy link

tef commented Feb 24, 2013

Note: 127.1 is a valid IPv4 address.

@clord
Copy link

clord commented Feb 24, 2013

should include block notation (/24) and ipv6 (::1)

@mojombo
Copy link
Member

mojombo commented Feb 24, 2013

I don't think IP addresses warrant their own data type. They can be expressed quite well as strings, and many languages won't have native IP types to map to. This would also add a ton of weight to the spec to get right.

@mojombo mojombo closed this Feb 24, 2013
@ocxo ocxo deleted the patch-1 branch February 25, 2013 02:25
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