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

Fixes #28 #29

Merged
merged 3 commits into from
Jul 31, 2016
Merged

Fixes #28 #29

merged 3 commits into from
Jul 31, 2016

Conversation

1000000000
Copy link
Collaborator

No description provided.

This reverts commit 3aa842b.

The reverted commit introduced a new bug where text separated
from the end of a debug flag by whitespace was being included
in the debug flag text when the program was parsing debug flags
@Wheatwizard
Copy link
Collaborator

Nice! Is there any reason you didn't already merge the request?

@1000000000
Copy link
Collaborator Author

I only tried it with one test case (()#dv ()#dv) and I felt that just in case anyone else had test cases they wanted to try then I should hold off on merging it until they had a chance

@Wheatwizard
Copy link
Collaborator

I didn't want to make it its own commit but that period really confused me last time and I thought it would be best to get it in now. Thank you for not being hasty about merging the pull request.

@1000000000
Copy link
Collaborator Author

Good thinking. Feel free to merge when you are ready

@1000000000 1000000000 merged commit 0e1c3ef into master Jul 31, 2016
@1000000000 1000000000 deleted the fix-28 branch July 31, 2016 13:55
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.

None yet

2 participants