Check if point's index exists before adding it to the list of points

that need to split faces when avoiding creating degenerate faces while merging CSG faces.

Check if point's index exists before adding it to the list of points
f5b99b57 · Marcel Admiraal · 277d2f1f · f5b99b57
Commit f5b99b57 authored Jun 21, 2020 by Marcel Admiraal
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

csg.cpp modules/csg/csg.cpp +6 -2

No files found.
--- a/modules/csg/csg.cpp
+++ b/modules/csg/csg.cpp
@@ -904,8 +904,12 @@ void CSGBrushOperation::Build2DFaces::_merge_faces(const Vector<int> &p_segment_
 				vertices[p_segment_indices[closest_idx]].point
 			};
 			if (are_segements_parallel(edge1, edge2, vertex_snap2)) {
-				degenerate_points.push_back(outer_edge_idx[0]);
-				degenerate_points.push_back(outer_edge_idx[1]);
+				if (!degenerate_points.find(outer_edge_idx[0])) {
+					degenerate_points.push_back(outer_edge_idx[0]);
+				}
+				if (!degenerate_points.find(outer_edge_idx[1])) {
+					degenerate_points.push_back(outer_edge_idx[1]);
+				}
 				continue;
 			}