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

Ele default value if not present #1

Open
PepijnK opened this issue Dec 12, 2016 · 2 comments
Open

Ele default value if not present #1

PepijnK opened this issue Dec 12, 2016 · 2 comments

Comments

@PepijnK
Copy link

PepijnK commented Dec 12, 2016

Thanks for your open source parser! What happens if the Ele is not present in the GPX file? I mean 0.0 elevation is a valid elevation value. I guess this should be a pointer value since it is optional in some cases?

@rndz
Copy link
Owner

rndz commented Dec 14, 2016

Thanks for pointing this out. That's a valid point, too bad I didn't think about that before hand. And I guess there might be more such cases where default value might affect it as it does with Ele value. Using pointers here is a solution, but I usually try to avoid using them in such cases, because accessing pointer variables becomes annoying. But I haven't thought of better solution to this problem at the moment.

Tho, I don't use this at the moment, so I don't know when I will get around to revisit this, to fix such issues and make it more usable.

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

3 participants
@PepijnK @rndz and others