cleanup: `cargo fix`, `cargo fmt` to reduce number of error messages

INSTALL.md

@@ -24,7 +24,7 @@ Change your working directory to your clone of Topola's repository:
 
 ### Command-line application
 
-Run the following to build and install Topola's command-line
+Run the following command to build and install Topola's command-line
 application:
 
     cargo install --locked --path . --features cli

src/board/board.rs

@@ -15,7 +15,6 @@ use crate::{
         Layout, NodeIndex,
     },
     math::Circle,
-    router::{navmesh::Navmesh, route::Route, Router, RouterError},
 };
 
 #[derive(Debug)]

src/layout/layout.rs

@@ -19,10 +19,7 @@ use crate::{
         wraparoundable::WraparoundableIndex,
         Drawing, Infringement, LayoutException,
     },
-    geometry::{
+    geometry::{compound::ManageCompounds, shape::MeasureLength, GenericNode},
-        compound::ManageCompounds, primitive::AccessPrimitiveShape, shape::MeasureLength,
-        GenericNode,
-    },
     graph::{GenericIndex, GetPetgraphIndex},
     layout::{
         poly::{Poly, PolyWeight},

src/router/draw.rs

@@ -1,5 +1,5 @@
 use contracts::debug_ensures;
-use geo::{EuclideanLength, Point};
+use geo::Point;
 use thiserror::Error;
 
 use crate::{

src/router/navmesh.rs

@@ -7,7 +7,7 @@ use petgraph::{
     graph::UnGraph,
     stable_graph::NodeIndex,
     visit::{
-        self, Data, EdgeRef, GraphBase, IntoEdgeReferences, IntoEdges, IntoNeighbors,
+        Data, EdgeRef, GraphBase, IntoEdgeReferences, IntoEdges, IntoNeighbors,
         IntoNodeIdentifiers, NodeIndexable,
     },
 };

src/router/route.rs

@@ -1,32 +1,9 @@
-use geo::EuclideanDistance;
-use petgraph::{
-    data::DataMap,
-    graph::{EdgeReference, NodeIndex, UnGraph},
-    visit::EdgeRef,
-};
-use thiserror::Error;
-
 use crate::{
-    drawing::{
+    drawing::{dot::FixedDotIndex, graph::PrimitiveIndex, rules::AccessRules},
-        band::BandFirstSegIndex,
+    geometry::primitive::PrimitiveShape,
-        dot::{DotIndex, FixedDotIndex},
-        graph::{MakePrimitive, PrimitiveIndex},
-        primitive::MakePrimitiveShape,
-        rules::AccessRules,
-    },
-    geometry::{
-        primitive::{AccessPrimitiveShape, PrimitiveShape},
-        shape::AccessShape,
-    },
-    graph::GetPetgraphIndex,
-    layout::Layout,
     router::{
-        astar::{astar, Astar, AstarError, AstarStatus, AstarStrategy, PathTracker},
+        astar::{Astar, AstarStatus},
-        draw::DrawException,
+        navmesh::Navmesh,
-        navmesh::{
-            BinavvertexNodeIndex, Navmesh, NavmeshEdgeReference, NavmeshError, NavvertexIndex,
-            NavvertexWeight,
-        },
         trace::Trace,
         tracer::Tracer,
         Router, RouterAstarStrategy, RouterError, RouterStatus,

src/router/router.rs

@@ -7,7 +7,7 @@ use crate::{
         band::BandFirstSegIndex,
         dot::{DotIndex, FixedDotIndex},
         graph::{MakePrimitive, PrimitiveIndex},
-        head::{GetFace, Head},
+        head::GetFace,
         primitive::MakePrimitiveShape,
         rules::AccessRules,
         Collision, Infringement, LayoutException,

src/router/tracer.rs

@@ -1,19 +1,12 @@
 use contracts::{debug_ensures, debug_requires};
-use petgraph::{
-    data::DataMap,
-    graph::{NodeIndex, UnGraph},
-};
 use thiserror::Error;
 
 use crate::{
-    drawing::{
+    drawing::{band::BandFirstSegIndex, dot::FixedDotIndex, rules::AccessRules},
-        band::BandFirstSegIndex, bend::LooseBendIndex, dot::FixedDotIndex, graph::PrimitiveIndex,
-        rules::AccessRules,
-    },
     layout::Layout,
     router::{
         draw::{Draw, DrawException},
-        navmesh::{BinavvertexNodeIndex, Navmesh, NavvertexIndex, NavvertexWeight},
+        navmesh::{Navmesh, NavvertexIndex},
         trace::Trace,
     },
 };

src/specctra/design.rs

@@ -19,7 +19,7 @@ use crate::{
         mesadata::SpecctraMesadata,
         read::{self, ListTokenizer},
         structure::{self, DsnFile, Layer, Pcb, Shape},
-        write::{self, ListWriter},
+        write::ListWriter,
     },
 };
 

tests/multilayer.rs

@@ -35,7 +35,7 @@ fn test_unrouted_lm317_breakout() {
 
 #[test]
 fn test_signal_integrity_test() {
-    let mut invoker = common::load_design_and_assert(
+    let invoker = common::load_design_and_assert(
         "tests/multilayer/data/signal_integrity_test/signal_integrity_test.dsn",
     );