From 66fb1934e39f5815e98ed7bb470f5b8db1bfa101 Mon Sep 17 00:00:00 2001 From: Hanno Braun Date: Wed, 9 Nov 2022 12:03:24 +0100 Subject: [PATCH] Remove unused argument from trait method --- crates/fj-operations/src/difference_2d.rs | 10 ++++---- crates/fj-operations/src/group.rs | 7 +++--- crates/fj-operations/src/lib.rs | 28 ++++++--------------- crates/fj-operations/src/shape_processor.rs | 5 ++-- crates/fj-operations/src/sketch.rs | 3 +-- crates/fj-operations/src/sweep.rs | 5 ++-- crates/fj-operations/src/transform.rs | 5 ++-- 7 files changed, 23 insertions(+), 40 deletions(-) diff --git a/crates/fj-operations/src/difference_2d.rs b/crates/fj-operations/src/difference_2d.rs index 4c6493793..c5c5e9e30 100644 --- a/crates/fj-operations/src/difference_2d.rs +++ b/crates/fj-operations/src/difference_2d.rs @@ -5,7 +5,7 @@ use fj_kernel::{ algorithms::reverse::Reverse, iter::ObjectIters, objects::{Face, Objects, Sketch}, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::Aabb; @@ -16,7 +16,6 @@ impl Shape for fj::Difference2d { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { @@ -28,9 +27,10 @@ impl Shape for fj::Difference2d { let mut exteriors = Vec::new(); let mut interiors = Vec::new(); - let [a, b] = self.shapes().each_ref_ext().try_map_ext(|shape| { - shape.compute_brep(config, objects, debug_info) - })?; + let [a, b] = self + .shapes() + .each_ref_ext() + .try_map_ext(|shape| shape.compute_brep(objects, debug_info))?; if let Some(face) = a.face_iter().next() { // If there's at least one face to subtract from, we can proceed. diff --git a/crates/fj-operations/src/group.rs b/crates/fj-operations/src/group.rs index 271754e9d..f34be965f 100644 --- a/crates/fj-operations/src/group.rs +++ b/crates/fj-operations/src/group.rs @@ -1,7 +1,7 @@ use fj_interop::debug::DebugInfo; use fj_kernel::{ objects::{FaceSet, Objects}, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::Aabb; @@ -12,14 +12,13 @@ impl Shape for fj::Group { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { let mut faces = FaceSet::new(); - let a = self.a.compute_brep(config, objects, debug_info)?; - let b = self.b.compute_brep(config, objects, debug_info)?; + let a = self.a.compute_brep(objects, debug_info)?; + let b = self.b.compute_brep(objects, debug_info)?; faces.extend(a); faces.extend(b); diff --git a/crates/fj-operations/src/lib.rs b/crates/fj-operations/src/lib.rs index dae5496d9..a74e6107a 100644 --- a/crates/fj-operations/src/lib.rs +++ b/crates/fj-operations/src/lib.rs @@ -27,7 +27,7 @@ mod transform; use fj_interop::debug::DebugInfo; use fj_kernel::{ objects::{FaceSet, Objects, Sketch}, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::Aabb; @@ -39,7 +39,6 @@ pub trait Shape { /// Compute the boundary representation of the shape fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result; @@ -56,20 +55,16 @@ impl Shape for fj::Shape { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { match self { - Self::Shape2d(shape) => Ok(shape - .compute_brep(config, objects, debug_info)? - .faces() - .clone()), - Self::Group(shape) => { - shape.compute_brep(config, objects, debug_info) + Self::Shape2d(shape) => { + Ok(shape.compute_brep(objects, debug_info)?.faces().clone()) } + Self::Group(shape) => shape.compute_brep(objects, debug_info), Self::Sweep(shape) => Ok(shape - .compute_brep(config, objects, debug_info)? + .compute_brep(objects, debug_info)? .shells() .map(|shell| shell.faces().clone()) .reduce(|mut a, b| { @@ -77,9 +72,7 @@ impl Shape for fj::Shape { a }) .unwrap_or_default()), - Self::Transform(shape) => { - shape.compute_brep(config, objects, debug_info) - } + Self::Transform(shape) => shape.compute_brep(objects, debug_info), } } @@ -98,17 +91,12 @@ impl Shape for fj::Shape2d { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { match self { - Self::Difference(shape) => { - shape.compute_brep(config, objects, debug_info) - } - Self::Sketch(shape) => { - shape.compute_brep(config, objects, debug_info) - } + Self::Difference(shape) => shape.compute_brep(objects, debug_info), + Self::Sketch(shape) => shape.compute_brep(objects, debug_info), } } diff --git a/crates/fj-operations/src/shape_processor.rs b/crates/fj-operations/src/shape_processor.rs index 6b32cef48..87c17fd21 100644 --- a/crates/fj-operations/src/shape_processor.rs +++ b/crates/fj-operations/src/shape_processor.rs @@ -7,7 +7,7 @@ use fj_kernel::{ triangulate::Triangulate, }, objects::Objects, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::Scalar; @@ -42,10 +42,9 @@ impl ShapeProcessor { Some(user_defined_tolerance) => user_defined_tolerance, }; - let config = ValidationConfig::default(); let objects = Objects::new(); let mut debug_info = DebugInfo::new(); - let shape = shape.compute_brep(&config, &objects, &mut debug_info)?; + let shape = shape.compute_brep(&objects, &mut debug_info)?; let mesh = (&shape, tolerance).triangulate(); Ok(ProcessedShape { diff --git a/crates/fj-operations/src/sketch.rs b/crates/fj-operations/src/sketch.rs index 0d786633b..3b0573d76 100644 --- a/crates/fj-operations/src/sketch.rs +++ b/crates/fj-operations/src/sketch.rs @@ -5,7 +5,7 @@ use fj_kernel::{ builder::HalfEdgeBuilder, objects::{Cycle, Face, HalfEdge, Objects, Sketch}, partial::HasPartial, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::{Aabb, Point}; @@ -16,7 +16,6 @@ impl Shape for fj::Sketch { fn compute_brep( &self, - _: &ValidationConfig, objects: &Objects, _: &mut DebugInfo, ) -> Result { diff --git a/crates/fj-operations/src/sweep.rs b/crates/fj-operations/src/sweep.rs index 28b7b1d70..9286c3d49 100644 --- a/crates/fj-operations/src/sweep.rs +++ b/crates/fj-operations/src/sweep.rs @@ -4,7 +4,7 @@ use fj_interop::debug::DebugInfo; use fj_kernel::{ algorithms::sweep::Sweep, objects::{Objects, Solid}, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::{Aabb, Vector}; @@ -15,11 +15,10 @@ impl Shape for fj::Sweep { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { - let sketch = self.shape().compute_brep(config, objects, debug_info)?; + let sketch = self.shape().compute_brep(objects, debug_info)?; let sketch = objects.sketches.insert(sketch)?; let path = Vector::from(self.path()); diff --git a/crates/fj-operations/src/transform.rs b/crates/fj-operations/src/transform.rs index 8e18db426..5cde79679 100644 --- a/crates/fj-operations/src/transform.rs +++ b/crates/fj-operations/src/transform.rs @@ -2,7 +2,7 @@ use fj_interop::debug::DebugInfo; use fj_kernel::{ algorithms::transform::TransformObject, objects::{FaceSet, Objects}, - validate::{ValidationConfig, ValidationError}, + validate::ValidationError, }; use fj_math::{Aabb, Transform, Vector}; @@ -13,13 +13,12 @@ impl Shape for fj::Transform { fn compute_brep( &self, - config: &ValidationConfig, objects: &Objects, debug_info: &mut DebugInfo, ) -> Result { let faces = self .shape - .compute_brep(config, objects, debug_info)? + .compute_brep(objects, debug_info)? .transform(&make_transform(self), objects)?; Ok(faces)