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

Tokenize whitespace strings correctly #15

Closed
wants to merge 1 commit into from

Conversation

ssured
Copy link

@ssured ssured commented Mar 14, 2013

Tokenizing " hello" (note leading space) currently results in 2 tokens: ["","hello"]. Should be ["hello"]. See PR for fix

@pangratz
Copy link

Seems like a good change! 👍

@ssured I've opened a PR to add a basic test ssured#1

@olivernn
Copy link
Owner

olivernn commented Apr 1, 2013

Thanks for this, your fix has been included in version 0.3.0

@olivernn olivernn closed this Apr 1, 2013
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.

3 participants