Architecture changes

diff --git a/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.cpp b/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.cpp
index e01f7d7a97e9aa23d8687e05ed211d26117e8038..3b0d3ca8b2462b5b33812439740a31d5be284cf9 100644 (file)
--- a/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.cpp
+++ b/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.cpp
@@ -14,19 +14,10 @@
 #include <TopTools_ListIteratorOfListOfShape.hxx>
 
 //=================================================================================================
-void GeomAlgoAPI_MakeShape::initialize() {
-  switch (myBuilderType) {
-    case OCCT_BRepBuilderAPI_MakeShape: {
-      myDone = implPtr<BRepBuilderAPI_MakeShape>()->IsDone() == Standard_True;
-      myShape->setImpl(new TopoDS_Shape(implPtr<BRepBuilderAPI_MakeShape>()->Shape()));
-      break;
-    }
-    case OCCT_BOPAlgo_Builder: {
-      myDone = true;
-      myShape->setImpl(new TopoDS_Shape(implPtr<BOPAlgo_Builder>()->Shape()));
-      break;
-    }
-  }
+GeomAlgoAPI_MakeShape::GeomAlgoAPI_MakeShape()
+: myBuilderType(OCCT_BRepBuilderAPI_MakeShape),
+  myDone(false)
+{
 }
 
 //=================================================================================================
@@ -100,15 +91,6 @@ bool GeomAlgoAPI_MakeShape::isDeleted(const std::shared_ptr<GeomAPI_Shape> theSh
   return isDeleted;
 }
 
-
-//=================================================================================================
-GeomAlgoAPI_MakeShape::GeomAlgoAPI_MakeShape()
-: myBuilderType(OCCT_BRepBuilderAPI_MakeShape),
-  myDone(false),
-  myShape(new GeomAPI_Shape())
-{
-}
-
 //=================================================================================================
 void GeomAlgoAPI_MakeShape::setBuilderType(const BuilderType theBuilderType)
 {
@@ -126,3 +108,21 @@ void GeomAlgoAPI_MakeShape::setShape(const std::shared_ptr<GeomAPI_Shape> theSha
 {
   myShape = theShape;
 }
+
+//=================================================================================================
+void GeomAlgoAPI_MakeShape::initialize() {
+  switch (myBuilderType) {
+    case OCCT_BRepBuilderAPI_MakeShape: {
+      myDone = implPtr<BRepBuilderAPI_MakeShape>()->IsDone() == Standard_True;
+      myShape.reset(new GeomAPI_Shape());
+      myShape->setImpl(new TopoDS_Shape(implPtr<BRepBuilderAPI_MakeShape>()->Shape()));
+      break;
+    }
+    case OCCT_BOPAlgo_Builder: {
+      myDone = true;
+      myShape.reset(new GeomAPI_Shape());
+      myShape->setImpl(new TopoDS_Shape(implPtr<BOPAlgo_Builder>()->Shape()));
+      break;
+    }
+  }
+}

diff --git a/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.h b/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.h
index 07e618263ddc989055203569c8a86f47459abd1b..272f9f9750f468786ba35f961d0bec8060713a32 100644 (file)
--- a/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.h
+++ b/src/GeomAlgoAPI/GeomAlgoAPI_MakeShape.h
@@ -26,18 +26,30 @@ public:
   };
 
 public:
+  /// \brief Empty constructor.
+  GEOMALGOAPI_EXPORT GeomAlgoAPI_MakeShape();
+
   /** \brief Constructor by builder and builder type.
    *  \param[in] theBuilder pointer to the builder.
    *  \param[in] theBuilderType builder type.
    */
   template<class T> explicit GeomAlgoAPI_MakeShape(T* theBuilder, const BuilderType theBuilderType = OCCT_BRepBuilderAPI_MakeShape)
   : GeomAPI_Interface(theBuilder),
-    myBuilderType(theBuilderType),
-    myShape(new GeomAPI_Shape())
+    myBuilderType(theBuilderType)
   {
     initialize();
   }
 
+  /** \brief Initializes internals.
+   *  \param[in] theBuilder pointer to the builder.
+   *  \param[in] theBuilderType builder type.
+   */
+  template<class T> void initialize(T* theBuilder, const BuilderType theBuilderType = OCCT_BRepBuilderAPI_MakeShape)
+  {
+    setImpl(theBuilder);
+    myBuilderType = theBuilder;
+    initialize();
+  }
 
   /// \return status of builder.
   GEOMALGOAPI_EXPORT bool isDone() const;
@@ -65,9 +77,6 @@ public:
   GEOMALGOAPI_EXPORT virtual bool isDeleted(const std::shared_ptr<GeomAPI_Shape> theShape);
 
 protected:
-  /// \brief Default constructor.
-  GEOMALGOAPI_EXPORT GeomAlgoAPI_MakeShape();
-
   /** \brief Sets builder type.
    *  \param[in] theBuilderType new builder type.
    */
@@ -83,8 +92,9 @@ protected:
    */
   GEOMALGOAPI_EXPORT void setShape(const std::shared_ptr<GeomAPI_Shape> theShape);
 
+private:
   /// \brief Initializes internals.
-  GEOMALGOAPI_EXPORT void initialize();
+  void initialize();
 
 private:
   GeomAlgoAPI_MakeShape::BuilderType myBuilderType; ///< Type of make shape builder.

diff --git a/src/GeomAlgoAPI/GeomAlgoAPI_MakeSweep.h b/src/GeomAlgoAPI/GeomAlgoAPI_MakeSweep.h
index a25bbe681b605e23f43cc51bc9d7bababa77eb17..fee768599be12c4c3ca64f3006f580432b4ddef2 100644 (file)
--- a/src/GeomAlgoAPI/GeomAlgoAPI_MakeSweep.h
+++ b/src/GeomAlgoAPI/GeomAlgoAPI_MakeSweep.h
@@ -35,7 +35,7 @@ public:
 protected:
   GeomAlgoAPI_MakeSweep(){};
 
-protected:
+private:
   ListOfShape myFromFaces;
   ListOfShape myToFaces;
 };