-
Notifications
You must be signed in to change notification settings - Fork 299
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
unexpected seg fault during union #736
Comments
Replace: Clipper2/CPP/Clipper2Lib/src/clipper.engine.cpp Lines 2761 to 2762 in bc5855a
Clipper2/CPP/Clipper2Lib/src/clipper.engine.cpp Lines 2787 to 2788 in bc5855a
with:
|
Thank you so much, that fixed the issue! :) |
AngusJohnson
added a commit
that referenced
this issue
Dec 1, 2023
Fixed bugs in PointInPolygon benchmark testing.
Closed
jiajuncccc
added a commit
to jiajuncccc/Clipper2
that referenced
this issue
Dec 15, 2023
* remotes/origin/main: Minor code tidy Added CLIPPER2_HI_PRECISION option in CMakeLists.txt Additional minor tweaks to cpp/benchmark tests Make googletest not be a git submodule, and instead clone it separately in the CI script (AngusJohnson#745) fix compiler warning of -Wunused-but-set-parameter with -fno-exceptions (AngusJohnson#744) Additional tweaks to GetIntersectPtBenchmark.cpp Additional GetIntersectPtBenchmark tweaks Fixed incorrect code comment Added new GetIntersectPoint benchmark test Updated other benchmark tests. Code formatting only (removed trailing spaces) Fixed minor typecasting issues (AngusJohnson#727) More tweaks to PointInPolygon benchmarking. Allow using external gtest with GTest:: qualified names (AngusJohnson#737) Fixed a significant (though uncommon) bug in polygon clipping (AngusJohnson#736) Fixed bugs in PointInPolygon benchmark testing. Minor tweaks to PointInPolygon benchmark testing. Fixed a minor syntax error tripping some C++ compilers Version 1.3.0 1. Fixed a significant offsetting bug introduced in previous update (AngusJohnson#733) 2. Fixed a C++ compile error when 32bit compiling (AngusJohnson#727) 3. Minor tweak to CMakeLists.txt (C++) (AngusJohnson#728) 4. Will now offset 'flat' polygons (Disc.AngusJohnson#725) 5. Reminder: recent change in open path offsetting behaviour (AngusJohnson#707) 6. Reminder: recent changes to clipper.export.h data structures. Fixed a typo and tweaked documentation Version 1.2.4 Changes since ver 1.2.3 include: 1. Important: Offsetting open path behaviour has changed. The delta now behaves as it did in Clipper1 (See Issue AngusJohnson#707) 2. Important: Data structures of exported values in clipper.export.h have changed. 3. Minor bugfixes to PolyTree nesting (AngusJohnson#679, AngusJohnson#687) 4. Numerous minor bugfixes to polygon offsetting (AngusJohnson#703, AngusJohnson#715, AngusJohnson#724) 5. Fixed an obscure bug in polygon clipping caused by horizontal spikes (AngusJohnson#720) 6. Significantly improved documentation. Additional minor bugfix to ClipperOffset (AngusJohnson#724 & Disc.AngusJohnson#726)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
I was testing a union in the following small C++ example when I encountered an unexpected seg fault:
There are a few exactly touching vertices among the polygons, and perturbing them does resolve the issue. I tried to pinpoint the precise location of the seg fault, and the issue arises inside
ClipperBase::Split
line 2746 (e.next_in_ael->join_with = JoinWith::None;
) wheree.next_in_ael
evaluates tonullptr
thus producing the seg fault. Do you have any thoughts on why this might be the case? Thanks for your help!The text was updated successfully, but these errors were encountered: