Fix regression: use only valid indexes for setting a merged segment #604

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

Merged

JordanMartinez merged 2 commits into FXMisc:master from JordanMartinez:fixRegression

Oct 3, 2017

Contributor

JordanMartinez commented Oct 2, 2017

Resolves #602

JordanMartinez added 2 commits

October 2, 2017 16:22


          Use different copy/paste approach, so that thrown exception fails test

23832d7


          Fix regression: only attempt segment merge when one is already stored

a01d9c8

Contributor Author

JordanMartinez commented Oct 3, 2017

AppVeyor not yet set up: ignore the failed build.

JordanMartinez merged commit ce00638 into FXMisc:master

JordanMartinez deleted the fixRegression branch

October 3, 2017 00:03

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

Labels

None yet