Fix clippy warnings

nsg-ethz · Sep 26, 2024 · ede06ee · ede06ee
1 parent 7624230
commit ede06ee
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/bgpsim-web/src/draw/canvas.rs b/bgpsim-web/src/draw/canvas.rs
@@ -208,7 +208,7 @@ fn prepare_touch() -> (Callback<TouchEvent>, Callback<TouchEvent>) {
                     n.dim.add_offset(delta);
                 });
             };
-        } else if new_touches.len() > 1 {
+        } else {
             log::debug!("multi-touch not yet supported");
         }
         *old_touches = new_touches;

diff --git a/bgpsim-web/src/sidebar/router_cfg/route_map_match_cfg.rs b/bgpsim-web/src/sidebar/router_cfg/route_map_match_cfg.rs
@@ -178,14 +178,14 @@ impl MatchValue {
             return Some(Self::Integer(x));
         }
         if let Some(vs) = s
-            .split(|c| c == ',' || c == ';')
+            .split([',', ';'])
             .map(|x| Pfx::from_str(x.trim()).ok())
             .collect::<Option<BTreeSet<Pfx>>>()
         {
             return Some(Self::PrefixList(vs));
         }
         if let Some(vs) = s
-            .split(|c| c == ',' || c == ';')
+            .split([',', ';'])
             .map(|x| x.trim().parse::<u32>().ok())
             .collect::<Option<BTreeSet<u32>>>()
         {

diff --git a/bgpsim/generate_topology_zoo.rs b/bgpsim/generate_topology_zoo.rs
@@ -53,7 +53,7 @@ fn main() {
         }
         let topo_name = file_name.trim_end_matches(".graphml");
         // check if all chars are ascii
-        if !topo_name.chars().all(|c| c.is_ascii()) {
+        if !topo_name.is_ascii() {
             continue;
         }
         // make sure the name only contains spaces, lowercase and uppercase letters, and starts

diff --git a/bgpsim/src/builder.rs b/bgpsim/src/builder.rs
@@ -798,7 +798,7 @@ impl<P: Prefix, Q: EventQueue<P>, Ospf: OspfImpl> NetworkBuilder<P, Q, Ospf>
         // build a local LUT
         let lut: HashMap<RouterId, GaoRexfordPeerType> = self
             .external_indices()
-            .map(|ext| (ext, peer_type(ext, &self, a.clone())))
+            .map(|ext| (ext, peer_type(ext, self, a.clone())))
             .collect();
 
         _build_gao_rexford(self, lut)