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

Make parsing much more robust/relaxed #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

craigds
Copy link

@craigds craigds commented Sep 6, 2018

  • Handle whitespace consistently between WKT/WKB methods
    • strip trailing/leading whitespace
    • strip blank/whitespace lines
  • Handle multiple WKB features on separate lines
  • Allow both WKB and WKT features in the same textbox (not sure why you would, but no reason to not allow it)

* Handle whitespace consistently
    - strip trailing/leading whitespace
    - strip blank/whitespace lines
* Handle multiple WKB features on separate lines
* Allow both WKB and WKT features in the same textbox (not sure why you would, but why not)
@craigds
Copy link
Author

craigds commented Nov 15, 2018

@elpaso Any chance of merging/releasing this?

@elpaso
Copy link
Owner

elpaso commented Dec 5, 2018

@craigds sorry, didn't have time to review this yet.

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