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

Infinite loop in cureLocalIntersections #89

Open
randomdude7332 opened this issue Jan 9, 2021 · 2 comments
Open

Infinite loop in cureLocalIntersections #89

randomdude7332 opened this issue Jan 9, 2021 · 2 comments
Labels

Comments

@randomdude7332
Copy link

The issue appears to be fairly simple (incorrect linked list pointers due to out of order items removal), but I'm not familiar with the code, so it might be something more serious. Patched version seem to produce correct-looking output, at least.

Diff (a PR seems rather excessive): diff.txt
And, one of the offending polygons: polygon.txt

@mourner
Copy link
Member

mourner commented Jan 11, 2021

a PR seems rather excessive

Not at all! The biggest advantage is that it will run all the tests against the fix.

@mourner mourner added the bug label Jan 11, 2021
@randomdude7332
Copy link
Author

Apologies for the delay, I don't use github so it took a while to notice. Opened a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants