mirror of
https://github.com/hannobraun/Fornjot
synced 2025-02-26 17:15:52 +00:00
Refactor
This commit is contained in:
parent
950fd90d98
commit
65cf917fe5
@ -28,6 +28,14 @@ impl ToShape for fj::Difference2d {
|
|||||||
cycle in each operand."
|
cycle in each operand."
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
if shape.faces().all().count() != 1 {
|
||||||
|
// See issue:
|
||||||
|
// https://github.com/hannobraun/Fornjot/issues/95
|
||||||
|
todo!(
|
||||||
|
"The 2-dimensional difference operation only supports one \
|
||||||
|
face in each operand."
|
||||||
|
);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// Can't panic, as we just verified that both shapes have one cycle.
|
// Can't panic, as we just verified that both shapes have one cycle.
|
||||||
@ -41,22 +49,11 @@ impl ToShape for fj::Difference2d {
|
|||||||
}
|
}
|
||||||
|
|
||||||
{
|
{
|
||||||
let (a, b) = if a.faces().all().count() == 1
|
// Can't panic, as we just verified that both shapes have one face.
|
||||||
&& b.faces().all().count() == 1
|
let (a, b) = (
|
||||||
{
|
a.faces().all().next().unwrap(),
|
||||||
// Can't panic. We just checked that length of `a` and `b` is 1.
|
b.faces().all().next().unwrap(),
|
||||||
(
|
);
|
||||||
a.faces().all().next().unwrap(),
|
|
||||||
b.faces().all().next().unwrap(),
|
|
||||||
)
|
|
||||||
} else {
|
|
||||||
// See issue:
|
|
||||||
// https://github.com/hannobraun/Fornjot/issues/95
|
|
||||||
todo!(
|
|
||||||
"The 2-dimensional difference operation only supports one \
|
|
||||||
face in each operand."
|
|
||||||
);
|
|
||||||
};
|
|
||||||
|
|
||||||
let (a, b, surface_a, surface_b) =
|
let (a, b, surface_a, surface_b) =
|
||||||
match ((*a).clone(), (*b).clone()) {
|
match ((*a).clone(), (*b).clone()) {
|
||||||
|
Loading…
Reference in New Issue
Block a user