rei-mirrors/Fornjot
1
0
Fork 0
mirror of https://github.com/hannobraun/Fornjot synced 2025-02-12 18:26:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Simplify CurveBoundary::normalize

Browse Source 
The additional `bool` return value was confusing and easy to misuse.
Callers that need this information can use `is_normalized`, which should
result in clearer (if a bit more verbose) code.
This commit is contained in:
Hanno Braun 2023-08-11 12:51:27 +02:00
parent bf60afae7d
commit 5fb76e8a02
2 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
crates/fj-core/src/geometry/boundary.rs
View File

@ -45,11 +45,11 @@ impl<T: CurveBoundaryElement> CurveBoundary<T> {
///
/// This can be used to compare a boundary while disregarding its direction.
#[must_use]
pub fn normalize(self) -> (Self, bool) {
pub fn normalize(self) -> Self {
if self.is_normalized() {
(self, false)
self
} else {
(self.reverse(), true)
self.reverse()
}
}
}

7
crates/fj-core/src/validate/shell.rs
View File

@ -235,7 +235,6 @@ impl ShellValidationError {
.bounding_vertices_of_edge(edge)
.expect("Expected edge to be part of shell")
.normalize()
.0
};
bounding_vertices_of(edge_a)
@ -315,8 +314,7 @@ impl ShellValidationError {
.expect(
"Cycle should provide bounds of its own half-edge",
)
.normalize()
.0;
.normalize();
let edge = (curve, bounding_vertices);
@ -396,8 +394,7 @@ impl ShellValidationError {
.expect(
"Just got edge from this cycle; must be part of it",
)
.normalize()
.0;
.normalize();
edges_by_coincidence
.entry((curve, boundary))