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

Mapsforge maps v5 support #429

Closed
wants to merge 1 commit into from

Conversation

Gustl22
Copy link

@Gustl22 Gustl22 commented Oct 16, 2017

This is a simple implementation of v5 support. Maybe the decoding should not happen in Readbuffer.
Tags work as expected, e.g. height for extrusions. See mapsforge/mapsforge#1006

@Gustl22 Gustl22 changed the title Mapsforge-map: support of v5 maps #1006 Mapsforge-map: support of v5 maps [#1006, mapsforge] Oct 16, 2017
@Gustl22 Gustl22 changed the title Mapsforge-map: support of v5 maps [#1006, mapsforge] Mapsforge-map: support of v5 maps [https://github.com/mapsforge/mapsforge/pull/1006] Oct 16, 2017
@Gustl22 Gustl22 changed the title Mapsforge-map: support of v5 maps [https://github.com/mapsforge/mapsforge/pull/1006] Mapsforge-map: support of v5 maps https://github.com/mapsforge/mapsforge/pull/1006 Oct 16, 2017
@Gustl22 Gustl22 changed the title Mapsforge-map: support of v5 maps https://github.com/mapsforge/mapsforge/pull/1006 Mapsforge-map: support of v5 maps Oct 16, 2017
@Gustl22
Copy link
Author

Gustl22 commented Oct 16, 2017

Sorry, thought I could reference PR in title :'D

devemux86 pushed a commit that referenced this pull request Oct 17, 2017
devemux86 added a commit that referenced this pull request Oct 17, 2017
@devemux86 devemux86 added this to the 0.9.0 milestone Oct 17, 2017
@devemux86
Copy link
Collaborator

Thanks!

Merged via cd5a62a.

@devemux86 devemux86 closed this Oct 17, 2017
@devemux86 devemux86 changed the title Mapsforge-map: support of v5 maps Mapsforge maps v5 support Oct 17, 2017
@Gustl22 Gustl22 deleted the mapsforge-map-reader branch October 17, 2017 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants