berkus
/
topola
mirror of https://codeberg.org/topola/topola.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix(geometry/geometry): Rewrite uses of `debug_assert!(...)` to not break release

This commit is contained in:
Mikolaj Wielgus 2025-09-18 00:03:24 +02:00
parent ffb96ffff1
commit d708c7177a
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/geometry/geometry.rs
View File

@ -307,7 +307,8 @@ impl<
} }
pub fn remove_primitive(&mut self, primitive: PI) { pub fn remove_primitive(&mut self, primitive: PI) {
debug_assert!(self.graph.remove_node(primitive.index().into()).is_some()); let maybe_removed = self.graph.remove_node(primitive.index().into());
debug_assert!(maybe_removed.is_some());
} }
pub fn move_dot(&mut self, dot: DI, to: Point) { pub fn move_dot(&mut self, dot: DI, to: Point) {
@ -354,7 +355,8 @@ impl<
.edges_directed(bend.index().into(), Incoming) .edges_directed(bend.index().into(), Incoming)
.find(|edge| matches!(edge.weight(), GeometryLabel::Outer)) .find(|edge| matches!(edge.weight(), GeometryLabel::Outer))
{ {
debug_assert!(self.graph.remove_edge(old_inner_edge.id()).is_some()); let maybe_removed = self.graph.remove_edge(old_inner_edge.id());
debug_assert!(maybe_removed.is_some());
} }
if let Some(new_inner) = maybe_new_inner { if let Some(new_inner) = maybe_new_inner {
@ -653,7 +655,8 @@ impl<PW: Copy + Retag<Index = PI>, DW, SW, BW, CW: Clone, Cel: Copy, PI: Copy, D
} }
fn remove_compound(&mut self, compound: GenericIndex<CW>) { fn remove_compound(&mut self, compound: GenericIndex<CW>) {
debug_assert!(self.graph.remove_node(compound.index().into()).is_some()); let maybe_removed = self.graph.remove_node(compound.index().into());
debug_assert!(maybe_removed.is_some());
} }
fn add_to_compound<I>(&mut self, primitive: I, entry_label: Cel, compound: GenericIndex<CW>) fn add_to_compound<I>(&mut self, primitive: I, entry_label: Cel, compound: GenericIndex<CW>)