diff options
author | Hristo Venev <hristo@venev.name> | 2019-09-28 18:20:33 +0000 |
---|---|---|
committer | Hristo Venev <hristo@venev.name> | 2019-09-29 16:11:35 +0300 |
commit | 56d37f135536c7d17ab98c6671094925dee64a5e (patch) | |
tree | 652b972cef0c1700b4c8ee457afd7b79861f1672 /src/model | |
parent | 8e6318f29eb022126e82e0c084262c77d9ee3300 (diff) |
I don't like `if let` that much.
Diffstat (limited to 'src/model')
-rw-r--r-- | src/model/ip.rs | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/model/ip.rs b/src/model/ip.rs index c20464f..490dada 100644 --- a/src/model/ip.rs +++ b/src/model/ip.rs @@ -159,10 +159,9 @@ macro_rules! per_proto { } pub fn insert(&mut self, mut net: $nett) { - let mut i = if let Err(i) = self.nets.binary_search(&net) { - i - } else { - return; + let mut i = match self.nets.binary_search(&net) { + Err(v) => v, + Ok(_) => return, }; let mut j = i; if i != 0 && self.nets[i - 1].contains(&net) { @@ -317,10 +316,9 @@ per_proto!(Ipv4Net(Ipv4Addr; "IPv4 network"); u32(4); Ipv4Set); per_proto!(Ipv6Net(Ipv6Addr; "IPv6 network"); u128(16); Ipv6Set); fn pfx_split(s: &str) -> Result<(&str, u8), NetParseError> { - let i = if let Some(i) = s.find('/') { - i - } else { - return Err(NetParseError); + let i = match s.find('/') { + Some(v) => v, + None => return Err(NetParseError), }; let (addr, pfx) = s.split_at(i); let pfx = u8::from_str(&pfx[1..]).map_err(|_| NetParseError)?; |