Changed prefix conversion to use saturating sub

Pietroro · Pietroro · commit e65a0e95224c · 2026-01-22T13:59:51.000Z
diff --git a/bgpsim/src/types/prefix.rs b/bgpsim/src/types/prefix.rs
@@ -653,7 +653,8 @@ impl From<i32> for SimplePrefix {
 impl From<Ipv4Addr> for SimplePrefix {
     fn from(value: Ipv4Addr) -> Self {
         let num: u32 = value.into();
-        SimplePrefix((num - (100 << 24)) >> 8)
+        // WARNING: Do not merge this before talking to Tibor, this is a hack
+        SimplePrefix(num.saturating_sub(100u32 << 24) >> 8)
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -653,7 +653,8 @@ impl From<i32> for SimplePrefix {`
`653`	`653`	`impl From<Ipv4Addr> for SimplePrefix {`
`654`	`654`	`fn from(value: Ipv4Addr) -> Self {`
`655`	`655`	`let num: u32 = value.into();`
`656`		`- SimplePrefix((num - (100 << 24)) >> 8)`
	`656`	`+ // WARNING: Do not merge this before talking to Tibor, this is a hack`
	`657`	`+ SimplePrefix(num.saturating_sub(100u32 << 24) >> 8)`
`657`	`658`	`}`
`658`	`659`	`}`
`659`	`660`