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.

Sign up for GitHub

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

Jump to bottom

Fix for #294 - reverse search crash - the tests still pass, but library doesn't crash #295

Closed

vsajip wants to merge 3 commits into kkawakam:master from vsajip:master

Contributor

vsajip commented Oct 22, 2019

I commented out the assertion which was causing the problem reported in #294, and added the simple program which was causing the crash.

vsajip added 2 commits

October 22, 2019 16:41


          Commented out problematic assert.

73aeb1a


          Added a simple tester program.

f67653d

Collaborator

gwenn commented Oct 22, 2019

The assert should not be removed.
Would you mind testing PR #296 ?
And close this PR.
Thanks.

gwenn added the invalid label


          Uncommented the assert.

326f787

vsajip closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels