0030553: Coding - TopOpeBRepDS_Surface missing default initalizations
authorkgv <kgv@opencascade.com>
Mon, 11 Mar 2019 20:20:44 +0000 (23:20 +0300)
committerapn <apn@opencascade.com>
Wed, 13 Mar 2019 08:00:22 +0000 (11:00 +0300)
All class fields are now initialized within every TopOpeBRepDS_Surface constructor.

src/TopOpeBRepDS/TopOpeBRepDS_Surface.cxx
src/TopOpeBRepDS/TopOpeBRepDS_Surface.hxx

index e495b93..1a0eddd 100644 (file)
 // Alternatively, this file may be used under the terms of Open CASCADE
 // commercial license or contractual agreement.
 
+#include <TopOpeBRepDS_Surface.hxx>
 
 #include <Geom_Surface.hxx>
-#include <TopOpeBRepDS_Surface.hxx>
 
 //=======================================================================
 //function : TopOpeBRepDS_Surface
 //purpose  : 
 //=======================================================================
 TopOpeBRepDS_Surface::TopOpeBRepDS_Surface()
+: myTolerance (0.0),
+  myKeep (Standard_False)
 {
 }
 
@@ -31,24 +33,26 @@ TopOpeBRepDS_Surface::TopOpeBRepDS_Surface()
 //purpose  : 
 //=======================================================================
 
-TopOpeBRepDS_Surface::TopOpeBRepDS_Surface
-  (const Handle(Geom_Surface)& P,
-   const Standard_Real T) :
-       mySurface(P),
-       myTolerance(T)
+TopOpeBRepDS_Surface::TopOpeBRepDS_Surface (const Handle(Geom_Surface)& theSurface,
+                                            const Standard_Real theTolerance)
+: mySurface (theSurface),
+  myTolerance (theTolerance),
+  myKeep (Standard_False)
 {
 }
 
-//
-//modified by NIZNHY-PKV Tue Oct 30 09:28:33 2001 f
 //=======================================================================
 //function : TopOpeBRepDS_Surface::TopOpeBRepDS_Surface
 //purpose  : 
 //=======================================================================
-TopOpeBRepDS_Surface::TopOpeBRepDS_Surface (const TopOpeBRepDS_Surface& Other) 
+TopOpeBRepDS_Surface::TopOpeBRepDS_Surface (const TopOpeBRepDS_Surface& theOther)
+: mySurface (theOther.mySurface),
+  myTolerance (theOther.myTolerance),
+  myKeep (theOther.myKeep)
 {
-  Assign(Other);
+  //
 }
+
 //=======================================================================
 //function : Assign
 //purpose  : 
@@ -59,55 +63,3 @@ void TopOpeBRepDS_Surface::Assign(const TopOpeBRepDS_Surface& Other)
   myTolerance=Other.myTolerance;
   myKeep=Other.myKeep;
 }
-//modified by NIZNHY-PKV Tue Oct 30 09:28:20 2001 t
-//
-
-//=======================================================================
-//function : Surface
-//purpose  : 
-//=======================================================================
-
-const Handle(Geom_Surface)&  TopOpeBRepDS_Surface::Surface()const 
-{
-  return mySurface;
-}
-
-
-//=======================================================================
-//function : Tolerance
-//purpose  : 
-//=======================================================================
-
-Standard_Real  TopOpeBRepDS_Surface::Tolerance()const 
-{
-  return myTolerance;
-}
-
-//=======================================================================
-//function : Tolerance
-//purpose  : 
-//=======================================================================
-
-void TopOpeBRepDS_Surface::Tolerance(const Standard_Real T)
-{
-  myTolerance = T;
-}
-
-//=======================================================================
-//function : Keep
-//purpose  : 
-//=======================================================================
-
-Standard_Boolean TopOpeBRepDS_Surface::Keep() const
-{
-  return myKeep;
-}
-//=======================================================================
-//function : ChangeKeep
-//purpose  : 
-//=======================================================================
-
-void TopOpeBRepDS_Surface::ChangeKeep(const Standard_Boolean b)
-{
-  myKeep = b;
-}
index 1524485..8799d11 100644 (file)
@@ -25,7 +25,6 @@
 #include <Standard_Boolean.hxx>
 class Geom_Surface;
 
-
 //! A Geom surface and a tolerance.
 class TopOpeBRepDS_Surface 
 {
@@ -33,7 +32,6 @@ public:
 
   DEFINE_STANDARD_ALLOC
 
-  
   Standard_EXPORT TopOpeBRepDS_Surface();
   
   Standard_EXPORT TopOpeBRepDS_Surface(const Handle(Geom_Surface)& P, const Standard_Real T);
@@ -45,42 +43,24 @@ void operator= (const TopOpeBRepDS_Surface& Other)
 {
   Assign(Other);
 }
-  
-  Standard_EXPORT const Handle(Geom_Surface)& Surface() const;
-  
-  Standard_EXPORT Standard_Real Tolerance() const;
-  
-  //! Update the tolerance
-  Standard_EXPORT void Tolerance (const Standard_Real tol);
-  
-  Standard_EXPORT Standard_Boolean Keep() const;
-  
-  Standard_EXPORT void ChangeKeep (const Standard_Boolean B);
-
-
-
 
-protected:
+  const Handle(Geom_Surface)& Surface() const { return mySurface; }
 
+  Standard_Real Tolerance() const { return myTolerance; }
 
+  //! Update the tolerance
+  void Tolerance (Standard_Real theTol) { myTolerance = theTol; }
 
+  Standard_Boolean Keep() const { return myKeep; }
 
+  void ChangeKeep (Standard_Boolean theToKeep) { myKeep = theToKeep; }
 
 private:
 
-
-
   Handle(Geom_Surface) mySurface;
   Standard_Real myTolerance;
   Standard_Boolean myKeep;
 
-
 };
 
-
-
-
-
-
-
 #endif // _TopOpeBRepDS_Surface_HeaderFile