Coverity fixes
authorkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 23 Feb 2012 16:47:56 +0000 (16:47 +0000)
committerkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 23 Feb 2012 16:47:56 +0000 (16:47 +0000)
PHOS/AliPHOSDigitizer.h
PHOS/AliPHOSEmcRecPoint.h
PHOS/AliPHOSPulseGenerator.h
PHOS/AliPHOSRecPoint.cxx

index 181ce37..68ca72d 100644 (file)
@@ -67,12 +67,10 @@ public:
 
   void    Print(const Option_t * = "")const ;
  
+private:
   AliPHOSDigitizer & operator = (const AliPHOSDigitizer & /*rvalue*/)  {
-    // assignement operator requested by coding convention but not needed
-    Fatal("operator =", "not implemented") ;
     return *this ; 
   }
-
   
 private:
 
index e8b17fa..d96c11a 100644 (file)
@@ -93,12 +93,10 @@ public:
   Float_t     GetDistanceToBadCrystal() const {return fDistToBadCrystal;}
   void        SetDistanceToBadCrystal(Float_t dist) {fDistToBadCrystal=dist;}
 
-  AliPHOSEmcRecPoint & operator = (const AliPHOSEmcRecPoint & /*rvalue*/)  {
-    Fatal("operator =", "not implemented");
-    return *this ;
-  }
+private:
+  AliPHOSEmcRecPoint & operator = (const AliPHOSEmcRecPoint & /*rvalue*/) {return *this;}
 
- protected:
+protected:
   virtual void  EvalLocalPosition(Float_t logWeight, TVector3 &vtx, TClonesArray * digits, TVector3 &vInc) ;// computes the position in the PHOS module 
   virtual void  EvalDispersion(Float_t logWeight, TClonesArray * digits, TVector3 &vInc) ;   // computes the dispersion of the shower
   virtual void  EvalElipsAxis(Float_t logWeight, TClonesArray * digits, TVector3 &vInc );   // computes the axis of shower ellipsoide
index 16278d8..30338a1 100644 (file)
@@ -47,10 +47,8 @@ public:
   virtual void Print(Option_t*) const;
   virtual void Draw (Option_t* opt = "all");
 
-  AliPHOSPulseGenerator& operator = (const AliPHOSPulseGenerator &) {
-    Fatal("operator =", "not implemented") ;
-    return *this;
-  }
+private:
+  AliPHOSPulseGenerator& operator = (const AliPHOSPulseGenerator &) {return *this;}
 
 private:
   static Int_t    fgOrder ;             // order of the gamma function
index ae5e5b9..e34210e 100644 (file)
@@ -97,19 +97,21 @@ AliPHOSRecPoint& AliPHOSRecPoint::operator= (const AliPHOSRecPoint &rp)
 {
   if(&rp == this) return *this;
 
-  fPHOSMod = rp.fPHOSMod;
-  fMulTrack = rp.fMulTrack;
-  fMaxDigit = rp.fMaxDigit;
-  fMulDigit = rp.fMulDigit;
-  fMaxTrack = rp.fMaxTrack;
-  fAmp = rp.fAmp;
+  fPHOSMod     = rp.fPHOSMod;
+  fMulTrack    = rp.fMulTrack;
+  fMaxDigit    = rp.fMaxDigit;
+  fMulDigit    = rp.fMulDigit;
+  fMaxTrack    = rp.fMaxTrack;
+  fAmp         = rp.fAmp;
   fIndexInList = rp.fIndexInList; 
   fLocPos = rp.fLocPos;
 
+  if (fDigitsList != 0) delete [] fDigitsList;
   if (rp.fMulDigit>0) fDigitsList = new Int_t[rp.fMulDigit];
   for(Int_t i=0; i<fMaxDigit; i++)
     fDigitsList[i] = rp.fDigitsList[i];
 
+  if (fTracksList != 0) delete [] fTracksList;
   if (rp.fMulTrack>0) fTracksList = new Int_t[rp.fMulTrack];
   for(Int_t i=0; i<fMaxTrack; i++)
     fTracksList[i] = rp.fTracksList[i];