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

Parser does not parse wkt's with whitespace after first '(' #36

Open
theoffricial opened this issue Jun 4, 2019 · 4 comments · May be fixed by #42
Open

Parser does not parse wkt's with whitespace after first '(' #36

theoffricial opened this issue Jun 4, 2019 · 4 comments · May be fixed by #42

Comments

@theoffricial
Copy link

theoffricial commented Jun 4, 2019

Hey,
I have a service which gives me wkt's with some whitespaces (Do not ask why..) .
After debugging the parse method, found that white() does not match the use case when there is a whitespace after the first '(' character.
Example:
LINESTRING ( 30 10, 10 30, 40 40)
It matches first two whitespaces, not matches the whitespace after '('.
Checked for other shapes as well.

@throberto
Copy link

throberto commented Dec 22, 2020

I spent almost an hour debugging my code until I found this issue.

is this repository still maintained?

@throberto
Copy link

@tmcw do you still deal with that library?

@tmcw
Copy link
Contributor

tmcw commented Dec 22, 2020

No, I don't work at Mapbox anymore or have commit rights to this repo.

@throberto
Copy link

I understand. I will try to apply a correction for this. thanks! 😄

@jlaamanen jlaamanen linked a pull request Oct 29, 2021 that will close this issue
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 a pull request may close this issue.

3 participants