0031020: Coding - invalid inline usage in BRepBlend_AppSurface
authormsv <msv@opencascade.com>
Tue, 1 Oct 2019 20:06:35 +0000 (23:06 +0300)
committerapn <apn@opencascade.com>
Tue, 22 Oct 2019 09:12:25 +0000 (12:12 +0300)
BRepBlend_AppSurface and some other classes have been corrected to avoid inlining member functions in .cxx code.

src/BRepBlend/BRepBlend_AppSurface.cxx
src/BRepMesh/BRepMesh_Delaun.cxx
src/MeshVS/MeshVS_DummySensitiveEntity.cxx
src/NCollection/NCollection_BaseSequence.cxx
src/NCollection/NCollection_BaseSequence.hxx
src/ShapeFix/ShapeFix_Edge.cxx

index 9126000..460d599 100644 (file)
@@ -126,8 +126,8 @@ Standard_Real BRepBlend_AppSurface::TolCurveOnSurf(const Standard_Integer Index)
   return approx.TolCurveOnSurf(Index);
 }
 
-inline void  BRepBlend_AppSurface::TolReached (Standard_Real& Tol3d,
-                                        Standard_Real& Tol2d) const
+void  BRepBlend_AppSurface::TolReached (Standard_Real& Tol3d,
+                                        Standard_Real& Tol2d) const
 {
   Tol3d = approx.MaxErrorOnSurf();
   Tol2d = 0;
index c87bc6c..761dabd 100644 (file)
@@ -1224,9 +1224,9 @@ Standard_Boolean BRepMesh_Delaun::checkIntersection(
 //function : addTriangle
 //purpose  : Add a triangle based on the given oriented edges into mesh
 //=======================================================================
-inline void BRepMesh_Delaun::addTriangle( const Standard_Integer (&theEdgesId)[3],
-                                          const Standard_Boolean (&theEdgesOri)[3],
-                                          const Standard_Integer (&theNodesId)[3] )
+void BRepMesh_Delaun::addTriangle( const Standard_Integer (&theEdgesId)[3],
+                                   const Standard_Boolean (&theEdgesOri)[3],
+                                   const Standard_Integer (&theNodesId)[3] )
 {
   Standard_Integer aNewTriangleId = 
     myMeshData->AddElement(BRepMesh_Triangle(theEdgesId, 
@@ -1890,7 +1890,7 @@ void BRepMesh_Delaun::meshPolygon(IMeshData::SequenceOfInteger&   thePolygon,
 //function : meshElementaryPolygon
 //purpose  : Triangulation of closed polygon containing only three edges.
 //=======================================================================
-inline Standard_Boolean BRepMesh_Delaun::meshElementaryPolygon( 
+Standard_Boolean BRepMesh_Delaun::meshElementaryPolygon(
   const IMeshData::SequenceOfInteger& thePolygon)
 {
   Standard_Integer aPolyLen = thePolygon.Length();
index 143f277..b02f25a 100644 (file)
@@ -80,7 +80,7 @@ Standard_Boolean MeshVS_DummySensitiveEntity::HasInitLocation() const
 //function : InvInitLocation
 //purpose  :
 //=======================================================================
-inline gp_GTrsf MeshVS_DummySensitiveEntity::InvInitLocation() const
+gp_GTrsf MeshVS_DummySensitiveEntity::InvInitLocation() const
 {
   return gp_GTrsf();
 }
index 6932152..08a0a45 100644 (file)
 #include <Standard_OutOfRange.hxx>
 #include <Standard_DomainError.hxx>
 
-inline void NCollection_BaseSequence::Nullify ()
-{
-  myFirstItem = myLastItem = myCurrentItem = NULL;
-  myCurrentIndex = mySize = 0;
-}
-
 //=======================================================================
 //function : ClearSeq
 //purpose  : removes all items from the current sequence
index 42f4197..bbb0df3 100644 (file)
@@ -163,7 +163,12 @@ public:
   // 
   Standard_EXPORT NCollection_BaseSequence
                            (const NCollection_BaseSequence& Other);
-  inline void Nullify ();
+  void Nullify()
+  {
+    myFirstItem = myLastItem = myCurrentItem = NULL;
+    myCurrentIndex = mySize = 0;
+  }
+
   friend class Iterator;
 };
 
index d7ff00f..1cbde04 100644 (file)
@@ -862,7 +862,7 @@ Standard_Boolean ShapeFix_Edge::FixSameParameter(const TopoDS_Edge& edge,
 //purpose  : 
 //=======================================================================
 
-inline Handle(ShapeBuild_ReShape) ShapeFix_Edge::Context() const
+Handle(ShapeBuild_ReShape) ShapeFix_Edge::Context() const
 {
   return myContext;
 }