Fix warnings

Author: Tom

Cargo.lock

@@ -420,5 +420,5 @@ impl<'a> Widget<'a> {
         });
     }
 
-    fn show_system_tab(&mut self, ui: &egui::Ui) {}
+    fn show_system_tab(&mut self, _ui: &egui::Ui) {}
 }

detailer/src/lib.rs

@@ -13,6 +13,4 @@ serde = { version = "1", features = ["derive"] }
 
 # For solver
 eq = {version = "0.1.0", path = "../eq"}
-#nalgebra = "0.31"
-gomez = "0.4"
 num = "0.4"

drawing/Cargo.toml

@@ -1,7 +1,6 @@
-use crate::system::{self, TermAllocator, TermRef, TermType};
+use crate::system::{TermRef, TermType};
 use crate::{Feature, FeatureKey};
 use eq::{Expression, Rational};
-use slotmap::HopSlotMap;
 
 slotmap::new_key_type! {
     pub struct ConstraintKey;
@@ -63,7 +62,7 @@ impl Constraint {
         match self {
             Fixed(..) => matches!(ft, &Feature::Point(..)),
             LineLength(..) => matches!(ft, &Feature::LineSegment(..)),
-            LineAlongCardinal(_, fk, ..) => matches!(ft, &Feature::LineSegment(..)),
+            LineAlongCardinal(..) => matches!(ft, &Feature::LineSegment(..)),
         }
     }
 
@@ -332,7 +331,7 @@ impl Constraint {
     }
 }
 
-fn distance_eq(d: &TermRef, x1: &TermRef, y1: &TermRef, x2: &TermRef, y2: &TermRef) -> Expression {
+fn distance_eq(_d: &TermRef, x1: &TermRef, y1: &TermRef, x2: &TermRef, y2: &TermRef) -> Expression {
     Expression::Sqrt(
         Box::new(Expression::Sum(
             Box::new(Expression::Power(

drawing/src/constraints.rs

@@ -221,7 +221,7 @@ impl Data {
                 }
             };
 
-            if let Some(Constraint::LineLength(_, fk, _, _, dd)) = self.constraint_mut(k) {
+            if let Some(Constraint::LineLength(_, _fk, _, _, dd)) = self.constraint_mut(k) {
                 let c = a.lerp(b, 0.5);
                 let v = c.to_vec2() - pos.to_vec2();
                 let reference = egui::Vec2::angled((a - b).angle() - v.angle()) * v.length();

drawing/src/data/mod.rs

@@ -102,19 +102,8 @@ impl Handler {
             },
             ToolResponse::NewLineCardinalConstraint(k, is_horizontal) => {
                 match drawing.features.get(k) {
-                    Some(Feature::LineSegment(_, f1, f2)) => {
-                        let (f1, f2) = (
-                            drawing.features.get(*f1).unwrap(),
-                            drawing.features.get(*f2).unwrap(),
-                        );
-                        let (p1, p2) = match (f1, f2) {
-                            (Feature::Point(_, x1, y1), Feature::Point(_, x2, y2)) => {
-                                (egui::Pos2 { x: *x1, y: *y1 }, egui::Pos2 { x: *x2, y: *y2 })
-                            }
-                            _ => panic!("unexpected subkey types: {:?} & {:?}", f1, f2),
-                        };
-
-                        let d = p1.distance(p2);
+                    Some(Feature::LineSegment(_, _f1, _f2)) => {
+                        // TODO: Delete/modify existing constraints that would clash, if any
 
                         drawing.add_constraint(Constraint::LineAlongCardinal(
                             ConstraintMeta::default(),

drawing/src/handler.rs

@@ -167,7 +167,7 @@ impl<'a> Widget<'a> {
                 (
                     Hover::Constraint {
                         k,
-                        constraint: Constraint::LineLength(_, _, _, _, dd),
+                        constraint: Constraint::LineLength(_, _, _, _, _dd),
                     },
                     false,
                     true,
@@ -224,7 +224,7 @@ impl<'a> Widget<'a> {
                     Some(Input::FeatureDrag(fk, new_pos))
                 }
 
-                (Some(DragState::Constraint(ck, offset)), _) => {
+                (Some(DragState::Constraint(ck, _offset)), _) => {
                     if released {
                         ui.memory_mut(|mem| mem.data.remove::<DragState>(select_id));
                     }
@@ -261,7 +261,7 @@ impl<'a> Widget<'a> {
         // Handle: clicks altering selection
         if hp.is_some()
             && response.clicked_by(egui::PointerButton::Primary)
-            && !matches!(current_input, Some(Input::EditingLineLength(ck)))
+            && !matches!(current_input, Some(Input::EditingLineLength(_)))
         {
             let shift_held = ui.input(|i| i.modifiers.shift);
 
@@ -541,7 +541,7 @@ pub struct DrawResponse {}
 
 #[cfg(test)]
 mod tests {
-    use super::*;
+    // use super::*;
 
     #[test]
     fn simplifications() {}