Hopefully now Coverity will be happy
authorlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 17 Feb 2012 08:39:40 +0000 (08:39 +0000)
committerlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 17 Feb 2012 08:39:40 +0000 (08:39 +0000)
MUON/AliMUONPolygon.cxx
MUON/AliMUONPolygon.h
MUON/AliMUONRejectList.cxx
MUON/AliMUONRejectList.h

index c2d7471..4d748f6 100644 (file)
@@ -76,13 +76,25 @@ AliMUONPolygon::~AliMUONPolygon()
 //______________________________________________________________________________
 AliMUONPolygon::AliMUONPolygon(const AliMUONPolygon& rhs) 
 : TObject(rhs), 
-fN(0),
+fN(rhs.fN),
 fX(0x0),
 fY(0x0)
 {
   /// Copy constructor.
   
-  ((AliMUONPolygon&)rhs).AliMUONPolygon::Copy(*this);
+  if ( fN > 0 ) 
+  {
+    fX = new Double_t[fN];
+    fY = new Double_t[fN];
+    
+    for ( Int_t i = 0; i < fN; ++i )
+    {
+      fX[i] = rhs.fX[i];
+      fY[i] = rhs.fY[i];
+    }
+    
+  }
+  
 }
 
 //______________________________________________________________________________
@@ -92,7 +104,28 @@ AliMUONPolygon::operator=(const AliMUONPolygon& rhs)
   /// Assignment operator
   if ( this != &rhs ) 
   {
-    rhs.AliMUONPolygon::Copy(*this);
+    static_cast<TObject&>(*this)=rhs;
+
+    delete[] fX;
+    delete[] fY;
+    
+    fX = 0;
+    fY = 0;
+    
+    fN = rhs.fN;
+    
+    if ( fN > 0 ) 
+    {
+      fX = new Double_t[fN];
+      fY = new Double_t[fN];
+      
+      for ( Int_t i = 0; i < fN; ++i )
+      {
+        fX[i] = rhs.fX[i];
+        fY[i] = rhs.fY[i];
+      }
+      
+    }
   }
   return *this;
 }
@@ -109,37 +142,6 @@ AliMUONPolygon::Contains(Double_t x, Double_t y) const
   return TMath::IsInside(x,y,fN,fX,fY);
 }
 
-//______________________________________________________________________________
-void AliMUONPolygon::Copy(TObject& obj) const
-{
-  /// Copy this to obj
-  
-  TObject::Copy(obj);
-
-  AliMUONPolygon& rhs = static_cast<AliMUONPolygon&>(obj);
-  
-  delete [] rhs.fX;
-  delete [] rhs.fY;
-
-  rhs.fX = 0;
-  rhs.fY = 0;
-  rhs.fN = 0;
-  
-  if ( fN > 0 )
-  {
-    rhs.fN = fN;
-
-    rhs.fX = new Double_t[rhs.fN];
-    rhs.fY = new Double_t[rhs.fN];
-    
-    for ( Int_t i = 0; i < rhs.fN; ++i )
-    {
-      rhs.fX[i] = fX[i];
-      rhs.fY[i] = fY[i];
-    }
-  }  
-}
-
 //_____________________________________________________________________________
 void
 AliMUONPolygon::Close()
index 745d35f..2b1c0cd 100644 (file)
@@ -50,8 +50,6 @@ public:
   
   void Print(Option_t* opt="") const;
   
-  void Copy(TObject& obj) const;
-  
   void Close();
   
 private:
index 0ac51c4..2906431 100644 (file)
@@ -148,7 +148,7 @@ namespace
     if ( src && n )
     {
       dest = new UInt_t[n];
-      memcpy(src,dest,n*sizeof(UInt_t));
+      memcpy(dest,src,n*sizeof(UInt_t));
     }
   }
 
@@ -160,7 +160,7 @@ namespace
     if ( src && n )
     {
       dest = new Float_t[n];
-      memcpy(src,dest,n*sizeof(Float_t));
+      memcpy(dest,src,n*sizeof(Float_t));
     }
   }
   
@@ -217,13 +217,13 @@ fChannels(0x0)
 //_____________________________________________________________________________
 AliMUONRejectList::AliMUONRejectList(const AliMUONRejectList& rl)
 : TObject(rl),
-fIsBinary(kTRUE),
-fMaxNofDEs(),
-fMaxNofBPs(),
-fMaxNofManus(),
-fNofDEs(), 
-fNofBPs(), 
-fNofManus(0), 
+fIsBinary(rl.fIsBinary),
+fMaxNofDEs(rl.fMaxNofDEs),
+fMaxNofBPs(rl.fMaxNofBPs),
+fMaxNofManus(rl.fMaxNofManus),
+fNofDEs(rl.fNofDEs), 
+fNofBPs(rl.fNofBPs), 
+fNofManus(rl.fNofManus), 
 fDEIds(0x0),
 fDEProbas(0x0),
 fBPIds(0x0),
@@ -233,7 +233,18 @@ fManuProbas(0x0),
 fChannels(0x0)
 {
   /// Copy ctor
-  rl.CopyTo(*this);
+  
+  ::Copy(rl.fMaxNofDEs,rl.fDEIds,fDEIds);
+  ::Copy(rl.fMaxNofDEs,rl.fDEProbas,fDEProbas);
+  ::Copy(rl.fMaxNofBPs,rl.fBPIds,fBPIds);
+  ::Copy(rl.fMaxNofBPs,rl.fBPProbas,fBPProbas);
+  ::Copy(rl.fMaxNofManus,rl.fManuIds,fManuIds);
+  ::Copy(rl.fMaxNofManus,rl.fManuProbas,fManuProbas);
+  
+  if ( rl.fChannels ) 
+  {
+    fChannels = static_cast<AliMUONVStore*>(rl.fChannels->Clone());
+  }
 }
 
 //_____________________________________________________________________________
@@ -242,37 +253,36 @@ AliMUONRejectList& AliMUONRejectList::operator=(const AliMUONRejectList& rl)
   /// assignement operator
   if ( this != &rl ) 
   {
-    rl.CopyTo(*this);
+    static_cast<TObject&>(*this)=rl;
+    
+    fIsBinary = rl.fIsBinary;
+    fMaxNofDEs = rl.fMaxNofDEs;
+    fMaxNofBPs = rl.fMaxNofBPs;
+    fMaxNofManus = rl.fMaxNofManus;
+    fNofDEs = rl.fNofDEs;
+    fNofBPs = rl.fNofBPs;
+    fNofManus = rl.fNofManus;
+    
+    ::Copy(rl.fMaxNofDEs,rl.fDEIds,fDEIds);
+    ::Copy(rl.fMaxNofDEs,rl.fDEProbas,fDEProbas);
+    ::Copy(rl.fMaxNofBPs,rl.fBPIds,fBPIds);
+    ::Copy(rl.fMaxNofBPs,rl.fBPProbas,fBPProbas);
+    ::Copy(rl.fMaxNofManus,rl.fManuIds,fManuIds);
+    ::Copy(rl.fMaxNofManus,rl.fManuProbas,fManuProbas);
+    
+    delete fChannels;
+    fChannels = 0x0;
+    
+    if ( rl.fChannels ) 
+    {
+      fChannels = static_cast<AliMUONVStore*>(rl.fChannels->Clone());
+    }
+    
   }  
   return *this;
 }
 
 //_____________________________________________________________________________
-void 
-AliMUONRejectList::CopyTo(AliMUONRejectList& rl) const
-{
-  /// Copy this to rl
-  TObject::Copy(rl);
-
-  rl.fIsBinary = fIsBinary;
-
-  ::Copy(rl.fMaxNofDEs,fDEIds,rl.fDEIds);
-  ::Copy(rl.fMaxNofDEs,fDEProbas,rl.fDEProbas);
-  ::Copy(rl.fMaxNofBPs,fBPIds,rl.fBPIds);
-  ::Copy(rl.fMaxNofBPs,fBPProbas,rl.fBPProbas);
-  ::Copy(rl.fMaxNofManus,fManuIds,rl.fManuIds);
-  ::Copy(rl.fMaxNofManus,fManuProbas,rl.fManuProbas);
-  
-  delete rl.fChannels;  
-  rl.fChannels = 0x0;
-  
-  if ( fChannels ) 
-  {
-    rl.fChannels = static_cast<AliMUONVStore*>(fChannels->Clone());
-  }
-}
-
-//_____________________________________________________________________________
 AliMUONRejectList::~AliMUONRejectList()
 {
   /// dtor
index 04e48ca..f8245b6 100644 (file)
@@ -47,8 +47,6 @@ public:
   void Print(Option_t* opt="") const;
   
 private:
-  void CopyTo(AliMUONRejectList& destination) const;
-
   void ZeroOrOne(Float_t proba);
   
 private: