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

distance and max: default values/ nullable #3

Closed
riju opened this issue Jun 23, 2016 · 1 comment
Closed

distance and max: default values/ nullable #3

riju opened this issue Jun 23, 2016 · 1 comment

Comments

@riju
Copy link
Contributor

riju commented Jun 23, 2016

@dontcallmedom raised the issue if we would prefer them to be nullable or assign default values (+Infinity).
Previous version of the spec had, distance and max, set to default as +Infinity. I initially had them null as on some platforms like Mac distance and max are both not retrievable using public API.

@riju
Copy link
Contributor Author

riju commented Jun 28, 2016

I think making both distance and max non-nullable and making them default to +Infinity is better.
On platforms like Mac, where there is no way to retrieve distance and max, default value should be used. It might be better not to leak platform information this way.

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

No branches or pull requests

1 participant