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