0029814: Modeling Data - add method TopoDS_Shape::NbChildren() for simple check of sub-shapes number

TopoDS_Shape::NbChildren() - new method has been introduced.
TopoDS_Shape.lxx, TopoDS_TShape.lxx - inline methods have been moved to hxx.
TopoDS_TShape_Flags has been redeclared as enumeration instead of preprocessor macros.

Cyclic dependency between headers TopoDS_Shape.hxx and TopoDS_TShape.hxx eliminated.

Places where TopoDS_Iterator is used only for calculation of number of sub-shapes are updated to use NbChildren() instead

src/AIS/AIS_ParallelRelation.cxx

@@ -48,7 +48,6 @@
 #include <Standard_Type.hxx>
 #include <TCollection_AsciiString.hxx>
 #include <TCollection_ExtendedString.hxx>
-#include <TopExp_Explorer.hxx>
 #include <TopoDS.hxx>
 #include <TopoDS_Shape.hxx>
 

src/AIS/AIS_Shape.cxx

@@ -65,8 +65,6 @@
 #include <StdSelect_DisplayMode.hxx>
 #include <TColStd_ListIteratorOfListOfInteger.hxx>
 #include <TopExp.hxx>
-#include <TopExp_Explorer.hxx>
-#include <TopoDS_Iterator.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(AIS_Shape,AIS_InteractiveObject)
 
@@ -112,12 +110,9 @@ void AIS_Shape::Compute(const Handle(PrsMgr_PresentationManager3d)& /*aPresentat
     aPrs->SetDisplayPriority(TheType+2);
   }
   // Shape vide -> Assemblage vide.
-  if (myshape.ShapeType() == TopAbs_COMPOUND) {
+  if (myshape.ShapeType() == TopAbs_COMPOUND && myshape.NbChildren() == 0)
-    TopoDS_Iterator anExplor (myshape);
+  {
-
+    return;
-    if (!anExplor.More()) {
-      return;
-    }
   }
 
   if (IsInfinite())
@@ -227,8 +222,7 @@ void AIS_Shape::computeHlrPresentation (const Handle(Prs3d_Projector)& theProjec
     }
     case TopAbs_COMPOUND:
     {
-      TopoDS_Iterator anExplor (theShape);
+      if (theShape.NbChildren() == 0)
-      if (!anExplor.More())
       {
         return;
       }
@@ -293,11 +287,10 @@ void AIS_Shape::ComputeSelection(const Handle(SelectMgr_Selection)& aSelection,
                                               const Standard_Integer aMode)
 {
   if(myshape.IsNull()) return;
-  if (myshape.ShapeType() == TopAbs_COMPOUND) {
+  if (myshape.ShapeType() == TopAbs_COMPOUND && myshape.NbChildren() == 0)
-    TopoDS_Iterator anExplor (myshape);
+  {
-
+    // empty Shape -> empty Assembly.
-    if (!anExplor.More()) // empty Shape -> empty Assembly.
+    return;
-      return;
   }
 
   TopAbs_ShapeEnum TypOfSel = AIS_Shape::SelectionType(aMode);
@@ -982,13 +975,11 @@ void AIS_Shape::LoadRecomputable(const Standard_Integer TheMode)
 
 const Bnd_Box& AIS_Shape::BoundingBox()  
 {
-  if (myshape.ShapeType() == TopAbs_COMPOUND) {
+  if (myshape.ShapeType() == TopAbs_COMPOUND && myshape.NbChildren() == 0)
-    TopoDS_Iterator anExplor (myshape);
+  {
-
+    // empty Shape  -> empty Assembly.
-    if (!anExplor.More()) { // empty Shape  -> empty Assembly.
+    myBB.SetVoid ();
-      myBB.SetVoid();
+    return myBB;
-      return myBB;
-    }
   }
 
   if(myCompBB) {

src/AIS/AIS_SymmetricRelation.cxx

@@ -46,7 +46,6 @@
 #include <SelectMgr_Selection.hxx>
 #include <Standard_NotImplemented.hxx>
 #include <Standard_Type.hxx>
-#include <TopExp_Explorer.hxx>
 #include <TopoDS.hxx>
 #include <TopoDS_Shape.hxx>
 

src/BOPAlgo/BOPAlgo_ArgumentAnalyzer.cxx

@@ -39,7 +39,6 @@
 #include <TopExp_Explorer.hxx>
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Shell.hxx>
 #include <TopoDS_Solid.hxx>

src/BOPAlgo/BOPAlgo_Builder_3.cxx

@@ -29,7 +29,6 @@
 #include <TopoDS_Face.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Solid.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shell.hxx>
 #include <TopoDS_Compound.hxx>
 //

src/BOPAlgo/BOPAlgo_Tools.cxx

@@ -712,8 +712,7 @@ Standard_Boolean BOPAlgo_Tools::WiresToFaces(const TopoDS_Shape& theWires,
   BOPTools_AlgoTools::CorrectShapeTolerances(aRFaces, aMEmpty, Standard_False);
   //
   theFaces = aRFaces;
-  TopoDS_Iterator aItF(theFaces);
+  return theFaces.NbChildren() > 0;
-  return aItF.More();
 }
 
 //=======================================================================

src/BOPAlgo/BOPAlgo_WireSplitter.cxx

@@ -25,7 +25,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Wire.hxx>
 #include <TopTools_IndexedDataMapOfShapeListOfShape.hxx>

src/BRep/BRep_TEdge.cxx

@@ -23,7 +23,7 @@
 #include <BRep_TEdge.hxx>
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(BRep_TEdge,TopoDS_TEdge)
 

src/BRep/BRep_TFace.cxx

@@ -21,7 +21,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopLoc_Location.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(BRep_TFace,TopoDS_TFace)
 

src/BRep/BRep_TVertex.cxx

@@ -19,7 +19,7 @@
 #include <gp_Pnt.hxx>
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(BRep_TVertex,TopoDS_TVertex)
 

src/BRepCheck/BRepCheck.cxx

@@ -22,7 +22,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Wire.hxx>
 
 //=======================================================================

src/BRepFill/BRepFill_TrimShellCorner.cxx

@@ -2182,15 +2182,12 @@ static Standard_Real ComputeAveragePlaneAndMaxDeviation(const TopoDS_Shape& aWir
                                                         gp_Pln& thePlane,
                                                         Standard_Boolean& IsSingular)
 {
-  Standard_Integer N = 40, nedges = 0;
+  Standard_Integer N = 40;
-
+  Standard_Integer nedges = aWire.NbChildren();
-  TopoDS_Iterator iter( aWire );
-  for (; iter.More(); iter.Next())
-    nedges++;
 
   TColgp_Array1OfPnt Pnts( 1, nedges*N );
   Standard_Integer ind = 1, i;
-  for (iter.Initialize(aWire); iter.More(); iter.Next())
+  for (TopoDS_Iterator iter (aWire); iter.More(); iter.Next())
     {
       const TopoDS_Edge& anEdge = TopoDS::Edge( iter.Value() );
       BRepAdaptor_Curve aCurve(anEdge);

src/BRepGProp/BRepGProp.cxx

@@ -110,8 +110,7 @@ static Standard_Real surfaceProperties(const TopoDS_Shape& S, GProp_GProps& Prop
     }
 
     BF.Load(F);
-    TopoDS_Iterator aWIter(F);
+    Standard_Boolean IsNatRestr = (F.NbChildren() == 0);
-    Standard_Boolean IsNatRestr = !aWIter.More();
     if(!IsNatRestr) BD.Init(F);
     if(Eps < 1.0) {
       G.Perform(BF, BD, Eps); 
@@ -199,8 +198,7 @@ static Standard_Real volumeProperties(const TopoDS_Shape& S, GProp_GProps& Props
 
     if (isFwd || isRvs){
       BF.Load(F);
-      TopoDS_Iterator aWIter(F);
+      Standard_Boolean IsNatRestr = (F.NbChildren () == 0);
-      Standard_Boolean IsNatRestr = !aWIter.More();
       if(!IsNatRestr) BD.Init(F);
       if(Eps < 1.0) {
         G.Perform(BF, BD, Eps); 
@@ -348,8 +346,7 @@ static Standard_Real volumePropertiesGK(const TopoDS_Shape     &theShape,
     if (isFwd || isRvs){
         aPropFace.Load(aFace);
 
-        TopoDS_Iterator aWIter(aFace);
+        Standard_Boolean IsNatRestr = (aFace.NbChildren () == 0);
-        Standard_Boolean IsNatRestr = !aWIter.More();
         if(IsNatRestr)
           aLocalError = aVProps.Perform(aPropFace, aTol, CGFlag, IFlag);
         else {
@@ -496,8 +493,7 @@ static Standard_Real volumePropertiesGK(const TopoDS_Shape     &theShape,
     if (isFwd || isRvs){
         aPropFace.Load(aFace);
 
-        TopoDS_Iterator aWIter(aFace);
+        Standard_Boolean IsNatRestr = (aFace.NbChildren () == 0);
-        Standard_Boolean IsNatRestr = !aWIter.More();
         if(IsNatRestr)
           aLocalError = aVProps.Perform(aPropFace, thePln, aTol, CGFlag, IFlag);
         else {

src/BRepGProp/BRepGProp_Gauss.cxx

@@ -585,8 +585,7 @@ Standard_Real BRepGProp_Gauss::Compute(
   //
   const Standard_Integer NumSubs = SUBS_POWER;
   const TopoDS_Face& aF = theSurface.GetFace(); 
-  TopoDS_Iterator aWIter(aF);
+  const Standard_Boolean isNaturalRestriction = (aF.NbChildren () == 0); //theSurface.NaturalRestriction();
-  const Standard_Boolean isNaturalRestriction = !aWIter.More(); //theSurface.NaturalRestriction();
 
   Standard_Real CIx, CIy, CIz, CIxy, CIxz, CIyz;
   Standard_Real CDim[2], CIxx[2], CIyy[2], CIzz[2];

src/BRepOffset/BRepOffset_MakeOffset.cxx

@@ -2229,36 +2229,30 @@ void BRepOffset_MakeOffset::CorrectConicalFaces()
         }
       }
     }
-    TopoDS_Iterator anIt(Sol);
+    Standard_Integer nbs = Sol.NbChildren();
-    Standard_Boolean SolIsNull = !anIt.More();
+    Standard_Boolean SolIsNull = (nbs == 0);
     //Checking solid
-    if(!SolIsNull)
+    if (nbs > 1)
     {
-      Standard_Integer nbs = 0;
+      BRepCheck_Analyzer aCheck (Sol, Standard_False);
-      while(anIt.More()) {anIt.Next(); ++nbs;}
+      if (!aCheck.IsValid ())
-      if(nbs > 1)
       {
-        BRepCheck_Analyzer aCheck(Sol, Standard_False);
+        TopTools_ListOfShape aSolList;
-        if(!aCheck.IsValid())
+        CorrectSolid (Sol, aSolList);
+        if (!aSolList.IsEmpty ())
         {
-          TopTools_ListOfShape aSolList;
+          BB.Add (NC, Sol);
-          CorrectSolid(Sol, aSolList);
+          TopTools_ListIteratorOfListOfShape aSLIt (aSolList);
-          if(!aSolList.IsEmpty())
+          for (; aSLIt.More (); aSLIt.Next ())
           {
-            BB.Add(NC, Sol);
+            BB.Add (NC, aSLIt.Value ());
-            TopTools_ListIteratorOfListOfShape aSLIt(aSolList);
-            for(; aSLIt.More(); aSLIt.Next())
-            {
-              BB.Add(NC, aSLIt.Value());
-            }
-            SolIsNull = Standard_True;
           }
+          SolIsNull = Standard_True;
         }
       }
     }
     //
-    anIt.Initialize(NC);
+    Standard_Boolean NCIsNull = (NC.NbChildren() == 0);
-    Standard_Boolean NCIsNull = !anIt.More();
     if((!SolIsNull) && (!NCIsNull))
     {
       BB.Add(NC, Sol);
@@ -3072,35 +3066,29 @@ void BRepOffset_MakeOffset::MakeSolid ()
       }
     }
   }
-  TopoDS_Iterator anIt(Sol);
+  Standard_Integer nbs = Sol.NbChildren();
-  Standard_Boolean SolIsNull = !anIt.More();
+  Standard_Boolean SolIsNull = (nbs == 0);
   //Checking solid
-  if(!SolIsNull)
+  if (nbs > 1)
   {
-    Standard_Integer nbs = 0;
+    BRepCheck_Analyzer aCheck (Sol, Standard_False);
-    while(anIt.More()) {anIt.Next(); ++nbs;}
+    if (!aCheck.IsValid ())
-    if(nbs > 1)
     {
-      BRepCheck_Analyzer aCheck(Sol, Standard_False);
+      TopTools_ListOfShape aSolList;
-      if(!aCheck.IsValid())
+      CorrectSolid (Sol, aSolList);
+      if (!aSolList.IsEmpty ())
       {
-        TopTools_ListOfShape aSolList;
+        B.Add (NC, Sol);
-        CorrectSolid(Sol, aSolList);
+        TopTools_ListIteratorOfListOfShape aSLIt (aSolList);
-        if(!aSolList.IsEmpty())
+        for (; aSLIt.More (); aSLIt.Next ())
         {
-          B.Add(NC, Sol);
+          B.Add (NC, aSLIt.Value ());
-          TopTools_ListIteratorOfListOfShape aSLIt(aSolList);
-          for(; aSLIt.More(); aSLIt.Next())
-          {
-            B.Add(NC, aSLIt.Value());
-          }
-          SolIsNull = Standard_True;
         }
+        SolIsNull = Standard_True;
       }
     }
   }
-  anIt.Initialize(NC);
+  Standard_Boolean NCIsNull = (NC.NbChildren() == 0);
-  Standard_Boolean NCIsNull = !anIt.More();
   if((!SolIsNull) && (!NCIsNull))
   {
     B.Add(NC, Sol);

src/BRepOffset/BRepOffset_MakeOffset_1.cxx

@@ -2386,9 +2386,7 @@ Standard_Boolean CheckInvertedBlock(const TopoDS_Shape& theCB,
 {
   // For possible removal of the block:
   // 1. There should be more than just one face in the block
-  TopoDS_Iterator aItF(theCB);
+  if (theCB.NbChildren() < 2) {
-  aItF.Next();
-  if (!aItF.More()) {
     return Standard_False;
   }
   //

src/BRepOffsetAPI/BRepOffsetAPI_ThruSections.cxx

@@ -398,9 +398,7 @@ void BRepOffsetAPI_ThruSections::Build()
       if (Georges.IsDegeneratedFirstSection())
         IndFirstSec = 2;
       TopoDS_Shape aWorkingSection = WorkingSections(IndFirstSec);
-      TopoDS_Iterator itw(aWorkingSection);
+      myNbEdgesInSection += aWorkingSection.NbChildren();
-      for (; itw.More(); itw.Next())
-        myNbEdgesInSection++;
       for (Standard_Integer ii = 1; ii <= myWires.Length(); ii++)
       {
         TopExp_Explorer Explo(myWires(ii), TopAbs_EDGE);
@@ -419,7 +417,7 @@ void BRepOffsetAPI_ThruSections::Build()
           {
             const TopoDS_Edge& aNewEdge = TopoDS::Edge(itl.Value());
             Standard_Integer inde = 1;
-            for (itw.Initialize(aWorkingSection); itw.More(); itw.Next(),inde++)
+            for (TopoDS_Iterator itw (aWorkingSection); itw.More(); itw.Next(), inde++)
             {
               const TopoDS_Shape& aWorkingEdge = itw.Value();
               if (aWorkingEdge.IsSame(aNewEdge))

src/BRepProj/BRepProj_Projection.cxx

@@ -39,7 +39,6 @@
 #include <TopExp_Explorer.hxx>
 #include <TopLoc_Location.hxx>
 #include <TopoDS.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopTools_ListIteratorOfListOfShape.hxx>
 #include <TopTools_ListOfShape.hxx>

src/BRepTools/BRepTools.cxx

@@ -57,7 +57,6 @@
 #include <TopoDS_CompSolid.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Shell.hxx>
 #include <TopoDS_Solid.hxx>

src/DBRep/DBRep_IsoBuilder.cxx

@@ -375,14 +375,11 @@ void DBRep_IsoBuilder::FillGaps(const TopoDS_Face& theFace,
 
     // Check the number of edges in the wire, not to
     // miss the wires containing one edge only
-    Standard_Boolean SingleEdge = Standard_True;
+    if (aW.NbChildren() == 0)
     {
-      TopoDS_Iterator itE(aW);
+      continue;
-      if (!itE.More())
-        continue;
-      itE.Next();
-      SingleEdge = !itE.More();
     }
+    Standard_Boolean SingleEdge = (aW.NbChildren() == 1);
 
     TopoDS_Edge aPrevEdge, aCurrEdge;
 

src/DNaming/DNaming.cxx

@@ -724,11 +724,8 @@ void DNaming::LoadResult(const TDF_Label& ResultLabel, BRepAlgoAPI_BooleanOperat
   TNaming_Builder Builder (ResultLabel);
   TopoDS_Shape aResult = MS.Shape();
   if (aResult.ShapeType() == TopAbs_COMPOUND) {
-    Standard_Integer nbSubResults = 0;
+    if (aResult.NbChildren() == 1) {
-    TopoDS_Iterator itr(aResult);
+      TopoDS_Iterator itr (aResult);
-    for (; itr.More(); itr.Next()) nbSubResults++;
-    if (nbSubResults == 1) {
-      itr.Initialize(aResult);
       if (itr.More()) aResult = itr.Value();
     }
   }

src/DNaming/DNaming_BooleanOperationDriver.cxx

@@ -373,8 +373,7 @@ Standard_Boolean DNaming_BooleanOperationDriver::CheckAndLoad
 
   if (theMkOpe.IsDone() && !theMkOpe.Shape().IsNull()) {
     if (theMkOpe.Shape().ShapeType() == TopAbs_COMPOUND) {
-      TopoDS_Iterator anItr(theMkOpe.Shape());
+      if (theMkOpe.Shape().NbChildren() == 0) {
-      if(!anItr.More()) {
 	theFunction->SetFailure(NULL_RESULT);
 	return Standard_False;
       }      

src/DNaming/DNaming_FilletDriver.cxx

@@ -179,11 +179,8 @@ void DNaming_FilletDriver::LoadNamingDS (const TDF_Label& theResultLabel,
   TopoDS_Shape aResult = theMkFillet.Shape();
 
   if (aResult.ShapeType() == TopAbs_COMPOUND) {
-    Standard_Integer nbSubResults = 0;
+    if (aResult.NbChildren() == 1) {
-    TopoDS_Iterator itr(aResult);
+      TopoDS_Iterator itr (aResult);
-    for (; itr.More(); itr.Next()) nbSubResults++;
-    if (nbSubResults == 1) {
-      itr.Initialize(aResult);
       if (itr.More()) aResult = itr.Value();
     }
   }

src/DrawDim/DrawDim_PlanarDimensionCommands.cxx

@@ -38,13 +38,14 @@
 #include <gp_Circ.hxx>
 #include <TopTools_MapOfShape.hxx>
 #include <TopExp_Explorer.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <gp_Ax3.hxx>
 #include <gp_Pln.hxx>
 #include <DrawTrSurf.hxx>
+
 #ifdef _MSC_VER
 #include <stdio.h>
 #endif
+
 //=======================================================================
 //function : DrawDim_DISTANCE
 //purpose  : 

src/HLRBRep/HLRBRep_ShapeToHLR.cxx

@@ -33,7 +33,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Vertex.hxx>
 #include <TopTools_IndexedDataMapOfShapeListOfShape.hxx>
 #include <TopTools_ListIteratorOfListOfShape.hxx>

src/IGESToBRep/IGESToBRep.cxx

@@ -72,7 +72,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Wire.hxx>
 #include <XSAlgo.hxx>

src/IGESToBRep/IGESToBRep_BRepEntity.cxx

@@ -71,7 +71,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Shell.hxx>
 #include <TopoDS_Solid.hxx>

src/IVtkOCC/IVtkOCC_SelectableObject.cxx

@@ -94,13 +94,10 @@ void IVtkOCC_SelectableObject::ComputeSelection (const Handle(SelectMgr_Selectio
 
   TopoDS_Shape anOcctShape = myShape->GetShape();
 
-  if (anOcctShape.ShapeType() == TopAbs_COMPOUND)
+  if (anOcctShape.ShapeType() == TopAbs_COMPOUND && anOcctShape.NbChildren() == 0)
   {
-    TopoDS_Iterator anExplor (anOcctShape);
+    // Shape empty -> go away
-    if (!anExplor.More()) // Shape empty -> go away
+    return;
-    {
-      return;
-    }
   }
 
   TopAbs_ShapeEnum aTypeOfSel = AIS_Shape::SelectionType (theMode);
@@ -160,14 +157,11 @@ const Bnd_Box& IVtkOCC_SelectableObject::BoundingBox()
 
   TopoDS_Shape anOcctShape = myShape->GetShape();
 
-  if (anOcctShape.ShapeType() == TopAbs_COMPOUND)
+  if (anOcctShape.ShapeType() == TopAbs_COMPOUND && anOcctShape.NbChildren() == 0)
   {
-    TopoDS_Iterator anExplor (anOcctShape);
+    // Shape empty -> nothing to do
-    if (!anExplor.More())
+    myBndBox.SetVoid ();
-    { // Shape empty -> nothing to do
+    return myBndBox;
-      myBndBox.SetVoid();
-      return myBndBox;
-    }
   }
 
   if (myBndBox.IsVoid())

src/STEPConstruct/STEPConstruct_Styles.cxx

@@ -69,7 +69,6 @@
 #include <StepVisual_SurfaceStyleUsage.hxx>
 #include <TCollection_HAsciiString.hxx>
 #include <TColStd_HSequenceOfTransient.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <Transfer_Binder.hxx>
 #include <TransferBRep.hxx>

src/STEPConstruct/STEPConstruct_ValidationProps.cxx

@@ -57,7 +57,6 @@
 #include <StepShape_ShapeRepresentation.hxx>
 #include <TCollection_HAsciiString.hxx>
 #include <TopLoc_Location.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <Transfer_Binder.hxx>
 #include <Transfer_SimpleBinderOfTransient.hxx>

src/ShapeAnalysis/ShapeAnalysis_Wire.cxx

@@ -77,7 +77,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDS_Vertex.hxx>
 #include <TopoDS_Wire.hxx>

src/ShapeCustom/ShapeCustom.cxx

@@ -60,11 +60,7 @@ TopoDS_Shape ShapeCustom::ApplyModifier (const TopoDS_Shape &S,
     BRep_Builder B;
     B.MakeCompound ( C );
 
-    Standard_Integer aShapeCount = 0;
+    Standard_Integer aShapeCount = SF.NbChildren();
-    {
-      for (TopoDS_Iterator it(SF); it.More(); it.Next()) ++aShapeCount;
-    }
-
     Message_ProgressSentry aPSentry(aProgress, "Applying Modifier For Solids", 0, aShapeCount, 1);
     for ( TopoDS_Iterator it(SF); it.More() && aPSentry.More(); it.Next(), aPSentry.Next() ) {
       TopoDS_Shape shape = it.Value();

src/ShapeFix/ShapeFix_FixSmallSolid.cxx

@@ -368,7 +368,7 @@ static TopoDS_Shape MergeShells (
   }
   
   // If there are no elements in the new shell, return null shape
-  if (!TopoDS_Iterator (aNewShell).More())
+  if (aNewShell.NbChildren() == 0)
     return TopoDS_Shape();
 
   return aNewShell;

src/ShapeFix/ShapeFix_Shape.cxx

@@ -147,9 +147,7 @@ Standard_Boolean ShapeFix_Shape::Perform(const Handle(Message_ProgressIndicator)
     Standard_Integer savFixSameParameterMode = myFixSameParameterMode;
     myFixSameParameterMode = Standard_False;
     myFixVertexTolMode = Standard_False;
-    Standard_Integer aShapesNb = 0;
+    Standard_Integer aShapesNb = S.NbChildren();
-    for ( TopoDS_Iterator anIter(S); anIter.More(); anIter.Next() )
-      ++aShapesNb;
 
     // Open progress indication scope for sub-shape fixing
     Message_ProgressSentry aPSentrySubShape(theProgress, "Fixing sub-shape", 0, aShapesNb, 1);

src/ShapeFix/ShapeFix_Shell.cxx

@@ -117,9 +117,7 @@ Standard_Boolean ShapeFix_Shell::Perform(const Handle(Message_ProgressIndicator)
     TopoDS_Shape S = Context()->Apply(myShell);
 
     // Get the number of faces for progress indication
-    Standard_Integer aNbFaces = 0;
+    Standard_Integer aNbFaces = S.NbChildren();
-    for ( TopExp_Explorer aFaceExp(S, TopAbs_FACE); aFaceExp.More(); aFaceExp.Next() )
-      ++aNbFaces;
 
     // Start progress scope (no need to check if progress exists -- it is safe)
     Message_ProgressSentry aPSentry(theProgress, "Fixing face", 0, aNbFaces, 1);

src/ShapePersistent/ShapePersistent_TopoDS.cxx

@@ -213,13 +213,7 @@ ShapePersistent_TopoDS::Translate (const TopoDS_Shape& theShape,
     S.Orientation(TopAbs_FORWARD);
     S.Location(TopLoc_Location());
     // Count the number of <sub-shape> of the Shape's TShape
-    Standard_Integer nbElem = 0;
+    Standard_Integer nbElem = S.NbChildren();
-    TopoDS_Iterator anItCount(S);
-    while (anItCount.More()) {
-      ++nbElem;
-      anItCount.Next();
-    }
-
     if (nbElem > 0)
     {
       Handle(StdLPersistent_HArray1OfPersistent) aShapes =

src/ShapeProcessAPI/ShapeProcessAPI_ApplySequence.cxx

@@ -23,7 +23,6 @@
 #include <ShapeProcess_ShapeContext.hxx>
 #include <ShapeProcessAPI_ApplySequence.hxx>
 #include <TCollection_AsciiString.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopTools_DataMapIteratorOfDataMapOfShapeShape.hxx>
 

src/StdPrs/StdPrs_ShadedShape.cxx

@@ -313,8 +313,7 @@ namespace
     for (TopExp_Explorer aFaceIter (theShape, TopAbs_FACE); aFaceIter.More(); aFaceIter.Next())
     {
       const TopoDS_Face& aFace = TopoDS::Face (aFaceIter.Current());
-      TopoDS_Iterator aSubShapeIter (aFace);
+      if (aFace.NbChildren() == 0)
-      if (!aSubShapeIter.More())
       {
         // handle specifically faces without boundary definition (triangulation-only)
         StdPrs_WFShape::AddEdgesOnTriangulation (aSeqPntsExtra, aFace, Standard_False);
@@ -554,15 +553,13 @@ void StdPrs_ShadedShape::Add (const Handle (Prs3d_Presentation)& thePrs,
     aBuilder.MakeCompound (anOpened);
     ExploreSolids (theShape, aBuilder, aClosed, anOpened, Standard_True);
 
-    TopoDS_Iterator aShapeIter (aClosed);
+    if (aClosed.NbChildren() > 0)
-    if (aShapeIter.More())
     {
       shadeFromShape (aClosed, thePrs, theDrawer,
                       theHasTexels, theUVOrigin, theUVRepeat, theUVScale, true);
     }
 
-    aShapeIter.Initialize (anOpened);
+    if (anOpened.NbChildren() > 0)
-    if (aShapeIter.More())
     {
       shadeFromShape (anOpened, thePrs, theDrawer,
                       theHasTexels, theUVOrigin, theUVRepeat, theUVScale, false);

src/StepToTopoDS/StepToTopoDS_TranslateCurveBoundedSurface.cxx

@@ -33,7 +33,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <Transfer_TransientProcess.hxx>
 
 //=======================================================================

src/TNaming/TNaming_Localizer.cxx

@@ -34,7 +34,6 @@
 #include <TNaming_UsedShapes.hxx>
 #include <TopExp.hxx>
 #include <TopExp_Explorer.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopTools_IndexedDataMapOfShapeListOfShape.hxx>
 #include <TopTools_ListIteratorOfListOfShape.hxx>

src/TNaming/TNaming_Name.cxx

@@ -793,31 +793,28 @@ static Standard_Boolean Intersection (const TDF_Label&                  L,
 #ifdef OCCT_DEBUG_INT
     cout <<"Kept: indxE = " << indxE  <<" maxENum = " << nbE << " indxW = " <<indxW << " nbW = " <<nbW<<endl;
 #endif      
-    Standard_Integer aNbW(0), aCaseW(0);
+    Standard_Integer aCaseW(0);
-    TopoDS_Iterator it2(aS);
+    Standard_Integer aNbW = aS.NbChildren();
-    for (;it2.More();it2.Next()) aNbW++;  
     if(aNbW == nbW) aCaseW = 1;//exact solution for wire (nb of wires is kept)
     else aCaseW = 2; // indefinite description ==> compound which can include expected wire    
     if(aCaseW == 1) {      
       TopoDS_Shape aWire;
-      Standard_Integer i(1);
+      Standard_Integer i = 1;
-      it2.Initialize(aS);
+      for (TopoDS_Iterator it2 (aS); it2.More(); it2.Next(), i++)
-      for (;it2.More();it2.Next(),i++) {
+      {
 	if(indxW == i) {
 	  aWire = it2.Value();
 	  break;
 	}
       }
-      Standard_Integer aNbE(0), aCaseE(0);
+      Standard_Integer aCaseE(0);
-      it2.Initialize(aWire);
+      Standard_Integer aNbE = aWire.NbChildren();
-      for (;it2.More();it2.Next()) aNbE++;
       if(aNbE == nbE) aCaseE = 1;//exact solution for edge
       else aCaseE = 2;
       if(aCaseE == 1) {
 	i=1;
 	TopoDS_Shape anEdge;
-	it2.Initialize(aWire);
+	for (TopoDS_Iterator it2 (aWire); it2.More(); it2.Next(), i++) {
-	for (;it2.More();it2.Next(),i++) {
 	  if(indxE == i) {
 	    anEdge = it2.Value();
 	    break;

src/TopOpeBRepBuild/TopOpeBRepBuild_Merge.cxx

@@ -145,11 +145,7 @@ static void CorrectUnclosedWire(TopoDS_Shape& aWire)
   for(; tdi.More(); tdi.Next()) {
     nbe++;
     const TopoDS_Shape& ed = tdi.Value();
-    Standard_Integer nbv = 0;
+    Standard_Integer nbv = ed.NbChildren();
-    TopoDS_Iterator tdie(ed, Standard_False, Standard_False);
-    for(; tdie.More(); tdie.Next()) {
-      nbv++;
-    }
 //    cout << "Edge " << nbe << " : " << nbv << endl;
     if(nbv <= 1) {
 //      cout << "Remove bad edge" << endl;

src/TopoDS/FILES

@@ -23,7 +23,6 @@ TopoDS_ListOfShape.hxx
 TopoDS_LockedShape.hxx
 TopoDS_Shape.cxx
 TopoDS_Shape.hxx
-TopoDS_Shape.lxx
 TopoDS_Shell.hxx
 TopoDS_Shell.lxx
 TopoDS_Solid.hxx
@@ -42,7 +41,6 @@ TopoDS_TFace.hxx
 TopoDS_TFace.lxx
 TopoDS_TShape.cxx
 TopoDS_TShape.hxx
-TopoDS_TShape.lxx
 TopoDS_TShell.cxx
 TopoDS_TShell.hxx
 TopoDS_TShell.lxx

src/TopoDS/TopoDS_Builder.cxx

@@ -94,7 +94,7 @@ void TopoDS_Builder::Add (TopoDS_Shape& aShape,
     const unsigned int iS=(unsigned int)aShape.ShapeType();
     //
     if ((aTb[iC] & (1<<iS)) != 0) {
-      TopoDS_ListOfShape& L = aShape.TShape()->ChangeShapes();
+      TopoDS_ListOfShape& L = aShape.TShape()->myShapes;
       L.Append(aComponent);
       TopoDS_Shape& S = L.Last();
       //
@@ -137,7 +137,7 @@ void TopoDS_Builder::Remove (TopoDS_Shape& aShape,
     S.Reverse();
   S.Location(S.Location().Predivided(aShape.Location()));
 
-  TopoDS_ListOfShape& L = aShape.TShape()->ChangeShapes();
+  TopoDS_ListOfShape& L = aShape.TShape()->myShapes;
   TopoDS_ListIteratorOfListOfShape It(L);
   while (It.More()) {
     if (It.Value() == S) {

src/TopoDS/TopoDS_Iterator.cxx

@@ -38,7 +38,7 @@ void TopoDS_Iterator::Initialize(const TopoDS_Shape& S,
     myOrientation = S.Orientation();
   else
     myOrientation = TopAbs_FORWARD;
-  myShapes.Initialize(S.TShape()->Shapes());
+  myShapes.Initialize(S.TShape()->myShapes);
   if (More()) {
     myShape = myShapes.Value();
     myShape.Orientation(TopAbs::Compose(myOrientation,myShape.Orientation()));

src/TopoDS/TopoDS_Shape.cxx

@@ -14,12 +14,12 @@
 // Alternatively, this file may be used under the terms of Open CASCADE
 // commercial license or contractual agreement.
 
+#include <TopoDS_Shape.hxx>
 
 #include <Standard_DomainError.hxx>
 #include <Standard_NullObject.hxx>
 #include <Standard_TypeMismatch.hxx>
 #include <TopLoc_Location.hxx>
-#include <TopoDS_Shape.hxx>
 #include <TopoDS_TShape.hxx>
 
 //=======================================================================

src/TopoDS/TopoDS_Shape.hxx

@@ -17,16 +17,12 @@
 #ifndef _TopoDS_Shape_HeaderFile
 #define _TopoDS_Shape_HeaderFile
 
-#include <Standard.hxx>
 #include <Standard_DefineAlloc.hxx>
 #include <Standard_Handle.hxx>
-
+#include <TopAbs.hxx>
-#include <TopLoc_Location.hxx>
 #include <TopAbs_Orientation.hxx>
-#include <Standard_Boolean.hxx>
+#include <TopLoc_Location.hxx>
-#include <TopAbs_ShapeEnum.hxx>
+#include <TopoDS_TShape.hxx>
-#include <Standard_Integer.hxx>
-class TopoDS_TShape;
 
 // resolve name collisions with X11 headers
 #ifdef Convex
@@ -49,204 +45,226 @@ public:
 
   DEFINE_STANDARD_ALLOC
 
-  
   //! Creates a NULL Shape referring to nothing.
-    TopoDS_Shape();
+  TopoDS_Shape() : myOrient (TopAbs_EXTERNAL) {}
-  
+
   //! Returns true if this shape is null. In other words, it
   //! references no underlying shape with the potential to
   //! be given a location and an orientation.
-    Standard_Boolean IsNull() const;
+  Standard_Boolean IsNull() const { return myTShape.IsNull(); }
-  
+
   //! Destroys the reference to the underlying shape
   //! stored in this shape. As a result, this shape becomes null.
-    void Nullify();
+  void Nullify() { myTShape.Nullify(); }
-  
+
   //! Returns the shape local coordinate system.
-    const TopLoc_Location& Location() const;
+  const TopLoc_Location& Location() const { return myLocation; }
-  
+
   //! Sets the shape local coordinate system.
-    void Location (const TopLoc_Location& Loc);
+  void Location (const TopLoc_Location& theLoc) { myLocation = theLoc; }
-  
+
   //! Returns a  shape  similar to <me> with   the local
   //! coordinate system set to <Loc>.
-    TopoDS_Shape Located (const TopLoc_Location& Loc) const;
+  TopoDS_Shape Located (const TopLoc_Location& theLoc) const
-  
+  {
+    TopoDS_Shape aShape (*this);
+    aShape.Location (theLoc);
+    return aShape;
+  }
+
   //! Returns the shape orientation.
-    TopAbs_Orientation Orientation() const;
+  TopAbs_Orientation Orientation() const { return myOrient; }
-  
+
   //! Sets the shape orientation.
-    void Orientation (const TopAbs_Orientation Orient);
+  void Orientation (TopAbs_Orientation theOrient) { myOrient = theOrient; }
-  
+
   //! Returns  a    shape  similar  to  <me>   with  the
   //! orientation set to <Or>.
-    TopoDS_Shape Oriented (const TopAbs_Orientation Or) const;
+  TopoDS_Shape Oriented (TopAbs_Orientation theOrient) const
-  
+  {
-    const Handle(TopoDS_TShape)& TShape() const;
+    TopoDS_Shape aShape (*this);
-  
+    aShape.Orientation (theOrient);
+    return aShape;
+  }
+
+  //! Returns a handle to the actual shape implementation.
+  const Handle(TopoDS_TShape)& TShape() const { return myTShape; }
+
   //! Returns the value of the TopAbs_ShapeEnum
   //! enumeration that corresponds to this shape, for
   //! example VERTEX, EDGE, and so on.
   //! Exceptions
   //! Standard_NullObject if this shape is null.
-    TopAbs_ShapeEnum ShapeType() const;
+  TopAbs_ShapeEnum ShapeType() const { return myTShape->ShapeType(); }
-  
+
   //! Returns the free flag.
-    Standard_Boolean Free() const;
+  Standard_Boolean Free() const { return myTShape->Free(); }
-  
+
   //! Sets the free flag.
-    void Free (const Standard_Boolean F);
+  void Free (Standard_Boolean theIsFree) { myTShape->Free (theIsFree); }
-  
+
   //! Returns the locked flag.
-    Standard_Boolean Locked() const;
+  Standard_Boolean Locked() const { return myTShape->Locked(); }
-  
+
   //! Sets the locked flag.
-    void Locked (const Standard_Boolean F);
+  void Locked (Standard_Boolean theIsLocked) { myTShape->Locked (theIsLocked); }
-  
+
   //! Returns the modification flag.
-    Standard_Boolean Modified() const;
+  Standard_Boolean Modified() const { return myTShape->Modified(); }
-  
+
   //! Sets the modification flag.
-    void Modified (const Standard_Boolean M);
+  void Modified (Standard_Boolean theIsModified) { myTShape->Modified (theIsModified); }
-  
+
   //! Returns the checked flag.
-    Standard_Boolean Checked() const;
+  Standard_Boolean Checked() const { return myTShape->Checked(); }
-  
+
   //! Sets the checked flag.
-    void Checked (const Standard_Boolean C);
+  void Checked (Standard_Boolean theIsChecked) { myTShape->Checked (theIsChecked); }
-  
+
   //! Returns the orientability flag.
-    Standard_Boolean Orientable() const;
+  Standard_Boolean Orientable() const { return myTShape->Orientable(); }
-  
+
   //! Sets the orientability flag.
-    void Orientable (const Standard_Boolean C);
+  void Orientable (const Standard_Boolean theIsOrientable) { myTShape->Orientable (theIsOrientable); }
-  
+
   //! Returns the closedness flag.
-    Standard_Boolean Closed() const;
+  Standard_Boolean Closed() const { return myTShape->Closed(); }
-  
+
   //! Sets the closedness flag.
-    void Closed (const Standard_Boolean C);
+  void Closed (Standard_Boolean theIsClosed) { myTShape->Closed (theIsClosed); }
-  
+
   //! Returns the infinity flag.
-    Standard_Boolean Infinite() const;
+  Standard_Boolean Infinite() const { return myTShape->Infinite(); }
-  
+
   //! Sets the infinity flag.
-    void Infinite (const Standard_Boolean C);
+  void Infinite (Standard_Boolean theIsInfinite) { myTShape->Infinite (theIsInfinite); }
-  
+
   //! Returns the convexness flag.
-    Standard_Boolean Convex() const;
+  Standard_Boolean Convex() const { return myTShape->Convex(); }
-  
+
   //! Sets the convexness flag.
-    void Convex (const Standard_Boolean C);
+  void Convex (Standard_Boolean theIsConvex) { myTShape->Convex (theIsConvex); }
-  
+
-  //! Multiplies the Shape location by <position>.
+  //! Multiplies the Shape location by thePosition.
-    void Move (const TopLoc_Location& position);
+  void Move (const TopLoc_Location& thePosition) { myLocation = thePosition * myLocation; }
-  
+
-  //! Returns  a shape similar  to  <me> with a location
+  //! Returns a shape similar to <me> with a location multiplied by thePosition.
-  //! multiplied  by <position>.
+  TopoDS_Shape Moved (const TopLoc_Location& thePosition) const
-    TopoDS_Shape Moved (const TopLoc_Location& position) const;
+  {
-  
+    TopoDS_Shape aShape (*this);
+    aShape.Move (thePosition);
+    return aShape;
+  }
+
   //! Reverses the orientation, using the Reverse method
   //! from the TopAbs package.
-    void Reverse();
+  void Reverse() { myOrient = TopAbs::Reverse (myOrient); }
-  
+
   //! Returns    a shape  similar    to  <me>  with  the
   //! orientation  reversed, using  the   Reverse method
   //! from the TopAbs package.
-    TopoDS_Shape Reversed() const;
+  TopoDS_Shape Reversed() const
-  
+  {
+    TopoDS_Shape aShape (*this);
+    aShape.Reverse();
+    return aShape;
+  }
+
   //! Complements the orientation, using the  Complement
   //! method from the TopAbs package.
-    void Complement();
+  void Complement() { myOrient = TopAbs::Complement (myOrient); }
-  
+
   //! Returns  a   shape  similar  to   <me>   with  the
   //! orientation complemented,  using   the  Complement
   //! method from the TopAbs package.
-    TopoDS_Shape Complemented() const;
+  TopoDS_Shape Complemented() const
-  
+  {
-  //! Updates the  Shape Orientation by composition with
+    TopoDS_Shape aShape (*this);
-  //! <Orient>, using the Compose method from the TopAbs
+    aShape.Complement();
-  //! package.
+    return aShape;
-    void Compose (const TopAbs_Orientation Orient);
+  }
-  
+
+  //! Updates the Shape Orientation by composition with theOrient,
+  //! using the Compose method from the TopAbs package.
+  void Compose (TopAbs_Orientation theOrient) { myOrient = TopAbs::Compose (myOrient, theOrient); }
+
   //! Returns  a  shape   similar   to  <me>   with  the
-  //! orientation  composed  with <Orient>,   using  the
+  //! orientation composed with theOrient, using the
   //! Compose method from the TopAbs package.
-    TopoDS_Shape Composed (const TopAbs_Orientation Orient) const;
+  TopoDS_Shape Composed (TopAbs_Orientation theOrient) const
-  
+  {
+    TopoDS_Shape aShape (*this);
+    aShape.Compose (theOrient);
+    return aShape;
+  }
+
+  //! Returns the number of direct sub-shapes (children).
+  //! @sa TopoDS_Iterator for accessing sub-shapes
+  Standard_Integer NbChildren() const { return myTShape.IsNull() ? 0 : myTShape->NbChildren(); }
+
   //! Returns True if two shapes  are partners, i.e.  if
   //! they   share   the   same  TShape.  Locations  and
   //! Orientations may differ.
-    Standard_Boolean IsPartner (const TopoDS_Shape& other) const;
+  Standard_Boolean IsPartner (const TopoDS_Shape& theOther) const { return (myTShape == theOther.myTShape); }
-  
+
   //! Returns True if two shapes are same, i.e.  if they
   //! share  the  same TShape  with the same  Locations.
   //! Orientations may differ.
-    Standard_Boolean IsSame (const TopoDS_Shape& other) const;
+  Standard_Boolean IsSame (const TopoDS_Shape& theOther) const
-  
+  {
+    return myTShape   == theOther.myTShape
+        && myLocation == theOther.myLocation;
+  }
+
   //! Returns True if two shapes are equal, i.e. if they
   //! share the same TShape with  the same Locations and
   //! Orientations.
-    Standard_Boolean IsEqual (const TopoDS_Shape& other) const;
+  Standard_Boolean IsEqual (const TopoDS_Shape& theOther) const
-  Standard_Boolean operator == (const TopoDS_Shape& other) const
+  {
-{
+    return myTShape   == theOther.myTShape
-  return IsEqual(other);
+        && myLocation == theOther.myLocation
-}
+        && myOrient   == theOther.myOrient;
+  }
+
+  Standard_Boolean operator == (const TopoDS_Shape& theOther) const { return IsEqual (theOther); }
   
   //! Negation of the IsEqual method.
-    Standard_Boolean IsNotEqual (const TopoDS_Shape& other) const;
+  Standard_Boolean IsNotEqual  (const TopoDS_Shape& theOther) const { return !IsEqual (theOther); }
-  Standard_Boolean operator != (const TopoDS_Shape& other) const
+  Standard_Boolean operator != (const TopoDS_Shape& theOther) const { return IsNotEqual (theOther); }
-{
+
-  return IsNotEqual(other);
-}
-  
   //! Returns a hashed value  denoting <me>.  This value
   //! is in the range  1..<Upper>.  It is  computed from
   //! the  TShape  and the  Location. The Orientation is
   //! not used.
   Standard_EXPORT Standard_Integer HashCode (const Standard_Integer Upper) const;
-  
+
   //! Replace   <me> by  a  new   Shape with the    same
   //! Orientation and Location and a new TShape with the
   //! same geometry and no sub-shapes.
-    void EmptyCopy();
+  void EmptyCopy() { myTShape = myTShape->EmptyCopy(); }
-  
+
   //! Returns a new Shape with the  same Orientation and
   //! Location and  a new TShape  with the same geometry
   //! and no sub-shapes.
-    TopoDS_Shape EmptyCopied() const;
+  TopoDS_Shape EmptyCopied() const
-  
+  {
-    void TShape (const Handle(TopoDS_TShape)& T);
+    TopoDS_Shape aShape (*this);
-
+    aShape.EmptyCopy();
-
+    return aShape;
-
+  }
-
-protected:
-
-
-
 
+  void TShape (const Handle(TopoDS_TShape)& theTShape) { myTShape = theTShape; }
 
 private:
 
-
-
   Handle(TopoDS_TShape) myTShape;
   TopLoc_Location myLocation;
   TopAbs_Orientation myOrient;
 
-
 };
 
-
-#include <TopoDS_Shape.lxx>
-
-
-
 inline Standard_Integer HashCode(const TopoDS_Shape& me,const Standard_Integer Upper) {
  return me.HashCode(Upper);
 }
 
-
-
 #endif // _TopoDS_Shape_HeaderFile

src/TopoDS/TopoDS_Shape.lxx

@@ -1,459 +0,0 @@
-// Created on: 1991-03-20
-// Created by: Remi Lequette
-// Copyright (c) 1991-1999 Matra Datavision
-// Copyright (c) 1999-2014 OPEN CASCADE SAS
-//
-// This file is part of Open CASCADE Technology software library.
-//
-// This library is free software; you can redistribute it and/or modify it under
-// the terms of the GNU Lesser General Public License version 2.1 as published
-// by the Free Software Foundation, with special exception defined in the file
-// OCCT_LGPL_EXCEPTION.txt. Consult the file LICENSE_LGPL_21.txt included in OCCT
-// distribution for complete text of the license and disclaimer of any warranty.
-//
-// Alternatively, this file may be used under the terms of Open CASCADE
-// commercial license or contractual agreement.
-
-#include <TopoDS_TShape.hxx>
-#include <TopAbs.hxx>
-
-
-//=======================================================================
-//function : TopoDS_Shape
-//purpose  : Constructor Null Shape
-//=======================================================================
-
-inline TopoDS_Shape::TopoDS_Shape ()
- : myOrient(TopAbs_EXTERNAL)
-{
-}
-
-//=======================================================================
-//function : IsNull
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::IsNull () const
-{
-  return myTShape.IsNull();
-}
-
-//=======================================================================
-//function : Nullify
-//purpose  : clear the TShape 
-//=======================================================================
-
-inline void TopoDS_Shape::Nullify ()
-{
-  myTShape.Nullify();
-}
-
-//=======================================================================
-//function : Location
-//purpose  : 
-//=======================================================================
-
-inline const TopLoc_Location& TopoDS_Shape::Location () const
-{
-  return myLocation;
-}
-
-//=======================================================================
-//function : Location
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Location (const TopLoc_Location& Loc)
-{
-  myLocation = Loc;
-}
-
-//=======================================================================
-//function : Located
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::Located (const TopLoc_Location& Loc) const
-{
-  TopoDS_Shape S(*this);
-  S.Location(Loc);
-  return S;
-}
-
-//=======================================================================
-//function : Orientation
-//purpose  : 
-//=======================================================================
-
-inline TopAbs_Orientation TopoDS_Shape::Orientation () const
-{
-  return myOrient;
-}
-
-//=======================================================================
-//function : Orientation
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Orientation (const TopAbs_Orientation Orient)
-{
-  myOrient = Orient;
-}
-
-//=======================================================================
-//function : Oriented
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::Oriented (const TopAbs_Orientation Or) const
-{
-  TopoDS_Shape S(*this);
-  S.Orientation(Or);
-  return S;
-}
-
-//=======================================================================
-//function : TShape
-//purpose  : 
-//=======================================================================
-
-inline const Handle(TopoDS_TShape)& TopoDS_Shape::TShape () const
-{
-  return myTShape;
-}
-
-//=======================================================================
-//function : ShapeType
-//purpose  : 
-//=======================================================================
-
-inline TopAbs_ShapeEnum TopoDS_Shape::ShapeType() const
-{
-  return myTShape->ShapeType();
-}
-
-//=======================================================================
-//function : Free
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Free () const
-{
-  return myTShape->Free();
-}
-
-//=======================================================================
-//function : Free
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Free (const Standard_Boolean B)
-{
-  myTShape->Free(B);
-}
-
-//=======================================================================
-//function : Locked
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Locked () const
-{
-  return myTShape->Locked();
-}
-
-//=======================================================================
-//function : Locked
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Locked (const Standard_Boolean B)
-{
-  myTShape->Locked(B);
-}
-
-//=======================================================================
-//function : Modified
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Modified () const
-{
-  return myTShape->Modified();
-}
-
-//=======================================================================
-//function : Modified
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Modified (const Standard_Boolean B)
-{
-  myTShape->Modified(B);
-}
-
-//=======================================================================
-//function : Checked
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Checked () const
-{
-  return myTShape->Checked();
-}
-
-//=======================================================================
-//function : Checked
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Checked (const Standard_Boolean B)
-{
-  myTShape->Checked(B);
-}
-
-//=======================================================================
-//function : Orientable
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Orientable () const
-{
-  return myTShape->Orientable();
-}
-
-//=======================================================================
-//function : Orientable
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Orientable (const Standard_Boolean B)
-{
-  myTShape->Orientable(B);
-}
-
-//=======================================================================
-//function : Closed
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Closed () const
-{
-  return myTShape->Closed();
-}
-
-//=======================================================================
-//function : Closed
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Closed (const Standard_Boolean B)
-{
-  myTShape->Closed(B);
-}
-
-//=======================================================================
-//function : Infinite
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Infinite () const
-{
-  return myTShape->Infinite();
-}
-
-//=======================================================================
-//function : Infinite
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Infinite (const Standard_Boolean B)
-{
-  myTShape->Infinite(B);
-}
-
-//=======================================================================
-//function : Convex
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::Convex () const
-{
-  return myTShape->Convex();
-}
-
-//=======================================================================
-//function : Convex
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Convex (const Standard_Boolean B)
-{
-  myTShape->Convex(B);
-}
-
-//=======================================================================
-//function : Move
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Move (const TopLoc_Location& position)
-{
-  myLocation = position * myLocation;
-}
-
-//=======================================================================
-//function : Moved
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::Moved 
-  (const TopLoc_Location& position) const
-{
-  TopoDS_Shape S(*this);
-  S.Move(position);
-  return S;
-}
-
-//=======================================================================
-//function : Reverse
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Reverse()
-{
-  myOrient = TopAbs::Reverse(myOrient);
-}
-
-//=======================================================================
-//function : Reversed
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape  TopoDS_Shape::Reversed() const
-{
-  TopoDS_Shape S(*this);
-  S.Reverse();
-  return S;
-}
-
-//=======================================================================
-//function : Complement
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Complement()
-{
-  myOrient = TopAbs::Complement(myOrient);
-}
-
-//=======================================================================
-//function : Complemented
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::Complemented() const
-{
-  TopoDS_Shape S(*this);
-  S.Complement();
-  return S;
-}
-
-//=======================================================================
-//function : Compose
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::Compose(const TopAbs_Orientation Orient)
-{
-  myOrient = TopAbs::Compose(myOrient,Orient);
-}
-
-//=======================================================================
-//function : Composed
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::Composed
-  (const TopAbs_Orientation Orient) const
-{
-  TopoDS_Shape S(*this);
-  S.Compose(Orient);
-  return S;
-}
-
-//=======================================================================
-//function : IsPartner
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::IsPartner (const TopoDS_Shape& other) const
-{
-  return (myTShape == other.myTShape);
-}
-
-//=======================================================================
-//function : IsSame
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::IsSame (const TopoDS_Shape& other) const
-{
-  return (myTShape == other.myTShape) &&
-         (myLocation == other.myLocation);
-}
-
-//=======================================================================
-//function : IsEqual
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::IsEqual (const TopoDS_Shape& other) const
-{
-  return (myTShape == other.myTShape) &&
-         (myLocation == other.myLocation) &&
-         (myOrient == other.myOrient);
-}
-
-//=======================================================================
-//function : IsNotEqual
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_Shape::IsNotEqual (const TopoDS_Shape& other) const
-{
-  return !IsEqual(other);
-}
-
-//=======================================================================
-//function : EmptyCopy
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::EmptyCopy()
-{
-  myTShape = myTShape->EmptyCopy();
-}
-
-//=======================================================================
-//function : EmptyCopied
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_Shape TopoDS_Shape::EmptyCopied() const
-{
-  TopoDS_Shape S(*this);
-  S.EmptyCopy();
-  return S;
-}
-  
-//=======================================================================
-//function : TShape
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_Shape::TShape (const Handle(TopoDS_TShape)& TS)
-{
-  myTShape = TS;
-}
-

src/TopoDS/TopoDS_TCompSolid.cxx

@@ -18,7 +18,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopoDS_TCompSolid.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TCompSolid,TopoDS_TShape)
 

src/TopoDS/TopoDS_TCompound.cxx

@@ -18,7 +18,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopoDS_TCompound.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TCompound,TopoDS_TShape)
 

src/TopoDS/TopoDS_TEdge.cxx

@@ -18,6 +18,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopoDS_TEdge.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TEdge,TopoDS_TShape)
 

src/TopoDS/TopoDS_TFace.cxx

@@ -18,7 +18,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopoDS_TFace.hxx>
-#include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TFace,TopoDS_TShape)
 

src/TopoDS/TopoDS_TShape.cxx

@@ -14,11 +14,7 @@
 // Alternatively, this file may be used under the terms of Open CASCADE
 // commercial license or contractual agreement.
 
-
-#include <Standard_ConstructionError.hxx>
-#include <Standard_Type.hxx>
-#include <TopoDS_Builder.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_TShape.hxx>
+#include <TopoDS_Shape.hxx>
 
-IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TShape,Standard_Transient)
+IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TShape,Standard_Transient)

src/TopoDS/TopoDS_TShape.hxx

@@ -17,14 +17,10 @@
 #ifndef _TopoDS_TShape_HeaderFile
 #define _TopoDS_TShape_HeaderFile
 
-#include <Standard.hxx>
 #include <Standard_Type.hxx>
-
+#include <TopAbs.hxx>
-#include <TopoDS_ListOfShape.hxx>
-#include <Standard_Integer.hxx>
-#include <Standard_Transient.hxx>
-#include <Standard_Boolean.hxx>
 #include <TopAbs_ShapeEnum.hxx>
+#include <TopoDS_ListOfShape.hxx>
 
 class TopoDS_Iterator;
 class TopoDS_Builder;
@@ -34,9 +30,6 @@ class TopoDS_Builder;
   #undef Convex
 #endif
 
-class TopoDS_TShape;
-DEFINE_STANDARD_HANDLE(TopoDS_TShape, Standard_Transient)
-
 //! A TShape  is a topological  structure describing a
 //! set of points in a 2D or 3D space.
 //!
@@ -67,55 +60,61 @@ class TopoDS_TShape : public Standard_Transient
 
 public:
 
-  
   //! Returns the free flag.
-    Standard_Boolean Free() const;
+  Standard_Boolean Free() const { return ((myFlags & TopoDS_TShape_Flags_Free) != 0); }
-  
+
   //! Sets the free flag.
-    void Free (const Standard_Boolean F);
+  void Free (Standard_Boolean theIsFree) { setFlag (TopoDS_TShape_Flags_Free, theIsFree); }
-  
+
   //! Returns the locked flag.
-    Standard_Boolean Locked() const;
+  Standard_Boolean Locked() const { return ((myFlags & TopoDS_TShape_Flags_Locked) != 0); }
-  
+
   //! Sets the locked flag.
-    void Locked (const Standard_Boolean F);
+  void Locked (Standard_Boolean theIsLocked) { setFlag (TopoDS_TShape_Flags_Locked, theIsLocked); }
-  
+
   //! Returns the modification flag.
-    Standard_Boolean Modified() const;
+  Standard_Boolean Modified() const { return ((myFlags & TopoDS_TShape_Flags_Modified) != 0); }
-  
+
   //! Sets the modification flag.
-    void Modified (const Standard_Boolean M);
+  void Modified (Standard_Boolean theIsModified)
-  
+  {
+    setFlag (TopoDS_TShape_Flags_Modified, theIsModified);
+    if (theIsModified)
+    {
+      setFlag (TopoDS_TShape_Flags_Checked, false); // when a TShape is modified it is also unchecked
+    }
+  }
+
   //! Returns the checked flag.
-    Standard_Boolean Checked() const;
+  Standard_Boolean Checked() const { return ((myFlags & TopoDS_TShape_Flags_Checked) != 0); }
-  
+
   //! Sets the checked flag.
-    void Checked (const Standard_Boolean C);
+  void Checked (Standard_Boolean theIsChecked) { setFlag (TopoDS_TShape_Flags_Checked, theIsChecked); }
-  
+
   //! Returns the orientability flag.
-    Standard_Boolean Orientable() const;
+  Standard_Boolean Orientable() const { return ((myFlags & TopoDS_TShape_Flags_Orientable) != 0); }
-  
+
   //! Sets the orientability flag.
-    void Orientable (const Standard_Boolean C);
+  void Orientable (Standard_Boolean theIsOrientable) { setFlag (TopoDS_TShape_Flags_Orientable, theIsOrientable); }
-  
+
   //! Returns the closedness flag.
-    Standard_Boolean Closed() const;
+  Standard_Boolean Closed() const { return ((myFlags & TopoDS_TShape_Flags_Closed) != 0); }
-  
+
   //! Sets the closedness flag.
-    void Closed (const Standard_Boolean C);
+  void Closed (Standard_Boolean theIsClosed) { setFlag (TopoDS_TShape_Flags_Closed, theIsClosed); }
-  
+
   //! Returns the infinity flag.
-    Standard_Boolean Infinite() const;
+  Standard_Boolean Infinite() const { return ((myFlags & TopoDS_TShape_Flags_Infinite) != 0); }
-  
+
   //! Sets the infinity flag.
-    void Infinite (const Standard_Boolean C);
+  void Infinite (Standard_Boolean theIsInfinite) { setFlag (TopoDS_TShape_Flags_Infinite, theIsInfinite); }
-  
+
   //! Returns the convexness flag.
-    Standard_Boolean Convex() const;
+  Standard_Boolean Convex() const { return ((myFlags & TopoDS_TShape_Flags_Convex) != 0); }
-  
+
   //! Sets the convexness flag.
-    void Convex (const Standard_Boolean C);
+  void Convex (Standard_Boolean theIsConvex) { setFlag (TopoDS_TShape_Flags_Convex, theIsConvex); }
-  
+
   //! Returns the type as a term of the ShapeEnum enum :
   //! VERTEX, EDGE, WIRE, FACE, ....
   Standard_EXPORT virtual TopAbs_ShapeEnum ShapeType() const = 0;
@@ -123,6 +122,9 @@ public:
   //! Returns a copy  of the  TShape  with no sub-shapes.
   Standard_EXPORT virtual Handle(TopoDS_TShape) EmptyCopy() const = 0;
 
+  //! Returns the number of direct sub-shapes (children).
+  //! @sa TopoDS_Iterator for accessing sub-shapes
+  Standard_Integer NbChildren() const { return myShapes.Size(); }
 
 friend class TopoDS_Iterator;
 friend class TopoDS_Builder;
@@ -132,7 +134,6 @@ friend class TopoDS_Builder;
 
 protected:
 
-  
   //! Constructs an empty TShape.
   //! Free       : True
   //! Modified   : True
@@ -141,28 +142,40 @@ protected:
   //! Closed     : False
   //! Infinite   : False
   //! Convex     : False
-    TopoDS_TShape();
+  TopoDS_TShape()
-
+  : myFlags (TopoDS_TShape_Flags_Free
-
+           | TopoDS_TShape_Flags_Modified
+           | TopoDS_TShape_Flags_Orientable) {}
 
 private:
 
-  
+  // Defined mask values
-    const TopoDS_ListOfShape& Shapes() const;
+  enum TopoDS_TShape_Flags
-  
+  {
-    TopoDS_ListOfShape& ChangeShapes();
+    TopoDS_TShape_Flags_Free       = 0x001,
+    TopoDS_TShape_Flags_Modified   = 0x002,
+    TopoDS_TShape_Flags_Checked    = 0x004,
+    TopoDS_TShape_Flags_Orientable = 0x008,
+    TopoDS_TShape_Flags_Closed     = 0x010,
+    TopoDS_TShape_Flags_Infinite   = 0x020,
+    TopoDS_TShape_Flags_Convex     = 0x040,
+    TopoDS_TShape_Flags_Locked     = 0x080
+  };
+
+  //! Set bit flag.
+  void setFlag (TopoDS_TShape_Flags theFlag,
+                Standard_Boolean    theIsOn)
+  {
+    if (theIsOn) myFlags |=  (Standard_Integer )theFlag;
+    else         myFlags &= ~(Standard_Integer )theFlag;
+  }
+
+private:
 
   TopoDS_ListOfShape myShapes;
-  Standard_Integer myFlags;
+  Standard_Integer   myFlags;
-
-
 };
 
-
+DEFINE_STANDARD_HANDLE(TopoDS_TShape, Standard_Transient)
-#include <TopoDS_TShape.lxx>
-
-
-
-
 
 #endif // _TopoDS_TShape_HeaderFile

src/TopoDS/TopoDS_TShape.lxx

@@ -1,234 +0,0 @@
-// Created on: 1991-03-27
-// Created by: Remi LEQUETTE
-// Copyright (c) 1991-1999 Matra Datavision
-// Copyright (c) 1999-2014 OPEN CASCADE SAS
-//
-// This file is part of Open CASCADE Technology software library.
-//
-// This library is free software; you can redistribute it and/or modify it under
-// the terms of the GNU Lesser General Public License version 2.1 as published
-// by the Free Software Foundation, with special exception defined in the file
-// OCCT_LGPL_EXCEPTION.txt. Consult the file LICENSE_LGPL_21.txt included in OCCT
-// distribution for complete text of the license and disclaimer of any warranty.
-//
-// Alternatively, this file may be used under the terms of Open CASCADE
-// commercial license or contractual agreement.
-
-#include <TopAbs.hxx>
-#include <TopoDS_Shape.hxx>
-#include <TopoDS_ListIteratorOfListOfShape.hxx>
-
-// Defined mask values
-#define TopoDS_TShape_Flags_Free       (1<<0)
-#define TopoDS_TShape_Flags_Modified   (1<<1)
-#define TopoDS_TShape_Flags_Checked    (1<<2)
-#define TopoDS_TShape_Flags_Orientable (1<<3)
-#define TopoDS_TShape_Flags_Closed     (1<<4)
-#define TopoDS_TShape_Flags_Infinite   (1<<5)
-#define TopoDS_TShape_Flags_Convex     (1<<6)
-#define TopoDS_TShape_Flags_Locked     (1<<7)
-
-
-//=======================================================================
-//function : TopoDS_TShape
-//purpose  : Constructor, Empty TShape
-//=======================================================================
-
-inline TopoDS_TShape::TopoDS_TShape()
- : myFlags(TopoDS_TShape_Flags_Free |
-           TopoDS_TShape_Flags_Modified |
-           TopoDS_TShape_Flags_Orientable)
-{
-}
-
-//=======================================================================
-//function : Free
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Free() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Free) != 0);
-}
-
-//=======================================================================
-//function : Free
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Free(const Standard_Boolean F)
-{
-  if (F) myFlags |=  TopoDS_TShape_Flags_Free;
-  else   myFlags &= ~TopoDS_TShape_Flags_Free;
-}
-
-//=======================================================================
-//function : Locked
-//purpose  : 
-//=======================================================================
-inline Standard_Boolean TopoDS_TShape::Locked() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Locked) != 0);
-}
-
-//=======================================================================
-//function : Locked
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Locked(const Standard_Boolean F)
-{
-  if (F) myFlags |=  TopoDS_TShape_Flags_Locked;
-  else   myFlags &= ~TopoDS_TShape_Flags_Locked;
-}
-
-//=======================================================================
-//function : Modified
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Modified() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Modified) != 0);
-}
-
-//=======================================================================
-//function : Modified
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Modified(const Standard_Boolean M)
-{
-  if (M)
-  {
-    myFlags |= TopoDS_TShape_Flags_Modified;
-    // When a TShape is modified it is also unchecked
-    myFlags &= ~TopoDS_TShape_Flags_Checked;
-  }
-  else myFlags &= ~TopoDS_TShape_Flags_Modified;
-}
-
-//=======================================================================
-//function : Checked
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Checked() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Checked) != 0);
-}
-
-//=======================================================================
-//function : Checked
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Checked(const Standard_Boolean M)
-{
-  if (M) myFlags |=  TopoDS_TShape_Flags_Checked;
-  else   myFlags &= ~TopoDS_TShape_Flags_Checked;
-}
-
-//=======================================================================
-//function : Orientable
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Orientable() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Orientable) != 0);
-}
-
-//=======================================================================
-//function : Orientable
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Orientable(const Standard_Boolean M)
-{
-  if (M) myFlags |=  TopoDS_TShape_Flags_Orientable;
-  else   myFlags &= ~TopoDS_TShape_Flags_Orientable;
-}
-
-//=======================================================================
-//function : Closed
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Closed() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Closed) != 0);
-}
-
-//=======================================================================
-//function : Closed
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Closed(const Standard_Boolean M)
-{
-  if (M) myFlags |=  TopoDS_TShape_Flags_Closed;
-  else   myFlags &= ~TopoDS_TShape_Flags_Closed;
-}
-
-//=======================================================================
-//function : Infinite
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Infinite() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Infinite) != 0);
-}
-
-//=======================================================================
-//function : Infinite
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Infinite(const Standard_Boolean M)
-{
-  if (M) myFlags |=  TopoDS_TShape_Flags_Infinite;
-  else   myFlags &= ~TopoDS_TShape_Flags_Infinite;
-}
-
-//=======================================================================
-//function : Convex
-//purpose  : 
-//=======================================================================
-
-inline Standard_Boolean TopoDS_TShape::Convex() const
-{
-  return ((myFlags & TopoDS_TShape_Flags_Convex) != 0);
-}
-
-//=======================================================================
-//function : Convex
-//purpose  : 
-//=======================================================================
-
-inline void TopoDS_TShape::Convex(const Standard_Boolean M)
-{
-  if (M) myFlags |=  TopoDS_TShape_Flags_Convex;
-  else   myFlags &= ~TopoDS_TShape_Flags_Convex;
-}
-
-//=======================================================================
-//function : Shapes
-//purpose  : 
-//=======================================================================
-
-inline const TopoDS_ListOfShape& TopoDS_TShape::Shapes() const
-{
-  return myShapes;
-}
-
-//=======================================================================
-//function : ChangeShapes
-//purpose  : 
-//=======================================================================
-
-inline TopoDS_ListOfShape& TopoDS_TShape::ChangeShapes()
-{
-  return myShapes;
-}

src/TopoDS/TopoDS_TShell.cxx

@@ -17,8 +17,8 @@
 
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
-#include <TopoDS_TShape.hxx>
 #include <TopoDS_TShell.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TShell,TopoDS_TShape)
 

src/TopoDS/TopoDS_TSolid.cxx

@@ -17,8 +17,8 @@
 
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
-#include <TopoDS_TShape.hxx>
 #include <TopoDS_TSolid.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TSolid,TopoDS_TShape)
 

src/TopoDS/TopoDS_TVertex.cxx

@@ -18,6 +18,7 @@
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
 #include <TopoDS_TVertex.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TVertex,TopoDS_TShape)
 

src/TopoDS/TopoDS_TWire.cxx

@@ -17,8 +17,8 @@
 
 #include <Standard_Type.hxx>
 #include <TopAbs.hxx>
-#include <TopoDS_TShape.hxx>
 #include <TopoDS_TWire.hxx>
+#include <TopoDS_Shape.hxx>
 
 IMPLEMENT_STANDARD_RTTIEXT(TopoDS_TWire,TopoDS_TShape)
 

src/TopoDSToStep/TopoDSToStep_MakeFacetedBrep.cxx

@@ -21,7 +21,6 @@
 #include <StepShape_FacetedBrep.hxx>
 #include <StepShape_TopologicalRepresentationItem.hxx>
 #include <TCollection_HAsciiString.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shell.hxx>
 #include <TopoDS_Solid.hxx>
 #include <TopoDSToStep.hxx>

src/TopoDSToStep/TopoDSToStep_MakeGeometricCurveSet.cxx

@@ -23,7 +23,6 @@
 #include <TCollection_HAsciiString.hxx>
 #include <TColStd_HSequenceOfTransient.hxx>
 #include <TopoDS.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <TopoDSToStep.hxx>
 #include <TopoDSToStep_Builder.hxx>

src/TopoDSToStep/TopoDSToStep_MakeStepEdge.cxx

@@ -47,7 +47,6 @@
 #include <TopExp_Explorer.hxx>
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDSToStep_MakeStepEdge.hxx>
 #include <TopoDSToStep_MakeStepVertex.hxx>
 #include <TopoDSToStep_Tool.hxx>
@@ -111,7 +110,6 @@ void TopoDSToStep_MakeStepEdge::Init(const TopoDS_Edge& aEdge,
   }
 
 #define Nbpt 21
-  TopoDS_Iterator  It;
   Standard_Integer i;
   Standard_Real    U, U1, U2;
   gp_Pnt           P;

src/TopoDSToStep/TopoDSToStep_MakeStepFace.cxx

@@ -76,7 +76,6 @@
 #include <TopExp_Explorer.hxx>
 #include <TopoDS.hxx>
 #include <TopoDS_Face.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDSToStep.hxx>
 #include <TopoDSToStep_MakeStepFace.hxx>
 #include <TopoDSToStep_MakeStepWire.hxx>
@@ -152,7 +151,6 @@ void TopoDSToStep_MakeStepFace::Init(const TopoDS_Face& aFace,
     return;
   }
   
-  TopoDS_Iterator  It;
   Standard_Integer i;
   
   //BRepAdaptor_Surface SA = BRepAdaptor_Surface(ForwardFace);  

src/TopoDSToStep/TopoDSToStep_MakeStepWire.cxx

@@ -40,7 +40,6 @@
 #include <TColStd_SequenceOfTransient.hxx>
 #include <TopExp.hxx>
 #include <TopoDS.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Wire.hxx>
 #include <TopoDSToStep_MakeStepEdge.hxx>
 #include <TopoDSToStep_MakeStepVertex.hxx>
@@ -90,7 +89,6 @@ void TopoDSToStep_MakeStepWire::Init(const TopoDS_Wire& aWire,
     return;
   }
 
-  TopoDS_Iterator  It;
   Standard_Integer i;
   
   if (aWire.Orientation() == TopAbs_INTERNAL ||

src/TransferBRep/TransferBRep.cxx

@@ -28,7 +28,6 @@
 #include <TopoDS.hxx>
 #include <TopoDS_Edge.hxx>
 #include <TopoDS_HShape.hxx>
-#include <TopoDS_Iterator.hxx>
 #include <TopoDS_Shape.hxx>
 #include <Transfer_Binder.hxx>
 #include <Transfer_FinderProcess.hxx>

src/XCAFDoc/XCAFDoc_ShapeTool.cxx

@@ -1957,9 +1957,7 @@ Standard_Boolean XCAFDoc_ShapeTool::updateComponent(const TDF_Label& theItemLabe
   // Compare the number of components in XDE structure with the number of
   // components in topological structure. A component may happen to be removed,
   // so we have to update the assembly compound
-  Standard_Integer aNumTopoComponents = 0;
+  Standard_Integer aNumTopoComponents = aCurrentRootShape.NbChildren();
-  for ( TopoDS_Iterator aTopIt(aCurrentRootShape); aTopIt.More(); aTopIt.Next() )
-    aNumTopoComponents++;
   //
   if ( aNumTopoComponents != aComponentLabs.Length() )
     isModified = Standard_True;

src/XCAFPrs/XCAFPrs.cxx

@@ -303,8 +303,7 @@ void XCAFPrs::CollectStyleSettings (const TDF_Label& theLabel,
     TopoDS_Shape aSubshape = XCAFDoc_ShapeTool::GetShape (aLabel);
     if (aSubshape.ShapeType() == TopAbs_COMPOUND)
     {
-      const TopoDS_Iterator aShapeIter (aSubshape);
+      if (aSubshape.NbChildren() == 0)
-      if (!aShapeIter.More())
       {
         continue;
       }

src/XCAFPrs/XCAFPrs_AISObject.cxx

@@ -225,8 +225,7 @@ void XCAFPrs_AISObject::Compute (const Handle(PrsMgr_PresentationManager3d)& the
 
   if (myshape.ShapeType() == TopAbs_COMPOUND)
   {
-    TopoDS_Iterator anExplor (myshape);
+    if (myshape.NbChildren() == 0)
-    if (!anExplor.More())
     {
       return;
     }