From 4e76d93bf1fe06ae2256d577828f4da7f6d27f0e Mon Sep 17 00:00:00 2001 From: omy Date: Fri, 16 Nov 2012 13:30:31 +0400 Subject: [PATCH] 0023132: Suspicious code snippets 1) Warning in Aspect.cxx couldn't be reproduced 2) Description of changes: added 'return' statements into ReadStep(...) functions of RWStepBasic_* classes. 3) BRepFill_OffsetWire.cxx - removed dead code; 4) IFSelect_WorkSession.cxx - swapped arguments; 5) TopOpeBRep_ShapeIntersector2d.cxx - removed 'brake' statement and changed loop to if-statement because of void TopOpeBRep_ShapeIntersector2d::FindFFIntersection() function's call features. 6) V2d_RectangularGraphicGrid.cxx - left constants instead of functions beacuse of faster perfomance. 7) Commented unreachable code in files GeomFill_LocationGuide.cxx and TopOpeBRepTool_mkTondgE.cxx --- src/BRepFill/BRepFill_OffsetWire.cxx | 29 ------------------- src/GeomFill/GeomFill_LocationGuide.cxx | 4 +-- src/IFSelect/IFSelect_WorkSession.cxx | 2 +- src/IntTools/IntTools_EdgeFace.cxx | 5 ++-- .../RWStepBasic_RWSiUnitAndAreaUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndLengthUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndMassUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndPlaneAngleUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndRatioUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndSolidAngleUnit.cxx | 20 +++++++++---- ...WSiUnitAndThermodynamicTemperatureUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndTimeUnit.cxx | 20 +++++++++---- .../RWStepBasic_RWSiUnitAndVolumeUnit.cxx | 20 +++++++++---- .../TopOpeBRep_ShapeIntersector2d.cxx | 4 +-- .../TopOpeBRepTool_mkTondgE.cxx | 6 ++-- 15 files changed, 146 insertions(+), 84 deletions(-) diff --git a/src/BRepFill/BRepFill_OffsetWire.cxx b/src/BRepFill/BRepFill_OffsetWire.cxx index 75673c6673..d83fbae172 100755 --- a/src/BRepFill/BRepFill_OffsetWire.cxx +++ b/src/BRepFill/BRepFill_OffsetWire.cxx @@ -1619,35 +1619,6 @@ Standard_Integer CutEdge (const TopoDS_Edge& E, if (CT2d->BasisCurve()->IsKind(STANDARD_TYPE(Geom2d_Circle)) && ( Abs(f-l) >= M_PI) ) { return 0; - //--------------------------- - // Cut closed circle. - //--------------------------- - Standard_Real m = (f + l)*0.5; -// Modified by Sergey KHROMOV - Wed Mar 6 17:37:28 2002 Begin - gp_Pnt P = aC->Value(m); - - VL = BRepLib_MakeVertex(P); - B.UpdateVertex(VL, aTol); -// Modified by Sergey KHROMOV - Wed Mar 6 17:37:30 2002 End - TopoDS_Shape aLocalShape = E.EmptyCopied(); - TopoDS_Edge FE = TopoDS::Edge(aLocalShape); - TopoDS_Edge LE = TopoDS::Edge(aLocalShape); -// TopoDS_Edge FE = TopoDS::Edge(E.EmptyCopied()); -// TopoDS_Edge LE = TopoDS::Edge(E.EmptyCopied()); - FE.Orientation(TopAbs_FORWARD); - LE.Orientation(TopAbs_FORWARD); - B.Add (FE,V1); - B.Add (FE,VL.Oriented(TopAbs_REVERSED)); - B.Range(FE, f, m); - B.Add (LE,V2); - B.Add (LE,VL.Oriented(TopAbs_FORWARD)); - B.Range(LE, m, l); - Cuts.Append(FE.Oriented(E.Orientation())); - Cuts.Append(LE.Oriented(E.Orientation())); - //-------- - // Return. - //-------- - return 2; } //------------------------- diff --git a/src/GeomFill/GeomFill_LocationGuide.cxx b/src/GeomFill/GeomFill_LocationGuide.cxx index 50fa4a5010..2a548c6806 100755 --- a/src/GeomFill/GeomFill_LocationGuide.cxx +++ b/src/GeomFill/GeomFill_LocationGuide.cxx @@ -743,7 +743,7 @@ static void InGoodPeriod(const Standard_Real Prec, if (rotation) { return Standard_False; - + /* #ifdef DEB Standard_Real U = myFirstS + ratio*(Param-myCurve->FirstParameter()); #else @@ -862,7 +862,7 @@ static void InGoodPeriod(const Standard_Real Prec, myStatus = GeomFill_ImpossibleContact; return Standard_False; } - +*/ }//if_rotation diff --git a/src/IFSelect/IFSelect_WorkSession.cxx b/src/IFSelect/IFSelect_WorkSession.cxx index a413f11e19..537368a173 100755 --- a/src/IFSelect/IFSelect_WorkSession.cxx +++ b/src/IFSelect/IFSelect_WorkSession.cxx @@ -2822,7 +2822,7 @@ void IFSelect_WorkSession::QueryCheckList (const Interface_CheckIterator& chl) { if (!IsLoaded()) return; Standard_Integer i,nb = themodel->NbEntities(); - thecheckana = TCollection_AsciiString (' ',nb+1); + thecheckana = TCollection_AsciiString (nb+1,' '); for (chl.Start(); chl.More(); chl.Next()) { Standard_Integer num = chl.Number(); const Handle(Interface_Check) ach = chl.Value(); diff --git a/src/IntTools/IntTools_EdgeFace.cxx b/src/IntTools/IntTools_EdgeFace.cxx index 7952ffe24c..b0a4912cb0 100755 --- a/src/IntTools/IntTools_EdgeFace.cxx +++ b/src/IntTools/IntTools_EdgeFace.cxx @@ -898,10 +898,10 @@ void IntTools_EdgeFace::CheckData() aCommonPrt.SetVertexParameter1(tm); aCommonPrt.SetRange1 (af1, al1); } - return 0; + return 0; } - // + /* dt=al1-af1; if (dt<1.e-5) { gp_Pnt aPF, aPL; @@ -940,6 +940,7 @@ void IntTools_EdgeFace::CheckData() aCommonPrt.SetType(TopAbs_EDGE); } } + */ return 0; } diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndAreaUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndAreaUnit.cxx index 60d95d2f82..76c61b5b37 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndAreaUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndAreaUnit.cxx @@ -54,19 +54,29 @@ void RWStepBasic_RWSiUnitAndAreaUnit::ReadStep(const Handle(StepData_StepReaderD if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (prefix) is not an enumeration"); } StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #3 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #3 (name) is not an enumeration"); ent->Init(hasAprefix,aPrefix,aName); ent->SetDimensions(aDimensions); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndLengthUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndLengthUnit.cxx index 4217aa541e..e736148262 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndLengthUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndLengthUnit.cxx @@ -66,20 +66,30 @@ void RWStepBasic_RWSiUnitAndLengthUnit::ReadStep(const Handle(StepData_StepReade if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); //--- Initialisation of the red entity --- ent->Init(hasAprefix,aPrefix,aName); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndMassUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndMassUnit.cxx index 599531fa9f..fa997df5b3 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndMassUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndMassUnit.cxx @@ -75,20 +75,30 @@ void RWStepBasic_RWSiUnitAndMassUnit::ReadStep(const Handle(StepData_StepReaderD if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); //--- Initialisation of the red entity --- ent->Init(hasAprefix,aPrefix,aName); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndPlaneAngleUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndPlaneAngleUnit.cxx index e0da75f877..0e7d46f1d1 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndPlaneAngleUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndPlaneAngleUnit.cxx @@ -59,20 +59,30 @@ void RWStepBasic_RWSiUnitAndPlaneAngleUnit::ReadStep(const Handle(StepData_StepR if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); //--- Initialisation of the red entity --- ent->Init(hasAprefix,aPrefix,aName); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndRatioUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndRatioUnit.cxx index 7e995e908e..6c359aec13 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndRatioUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndRatioUnit.cxx @@ -58,20 +58,30 @@ void RWStepBasic_RWSiUnitAndRatioUnit::ReadStep (const Handle(StepData_StepReade if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #1 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #1 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); //--- Initialisation of the red entity --- ent->Init(hasAprefix,aPrefix,aName); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndSolidAngleUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndSolidAngleUnit.cxx index 9b4aae2ec0..ee36aaa309 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndSolidAngleUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndSolidAngleUnit.cxx @@ -55,20 +55,30 @@ void RWStepBasic_RWSiUnitAndSolidAngleUnit::ReadStep (const Handle(StepData_Step if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #1 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #1 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); // --- Instance of plex componant SolidAngleUnit --- num = data->NextForComplex(num); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndThermodynamicTemperatureUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndThermodynamicTemperatureUnit.cxx index f10029a516..656af63e02 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndThermodynamicTemperatureUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndThermodynamicTemperatureUnit.cxx @@ -67,20 +67,30 @@ void RWStepBasic_RWSiUnitAndThermodynamicTemperatureUnit::ReadStep if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #1 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #1 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); // --- Instance of plex componant SolidAngleUnit --- num = data->NextForComplex(num); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndTimeUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndTimeUnit.cxx index 4177eb19bb..1a9db85bb8 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndTimeUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndTimeUnit.cxx @@ -66,20 +66,30 @@ void RWStepBasic_RWSiUnitAndTimeUnit::ReadStep (const Handle(StepData_StepReader if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #1 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #1 (prefix) is not an enumeration"); } // --- field : name --- StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (name) is not an enumeration"); // --- Instance of plex componant TimeUnit --- num = data->NextForComplex(num); diff --git a/src/RWStepBasic/RWStepBasic_RWSiUnitAndVolumeUnit.cxx b/src/RWStepBasic/RWStepBasic_RWSiUnitAndVolumeUnit.cxx index 67a39b25a5..4c00ee099d 100755 --- a/src/RWStepBasic/RWStepBasic_RWSiUnitAndVolumeUnit.cxx +++ b/src/RWStepBasic/RWStepBasic_RWSiUnitAndVolumeUnit.cxx @@ -49,19 +49,29 @@ void RWStepBasic_RWSiUnitAndVolumeUnit::ReadStep(const Handle(StepData_StepReade if (data->ParamType(num,1) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,1); hasAprefix = reader.DecodePrefix(aPrefix,text); - if(!hasAprefix) - ach->AddFail("Enumeration si_prefix has not an allowed value"); + if(!hasAprefix){ + ach->AddFail("Enumeration si_prefix has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #2 (prefix) is not an enumeration"); + return; } - else ach->AddFail("Parameter #2 (prefix) is not an enumeration"); } StepBasic_SiUnitName aName; if (data->ParamType(num,2) == Interface_ParamEnum) { Standard_CString text = data->ParamCValue(num,2); - if(!reader.DecodeName(aName,text)) + if(!reader.DecodeName(aName,text)){ ach->AddFail("Enumeration si_unit_name has not an allowed value"); + return; + } + } + else{ + ach->AddFail("Parameter #3 (name) is not an enumeration"); + return; } - else ach->AddFail("Parameter #3 (name) is not an enumeration"); data->NamedForComplex("VOLUME_UNIT",num0,num,ach); if (!data->CheckNbParams(num,0,ach,"volume_unit")) return; diff --git a/src/TopOpeBRep/TopOpeBRep_ShapeIntersector2d.cxx b/src/TopOpeBRep/TopOpeBRep_ShapeIntersector2d.cxx index e11f2196c8..b33881a465 100755 --- a/src/TopOpeBRep/TopOpeBRep_ShapeIntersector2d.cxx +++ b/src/TopOpeBRep/TopOpeBRep_ShapeIntersector2d.cxx @@ -258,7 +258,7 @@ void TopOpeBRep_ShapeIntersector2d::FindFFIntersection() myFFDone = Standard_False; // myFFSameDomain = Standard_False; - while ( MoreFFCouple() ) { + if ( MoreFFCouple() ) { // The two candidate intersecting GeomShapes GS1,GS2 and their types t1,t2 const TopoDS_Shape& GS1 = myFaceScanner.Current(); @@ -277,8 +277,6 @@ void TopOpeBRep_ShapeIntersector2d::FindFFIntersection() BS.Box(GS1); BS.Box(GS2); myFFDone = Standard_True; - break; - NextFFCouple(); } SetIntersectionDone(); diff --git a/src/TopOpeBRepTool/TopOpeBRepTool_mkTondgE.cxx b/src/TopOpeBRepTool/TopOpeBRepTool_mkTondgE.cxx index 7490a65d80..e72db5fc6e 100755 --- a/src/TopOpeBRepTool/TopOpeBRepTool_mkTondgE.cxx +++ b/src/TopOpeBRepTool/TopOpeBRepTool_mkTondgE.cxx @@ -366,7 +366,7 @@ Standard_Boolean TopOpeBRepTool_mkTondgE::MkTonE(const TopoDS_Edge& ei, Standard else mkT = outin ? MKI2 : MKI1; return Standard_True; //////////////////////////////////////////////// - +/* // xxri : Standard_Real ddot = tgin1di.Dot(faxis); Standard_Boolean tgaxis = Abs(1-(Abs(ddot))) < FUN_tola(); //=true : edge is tangent to sphere's axis @@ -378,7 +378,7 @@ Standard_Boolean TopOpeBRepTool_mkTondgE::MkTonE(const TopoDS_Edge& ei, Standard if ((!onfi) && (!onli)) {mkT = MKI12; return Standard_True;} // @190499 if (closedi) {mkT = MKI12; return Standard_True;}// onfi || onli @190499 xxri = tgin1di; - } + }*/ }//!isT2d mkT = MKI12; // without restrictions. @@ -387,5 +387,7 @@ Standard_Boolean TopOpeBRepTool_mkTondgE::MkTonE(const TopoDS_Edge& ei, Standard Standard_Boolean keepang = (dot > 0); if (keepang) mkT = outin ? MKI1 : MKI2; else mkT = outin ? MKI2 : MKI1; + return Standard_True; + }