From a9c4456a544edb8f0be68bf2bd619800a3e531d3 Mon Sep 17 00:00:00 2001 From: Mikolaj Wielgus Date: Thu, 17 Jul 2025 01:42:04 +0200 Subject: [PATCH] fix(geometry/geometry): Debug-assert against failed node and edge removals --- src/geometry/geometry.rs | 18 +++--------------- 1 file changed, 3 insertions(+), 15 deletions(-) diff --git a/src/geometry/geometry.rs b/src/geometry/geometry.rs index 556e38b..2753014 100644 --- a/src/geometry/geometry.rs +++ b/src/geometry/geometry.rs @@ -302,20 +302,8 @@ impl< ); } - pub(super) fn init_bend_inner>( - &mut self, - bend: GenericIndex, - inner: BI, - ) { - self.graph.update_edge( - inner.petgraph_index(), - bend.petgraph_index(), - GeometryLabel::Outer, - ); - } - pub fn remove_primitive(&mut self, primitive: PI) { - self.graph.remove_node(primitive.petgraph_index()); + debug_assert!(self.graph.remove_node(primitive.petgraph_index()).is_some()); } pub fn move_dot(&mut self, dot: DI, to: Point) { @@ -362,7 +350,7 @@ impl< .edges_directed(bend.petgraph_index(), Incoming) .find(|edge| matches!(edge.weight(), GeometryLabel::Outer)) { - self.graph.remove_edge(old_inner_edge.id()); + debug_assert!(self.graph.remove_edge(old_inner_edge.id()).is_some()); } if let Some(new_inner) = maybe_new_inner { @@ -611,7 +599,7 @@ impl, DW, SW, BW, CW: Clone, Cel: Copy, PI: Copy, D } fn remove_compound(&mut self, compound: GenericIndex) { - self.graph.remove_node(compound.petgraph_index()); + debug_assert!(self.graph.remove_node(compound.petgraph_index()).is_some()); } fn add_to_compound(&mut self, primitive: I, entry_label: Cel, compound: GenericIndex)