Skip to content
This repository was archived by the owner on Sep 14, 2019. It is now read-only.

address issues #44 and #45#62

Merged
kevinzwang merged 2 commits intoDeepBlueRobotics:masterfrom
kevinzwang:master
Feb 10, 2018
Merged

address issues #44 and #45#62
kevinzwang merged 2 commits intoDeepBlueRobotics:masterfrom
kevinzwang:master

Conversation

@kevinzwang
Copy link
Copy Markdown
Member

change problems in script parsing from warning to error and allow
coordinate points with spaces between

change problems in script parsing from warning to error and allow
coordinate points with spaces between
Copy link
Copy Markdown
Member

@brettle brettle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty painful but it works.

Fwiw, using String.replaceAll() might have been easier.

@kevinzwang kevinzwang self-assigned this Feb 10, 2018
@kevinzwang
Copy link
Copy Markdown
Member Author

gabe concluded that replaceAll() with regex is not as good as one loop

@kevinzwang kevinzwang merged commit 869da04 into DeepBlueRobotics:master Feb 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants