Skip to content

Conversation

colivi
Copy link

@colivi colivi commented Nov 12, 2016

Hello,

This is a fix for:

Thanks,
Charles

@colivi
Copy link
Author

colivi commented Nov 15, 2016

Hi,
Any chance to have this PR reviewed ? :)

@Beman
Copy link
Contributor

Beman commented Nov 23, 2016

Hi Charles,

I've fixed the problem, making good use of your test cases. I reworked the fix, not because there was necessarily anything wrong with yours, but rather than I wanted to be sure my understanding of the fix was correct.

Thanks,

--Beman

@Beman Beman closed this Nov 23, 2016
@colivi
Copy link
Author

colivi commented Nov 23, 2016

Thanks a lot Beman !!

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