Remove some TODOs

diff --git a/src/BuildAPI/BuildAPI_Edge.cpp b/src/BuildAPI/BuildAPI_Edge.cpp
index 5ccb854e0f43fd8068453478fb611bb8c106fd2e..a1a1a03dfd5b7b6a43ef2d3ea335707454426aea 100644 (file)
--- a/src/BuildAPI/BuildAPI_Edge.cpp
+++ b/src/BuildAPI/BuildAPI_Edge.cpp
@@ -39,7 +39,6 @@ void BuildAPI_Edge::setBase(const std::list<ModelHighAPI_Selection>& theBaseObje
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 EdgePtr addEdge(const std::shared_ptr<ModelAPI_Document>& thePart,
                 const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/BuildAPI/BuildAPI_Face.cpp b/src/BuildAPI/BuildAPI_Face.cpp
index 930021d3911b27fe15c8bf0b6bb080d74d281c82..3f5cf71df35a59bd4b8756491590cabd8741457a 100644 (file)
--- a/src/BuildAPI/BuildAPI_Face.cpp
+++ b/src/BuildAPI/BuildAPI_Face.cpp
@@ -39,7 +39,6 @@ void BuildAPI_Face::setBase(const std::list<ModelHighAPI_Selection>& theBaseObje
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 FacePtr addFace(const std::shared_ptr<ModelAPI_Document>& thePart,
                 const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/BuildAPI/BuildAPI_Shell.cpp b/src/BuildAPI/BuildAPI_Shell.cpp
index d504c261ade4229a5dc430e39d2576b760e8c1c5..86151b4cfffabe3f5bb292062cfaf23db19293d4 100644 (file)
--- a/src/BuildAPI/BuildAPI_Shell.cpp
+++ b/src/BuildAPI/BuildAPI_Shell.cpp
@@ -39,7 +39,6 @@ void BuildAPI_Shell::setBase(const std::list<ModelHighAPI_Selection>& theBaseObj
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 ShellPtr addShell(const std::shared_ptr<ModelAPI_Document>& thePart,
                   const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/BuildAPI/BuildAPI_SubShapes.cpp b/src/BuildAPI/BuildAPI_SubShapes.cpp
index b6e2a729bb657082176b3514fa13f11879a34a45..c4eaafb4fe371bf6d79605ad12ead1eba6db4e52 100644 (file)
--- a/src/BuildAPI/BuildAPI_SubShapes.cpp
+++ b/src/BuildAPI/BuildAPI_SubShapes.cpp
@@ -49,7 +49,6 @@ void BuildAPI_SubShapes::setSubShapes(const std::list<ModelHighAPI_Selection>& t
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 SubShapesPtr addSubShapes(const std::shared_ptr<ModelAPI_Document>& thePart,
                           const ModelHighAPI_Selection& theBaseShape,

diff --git a/src/BuildAPI/BuildAPI_Vertex.cpp b/src/BuildAPI/BuildAPI_Vertex.cpp
index a3b71ae5f1e2fcc3665d2fac291b3deb79e9d0d4..b1723c84e3141bcd69e63b42657525827dacdc27 100644 (file)
--- a/src/BuildAPI/BuildAPI_Vertex.cpp
+++ b/src/BuildAPI/BuildAPI_Vertex.cpp
@@ -39,7 +39,6 @@ void BuildAPI_Vertex::setBase(const std::list<ModelHighAPI_Selection>& theBaseOb
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 VertexPtr addVertex(const std::shared_ptr<ModelAPI_Document>& thePart,
                     const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/BuildAPI/BuildAPI_Wire.cpp b/src/BuildAPI/BuildAPI_Wire.cpp
index 66631c74f4e3958db8ae4e02503b84505969f1dd..58a498954f066148c5bd6e68b224efbf4ebc09af 100644 (file)
--- a/src/BuildAPI/BuildAPI_Wire.cpp
+++ b/src/BuildAPI/BuildAPI_Wire.cpp
@@ -45,7 +45,6 @@ void BuildAPI_Wire::addContour()
   feature()->customAction(BuildPlugin_Wire::ADD_CONTOUR_ACTION_ID());
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 WirePtr addWire(const std::shared_ptr<ModelAPI_Document>& thePart,
                 const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/ConnectorAPI/ConnectorAPI_Connector.cpp b/src/ConnectorAPI/ConnectorAPI_Connector.cpp
index 2a48f702c83e0e692f9503eec1f04fcccc8f1bba..1aa191693986e3c06f5f75a80332134489f8cab1 100644 (file)
--- a/src/ConnectorAPI/ConnectorAPI_Connector.cpp
+++ b/src/ConnectorAPI/ConnectorAPI_Connector.cpp
@@ -12,7 +12,7 @@
 //--------------------------------------------------------------------------------------
 void exportToGEOM(const std::shared_ptr<ModelAPI_Document> & thePart)
 {
-  // TODO(spo): check that theConnector is not empty
+  // TODO(spo): check that thePart is not empty
   std::shared_ptr<ModelAPI_Feature> aFeature = thePart->addFeature("ExportToGEOM");
   aFeature->execute();
 }

diff --git a/src/ConstructionAPI/ConstructionAPI_Axis.cpp b/src/ConstructionAPI/ConstructionAPI_Axis.cpp
index 2d42f790e252de0812fa73b61d2ebbeaecf7bfee..352320e1d9a8b551a90e899fcef929eeab73c12b 100644 (file)
--- a/src/ConstructionAPI/ConstructionAPI_Axis.cpp
+++ b/src/ConstructionAPI/ConstructionAPI_Axis.cpp
@@ -89,8 +89,6 @@ void ConstructionAPI_Axis::setPointAndDirection(
 }
 
 //--------------------------------------------------------------------------------------
-// TODO(spo): make add* as static functions of the class
-
 AxisPtr addAxis(const std::shared_ptr<ModelAPI_Document> & thePart,
                 const ModelHighAPI_Selection & thePoint1,
                 const ModelHighAPI_Selection & thePoint2)

diff --git a/src/ConstructionAPI/ConstructionAPI_Plane.cpp b/src/ConstructionAPI/ConstructionAPI_Plane.cpp
index 5799f54985be2fc3c5036ce5132d399d00d146fc..61a72e06141b9c2e591b970908b11999fe970242 100644 (file)
--- a/src/ConstructionAPI/ConstructionAPI_Plane.cpp
+++ b/src/ConstructionAPI/ConstructionAPI_Plane.cpp
@@ -69,8 +69,6 @@ void ConstructionAPI_Plane::setGeneralEquation(
 }
 
 //--------------------------------------------------------------------------------------
-// TODO(spo): make add* as static functions of the class
-
 PlanePtr addPlane(const std::shared_ptr<ModelAPI_Document> & thePart,
                   const ModelHighAPI_Selection & theFace,
                   const ModelHighAPI_Double & theDistance)

diff --git a/src/ConstructionAPI/ConstructionAPI_Point.cpp b/src/ConstructionAPI/ConstructionAPI_Point.cpp
index bcfd9384899eafe093e95a6bb1c059e64931b3b9..ff5a89de3686ed1d515502f8d602c6cafa9f7692 100644 (file)
--- a/src/ConstructionAPI/ConstructionAPI_Point.cpp
+++ b/src/ConstructionAPI/ConstructionAPI_Point.cpp
@@ -45,7 +45,6 @@ void ConstructionAPI_Point::setPoint(const ModelHighAPI_Double & theX,
 }
 
 //--------------------------------------------------------------------------------------
-// TODO(spo): make add* as static functions of the class
 PointPtr addPoint(
     const std::shared_ptr<ModelAPI_Document> & thePart,
     const ModelHighAPI_Double& theX,

diff --git a/src/ExchangeAPI/ExchangeAPI_Import.cpp b/src/ExchangeAPI/ExchangeAPI_Import.cpp
index fe1bf0bf0648435963aa728ba64bc75948c1b500..0c2e7ff7d33fe4ebebaa1a58fe99693550f26a1f 100644 (file)
--- a/src/ExchangeAPI/ExchangeAPI_Import.cpp
+++ b/src/ExchangeAPI/ExchangeAPI_Import.cpp
@@ -39,7 +39,6 @@ void ExchangeAPI_Import::setFilePath(const std::string & theFilePath)
 }
 
 //--------------------------------------------------------------------------------------
-// TODO(spo): make add* as static functions of the class
 ImportPtr addImport(
     const std::shared_ptr<ModelAPI_Document> & thePart,
     const std::string & theFilePath)

diff --git a/src/FeaturesAPI/FeaturesAPI_Boolean.cpp b/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
index 128b9abfcff9ddc064639428ef86689cf16ecc3a..0774fb529c656181ca682e93be99acc83dd00506 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Boolean.cpp
@@ -63,7 +63,6 @@ void FeaturesAPI_Boolean::setToolObjects(const std::list<ModelHighAPI_Selection>
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 BooleanPtr addCut(const std::shared_ptr<ModelAPI_Document>& thePart,
                   const std::list<ModelHighAPI_Selection>& theMainObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp b/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
index 7c993db0be3edc4b1ef40b3dfd56c781c1ba0200..d79749b5f28389b1114568008bcb158d71327588 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Extrusion.cpp
@@ -160,7 +160,6 @@ void FeaturesAPI_Extrusion::setPlanesAndOffsets(const ModelHighAPI_Selection& th
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 ExtrusionPtr addExtrusion(const std::shared_ptr<ModelAPI_Document>& thePart,
                           const std::list<ModelHighAPI_Selection>& theBaseObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_ExtrusionBoolean.cpp b/src/FeaturesAPI/FeaturesAPI_ExtrusionBoolean.cpp
index 92587dc8523e80d767c928e000f74ca405740942..357421b6ee9bf181b292e4f9fb448e875efbd89c 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_ExtrusionBoolean.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_ExtrusionBoolean.cpp
@@ -195,7 +195,6 @@ FeaturesAPI_ExtrusionCut::FeaturesAPI_ExtrusionCut(const std::shared_ptr<ModelAP
   }
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 ExtrusionCutPtr addExtrusionCut(const std::shared_ptr<ModelAPI_Document>& thePart,
                                 const std::list<ModelHighAPI_Selection>& theBaseObjects,
@@ -401,7 +400,6 @@ FeaturesAPI_ExtrusionFuse::FeaturesAPI_ExtrusionFuse(const std::shared_ptr<Model
   }
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 ExtrusionFusePtr addExtrusionFuse(const std::shared_ptr<ModelAPI_Document>& thePart,
                                   const std::list<ModelHighAPI_Selection>& theBaseObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Group.cpp b/src/FeaturesAPI/FeaturesAPI_Group.cpp
index f942c0925b85558b1c6e322e463463c0a6e5d1f9..274fe8e40e9e1d3cdcae2f3fb9f1000edd848d89 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Group.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Group.cpp
@@ -41,7 +41,6 @@ void FeaturesAPI_Group::setGroupList(const std::list<ModelHighAPI_Selection>& th
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 GroupPtr addGroup(const std::shared_ptr<ModelAPI_Document>& thePart,
                   const std::list<ModelHighAPI_Selection>& theGroupList)

diff --git a/src/FeaturesAPI/FeaturesAPI_Intersection.cpp b/src/FeaturesAPI/FeaturesAPI_Intersection.cpp
index cefd80e6e506b47899fa4001f3c66d87337dddd0..ce33f83da978172674e9eff8172225db1796112b 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Intersection.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Intersection.cpp
@@ -51,7 +51,6 @@ void FeaturesAPI_Intersection::setTools(const std::list<ModelHighAPI_Selection>&
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 IntersectionPtr addIntersection(const std::shared_ptr<ModelAPI_Document>& thePart,
                                 const std::list<ModelHighAPI_Selection>& theObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Partition.cpp b/src/FeaturesAPI/FeaturesAPI_Partition.cpp
index b0e3589cdd8cebbf0cb98b8c926ef5b4115f9921..2b4515fc2360dd1f8b0592941853c134e36fc2e0 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Partition.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Partition.cpp
@@ -39,7 +39,6 @@ void FeaturesAPI_Partition::setBase(const std::list<ModelHighAPI_Selection>& the
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 PartitionPtr addPartition(const std::shared_ptr<ModelAPI_Document>& thePart,
                           const std::list<ModelHighAPI_Selection>& theBaseObjects)

diff --git a/src/FeaturesAPI/FeaturesAPI_Pipe.cpp b/src/FeaturesAPI/FeaturesAPI_Pipe.cpp
index a7e49e031f14e7fa710b509d0f97524f1d85ddbb..3ca9a4e54e78263ea85fd030522843c8feeee35e 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Pipe.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Pipe.cpp
@@ -110,7 +110,6 @@ void FeaturesAPI_Pipe::setByBasePathLocations(const std::list<ModelHighAPI_Selec
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 PipePtr addPipe(const std::shared_ptr<ModelAPI_Document>& thePart,
                 const std::list<ModelHighAPI_Selection>& theBaseObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Placement.cpp b/src/FeaturesAPI/FeaturesAPI_Placement.cpp
index e704114179d49c3ec5f7fdd076323663f5952fef..f45264e14dc22ecf832f6dad7a6bc8bec9297926 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Placement.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Placement.cpp
@@ -79,7 +79,6 @@ void FeaturesAPI_Placement::setCentering(const bool theCentering)
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 PlacementPtr addPlacement(const std::shared_ptr<ModelAPI_Document>& thePart,
                           const std::list<ModelHighAPI_Selection>& theObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_RemoveSubShapes.cpp b/src/FeaturesAPI/FeaturesAPI_RemoveSubShapes.cpp
index 79c3c6f0b9b03390ec04d6eb83e14ad3307deaea..4d0726effad4865eb9d5b1e15d591cfc3e7241e3 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_RemoveSubShapes.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_RemoveSubShapes.cpp
@@ -49,7 +49,6 @@ void FeaturesAPI_RemoveSubShapes::setSubShapesToKeep(const std::list<ModelHighAP
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 RemoveSubShapesPtr addRemoveSubShapes(const std::shared_ptr<ModelAPI_Document>& thePart,
                                       const ModelHighAPI_Selection& theBase)

diff --git a/src/FeaturesAPI/FeaturesAPI_Revolution.cpp b/src/FeaturesAPI/FeaturesAPI_Revolution.cpp
index bae189abf281101b2b6215c0c85e5ea0d9344afe..fc982e9c1e00431c407fab17dc869489ae7f62b8 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Revolution.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Revolution.cpp
@@ -120,7 +120,6 @@ void FeaturesAPI_Revolution::setPlanesAndOffsets(const ModelHighAPI_Selection& t
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 RevolutionPtr addRevolution(const std::shared_ptr<ModelAPI_Document>& thePart,
                             const std::list<ModelHighAPI_Selection>& theBaseObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_RevolutionBoolean.cpp b/src/FeaturesAPI/FeaturesAPI_RevolutionBoolean.cpp
index 6590d1acc0568d2ab84f47c60a609d3b6b1456ce..3ca6f2b7d55612da19c0cbd3bfe6816cd2c3e730 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_RevolutionBoolean.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_RevolutionBoolean.cpp
@@ -144,7 +144,6 @@ FeaturesAPI_RevolutionCut::FeaturesAPI_RevolutionCut(const std::shared_ptr<Model
   }
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 RevolutionCutPtr addRevolutionCut(const std::shared_ptr<ModelAPI_Document>& thePart,
                                   const std::list<ModelHighAPI_Selection>& theBaseObjects,
@@ -259,7 +258,6 @@ FeaturesAPI_RevolutionFuse::FeaturesAPI_RevolutionFuse(const std::shared_ptr<Mod
   }
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 RevolutionFusePtr addRevolutionFuse(const std::shared_ptr<ModelAPI_Document>& thePart,
                                     const std::list<ModelHighAPI_Selection>& theBaseObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Rotation.cpp b/src/FeaturesAPI/FeaturesAPI_Rotation.cpp
index 053a10f7df6ab8dac3019d9e50a86ef22d35a670..861f3da0006cb8be4b1f650ff9ffc374cd94de03 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Rotation.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Rotation.cpp
@@ -59,7 +59,6 @@ void FeaturesAPI_Rotation::setAngle(const ModelHighAPI_Double& theAngle)
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 RotationPtr addRotation(const std::shared_ptr<ModelAPI_Document>& thePart,
                         const std::list<ModelHighAPI_Selection>& theMainObjects,

diff --git a/src/FeaturesAPI/FeaturesAPI_Translation.cpp b/src/FeaturesAPI/FeaturesAPI_Translation.cpp
index 5037e4bfd391126b7dc51b40e26810b96b21791b..46a977c964551e26e030b8fd0ba64b2e0b4e54ab 100644 (file)
--- a/src/FeaturesAPI/FeaturesAPI_Translation.cpp
+++ b/src/FeaturesAPI/FeaturesAPI_Translation.cpp
@@ -59,7 +59,6 @@ void FeaturesAPI_Translation::setDistance(const ModelHighAPI_Double& theDistance
   execute();
 }
 
-// TODO(spo): make add* as static functions of the class
 //==================================================================================================
 TranslationPtr addTranslation(const std::shared_ptr<ModelAPI_Document>& thePart,
                               const std::list<ModelHighAPI_Selection>& theMainObjects,

diff --git a/src/ParametersAPI/ParametersAPI_Parameter.cpp b/src/ParametersAPI/ParametersAPI_Parameter.cpp
index fc89a23923fe6a4a987b5af00b6bf6d4e448b94c..0faa1346af1cc6a013a1499e88444aa5ea049016 100644 (file)
--- a/src/ParametersAPI/ParametersAPI_Parameter.cpp
+++ b/src/ParametersAPI/ParametersAPI_Parameter.cpp
@@ -37,12 +37,12 @@ ParametersAPI_Parameter::~ParametersAPI_Parameter()
 }
 
 //--------------------------------------------------------------------------------------
-ParameterPtr addParameter(const std::shared_ptr<ModelAPI_Document> & theParameter,
+ParameterPtr addParameter(const std::shared_ptr<ModelAPI_Document> & thePart,
                           const std::string & theName,
                           const std::string & theExpression,
                           const std::string & theComment)
 {
-  // TODO(spo): check that theParameter is not empty
-  std::shared_ptr<ModelAPI_Feature> aFeature = theParameter->addFeature(ParametersAPI_Parameter::ID());
+  // TODO(spo): check that thePart is not empty
+  std::shared_ptr<ModelAPI_Feature> aFeature = thePart->addFeature(ParametersAPI_Parameter::ID());
   return ParameterPtr(new ParametersAPI_Parameter(aFeature, theName, theExpression, theComment));
 }

diff --git a/src/SketchAPI/SketchAPI_Sketch.h b/src/SketchAPI/SketchAPI_Sketch.h
index 35edecfe2abc47406f69eb0e6223448abf7f2e63..e23f9dd905861f19220192606950b8ff975afeca 100644 (file)
--- a/src/SketchAPI/SketchAPI_Sketch.h
+++ b/src/SketchAPI/SketchAPI_Sketch.h
@@ -75,8 +75,6 @@ public:
   SKETCHAPI_EXPORT
   void setExternal(const ModelHighAPI_Selection & theExternal);
 
-  // TODO(spo): addIntersection
-
   /// Add point
   SKETCHAPI_EXPORT
   std::shared_ptr<SketchAPI_Point> addPoint(