0025077: [OOP] Explicit initialization of copy constructor
authorabv <abv@opencascade.com>
Thu, 25 Sep 2014 09:50:45 +0000 (13:50 +0400)
committerbugmaster <bugmaster@opencascade.com>
Thu, 25 Sep 2014 11:58:14 +0000 (15:58 +0400)
Explicit calls of copy constructor of base class added where compiler warnings are reported

Minor correction

src/BOPDS/BOPDS_PassKey.lxx
src/BOPDS/BOPDS_PassKeyBoolean.lxx
src/Graphic3d/Graphic3d_ClipPlane.cxx
src/Standard/Standard_Failure.cxx

index dabd82b..cffef3b 100644 (file)
@@ -46,23 +46,11 @@ static
 //function :
 //purpose  : 
 //=======================================================================
-  inline BOPDS_PassKey::BOPDS_PassKey(const BOPDS_PassKey& aOther)
+  inline BOPDS_PassKey::BOPDS_PassKey(const BOPDS_PassKey& theOther)
+:
+  myPtr(0)
 {
-  Standard_Integer i;
-  //
-  myPtr=NULL;
-  Clear();
-  //
-  myAllocator=aOther.myAllocator;
-  myNbIds=aOther.myNbIds;
-  mySum=aOther.mySum;
-  //
-  if (myNbIds) {
-    myPtr=Allocate(myNbIds);
-    for (i=0; i<myNbIds; ++i) {
-      myPtr[i]=aOther.myPtr[i];
-    }
-  }
+  (*this) = theOther;
 }
 //=======================================================================
 //function :operator =
@@ -70,8 +58,6 @@ static
 //=======================================================================
   inline BOPDS_PassKey& BOPDS_PassKey::operator =(const BOPDS_PassKey& aOther)
 {
-  Standard_Integer i;
-  //
   Clear();
   //
   myAllocator=aOther.myAllocator;
@@ -80,7 +66,7 @@ static
   //
   if (myNbIds) {
     myPtr=Allocate(myNbIds);
-    for (i=0; i<myNbIds; ++i) {
+    for (int i=0; i<myNbIds; ++i) {
       myPtr[i]=aOther.myPtr[i];
     }
   }
index e3f5b02..abeb0c6 100644 (file)
 //function :
 //purpose  : 
 //=======================================================================
-  inline BOPDS_PassKeyBoolean::BOPDS_PassKeyBoolean(const BOPDS_PassKeyBoolean& aOther)
+  inline BOPDS_PassKeyBoolean::BOPDS_PassKeyBoolean(const BOPDS_PassKeyBoolean& theOther)
+:
+  BOPDS_PassKey(theOther)
 {
-  myPtr=NULL;
-  BOPDS_PassKey::operator =(aOther);
-  myFlag=aOther.myFlag;
+  myFlag = theOther.myFlag;
 }
 //=======================================================================
 //function :~
index 31963d9..71b9325 100755 (executable)
@@ -69,7 +69,8 @@ Graphic3d_ClipPlane::Graphic3d_ClipPlane(const Equation& theEquation)
 // purpose  :
 // =======================================================================
 Graphic3d_ClipPlane::Graphic3d_ClipPlane(const Graphic3d_ClipPlane& theOther)
-: myEquation (theOther.myEquation),
+: Standard_Transient(theOther),
+  myEquation (theOther.myEquation),
   myIsOn (theOther.myIsOn),
   myIsCapping (theOther.myIsCapping),
   myMaterial (theOther.myMaterial),
index 4ec1917..622dbe1 100644 (file)
@@ -78,9 +78,10 @@ Standard_Failure::Standard_Failure (const Standard_CString AString)
   myMessage = allocate_message(AString);
 }
 
-Standard_Failure::Standard_Failure (const Standard_Failure& aFailure) 
+Standard_Failure::Standard_Failure (const Standard_Failure& theFailure) 
+: Standard_Transient(theFailure)
 {
-  myMessage = copy_message(aFailure.myMessage);
+  myMessage = copy_message(theFailure.myMessage);
 }
 
 void Standard_Failure::Destroy()