Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

koenlavooij
Copy link

No description provided.

@cowtowncoder
Copy link
Member

Hmmmh. I have to think about this -- it will add measurable performance overhead for all use, regardless of whether user actually needs or wants this location information.
It probably would make sense to require explicit enabling of such support, I think.

Second thing: there need to be unit tests to verify that the new functionality works.

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.

2 participants