OpenCascade/occt
1
0
Fork 0
mirror of https://git.dev.opencascade.org/repos/occt.git synced 2025-04-03 17:56:21 +03:00
Code Packages Releases Activity

0025272: Regression in BRepOffsetAPI_MakePipe: invalid shape is created

Browse Source 
Fix of regressions
Test-case for issue #25272
This commit is contained in:
jgv 2014-10-02 15:35:38 +04:00 committed by bugmaster
parent 8625ef7e94
commit d3dfddaebc
2 changed files with 23 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/BRepFill/BRepFill_Sweep.cxx
View File

@ -2489,9 +2489,13 @@ BRepFill_Sweep::BRepFill_Sweep(const Handle(BRepFill_SectionLaw)& Section,
if (VEdge(isec, ipath).IsNull()) if (VEdge(isec, ipath).IsNull())
VEdge(isec, ipath) = aNewFirstEdge; VEdge(isec, ipath) = aNewFirstEdge;
else //rebuild first edge else //rebuild first edge
{
RebuildTopOrBottomEdge(aNewFirstEdge, RebuildTopOrBottomEdge(aNewFirstEdge,
TopoDS::Edge(VEdge(isec, ipath)), TopoDS::Edge(VEdge(isec, ipath)),
ReversedEdges); ReversedEdges);
if (ReversedEdges.Contains(VEdge(isec, ipath)))
StartEdges(isec).Reverse();
}
} }
else UpdateEdge(TopoDS::Edge(VEdge(isec, ipath)), else UpdateEdge(TopoDS::Edge(VEdge(isec, ipath)),
@ -2660,7 +2664,7 @@ BRepFill_Sweep::BRepFill_Sweep(const Handle(BRepFill_SectionLaw)& Section,
for (j = 1; j <= NbPath; j++) for (j = 1; j <= NbPath; j++)
Tapes(StartEdges(isec))->SetValue(6, j, myFaces->Value(isec, j)); Tapes(StartEdges(isec))->SetValue(6, j, myFaces->Value(isec, j));
TopoDS_Vertex Vfirst, Vlast; TopoDS_Vertex Vfirst, Vlast;
TopExp::Vertices(TopoDS::Edge(StartEdges(isec)), Vfirst, Vlast); TopExp::Vertices(TopoDS::Edge(StartEdges(isec)), Vfirst, Vlast, Standard_True); //with orientation
if (!Rails.IsBound(Vfirst)) if (!Rails.IsBound(Vfirst))
{ {
Handle(TopTools_HArray2OfShape) anArray = new TopTools_HArray2OfShape(1, 2, 1, NbPath+1); Handle(TopTools_HArray2OfShape) anArray = new TopTools_HArray2OfShape(1, 2, 1, NbPath+1);

18
tests/bugs/modalg_5/bug25272 Normal file
View File

@ -0,0 +1,18 @@
puts "========"
puts "OCC25272"
puts "========"
puts ""
##################################################################
# Regression in BRepOffsetAPI_MakePipe: invalid shape is created
##################################################################
restore [locate_data_file bug25272_base.brep] b
restore [locate_data_file bug25272_path.brep] p
pipe r p b
set bug_info [checkshape r]
if {[string compare $bug_info "This shape seems to be valid"] != 0} {
puts "ERROR: OCC25272 is reproduced"
puts " shape result is invalid"
}