From 952c2ff8458154599c5c10df39b89b51be01ebb9 Mon Sep 17 00:00:00 2001 From: Hanno Braun Date: Fri, 29 Sep 2023 10:24:47 +0200 Subject: [PATCH] Rename variables to improve clarity, consistency --- crates/fj-core/src/algorithms/approx/curve/segment.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/crates/fj-core/src/algorithms/approx/curve/segment.rs b/crates/fj-core/src/algorithms/approx/curve/segment.rs index 3e8e43932..d3735ce36 100644 --- a/crates/fj-core/src/algorithms/approx/curve/segment.rs +++ b/crates/fj-core/src/algorithms/approx/curve/segment.rs @@ -101,18 +101,18 @@ impl CurveApproxSegment { ); assert!(other.is_normalized(), "Can't merge non-normalized segment."); - let [a_min, a_max] = self.boundary.inner; - let [b_min, b_max] = other.boundary.inner; + let [self_min, self_max] = self.boundary.inner; + let [other_min, other_max] = other.boundary.inner; - let min = cmp::min(a_min, b_min); - let max = cmp::max(a_max, b_max); + let min = cmp::min(self_min, other_min); + let max = cmp::max(self_max, other_max); self.boundary.inner = [min, max]; self.points.retain(|point| { // Only retain points that don't overlap with the other segment, or // we might end up with duplicates. - point.local_form < b_min || point.local_form > b_max + point.local_form < other_min || point.local_form > other_max }); self.points.extend(&other.points); self.points.sort();