OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-07 18:30:55 +03:00
Code Packages Releases Activity

0029765: BOPTools_AlgoTools::MakeSplitEdge Creates Illegal Edge

Browse Source 
Method MakeSplitEdge checks arguments order.
This makes the method more generic.
Taking vertex orientation into account.
This commit is contained in:
Benjamin Bihler 2018-05-15 10:43:57 +03:00 committed by bugmaster
parent e52ba46e3b
commit 012264339e
1 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/BOPTools/BOPTools_AlgoTools_2.cxx
View File

@ -162,12 +162,29 @@ void BOPTools_AlgoTools::MakeSplitEdge(const TopoDS_Edge& aE,
// //
BRep_Builder BB; BRep_Builder BB;
if (!aV1.IsNull()) { if (!aV1.IsNull()) {
BB.Add (E, aV1); if (aP1 < aP2) {
BB.Add (E, TopoDS::Vertex(aV1.Oriented(TopAbs_FORWARD)));
}
else {
BB.Add (E, TopoDS::Vertex(aV1.Oriented(TopAbs_REVERSED)));
}
} }
if (!aV2.IsNull()) { if (!aV2.IsNull()) {
BB.Add (E, aV2); if (aP1 < aP2) {
BB.Add (E, TopoDS::Vertex(aV2.Oriented(TopAbs_REVERSED)));
}
else {
BB.Add (E, TopoDS::Vertex(aV2.Oriented(TopAbs_FORWARD)));
}
} }
BB.Range(E, aP1, aP2);
if (aP1 < aP2) {
BB.Range(E, aP1, aP2);
}
else {
BB.Range(E, aP2, aP1);
}
aNewEdge=E; aNewEdge=E;
aNewEdge.Orientation(aE.Orientation()); aNewEdge.Orientation(aE.Orientation());
} }