Coverity fixed
authormrodrigu <mrodrigu@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 10 Jan 2012 11:19:46 +0000 (11:19 +0000)
committermrodrigu <mrodrigu@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 10 Jan 2012 11:19:46 +0000 (11:19 +0000)
ACORDE/AliACORDEAlign.cxx
ACORDE/AliACORDECalibData.cxx
ACORDE/AliACORDEDataDCS.cxx
ACORDE/AliACORDEQAChecker.cxx

index d344e53..8b6a64b 100644 (file)
@@ -96,19 +96,25 @@ AliACORDEAlign::AliACORDEAlign(const AliACORDEAlign &align):
 {
   //
   //  default copy constructor
+//TObject = align.TObject;
+fFileGlob = align.fFileGlob;
+fRepLoc = align.fRepLoc;
+fRepGlob = align.fRepGlob;
+fUser = align.fUser;
+fX = align.fX;
+fAlignACORDEObjArray = align.fAlignACORDEObjArray;
 fDebug = align.fDebug;
 }
 
 //__________________________________________________________________________
 AliACORDEAlign & AliACORDEAlign::operator =(const AliACORDEAlign &align)
-
 {
   //
   // assignment operator - dummy
 
  //
-fDebug = align.fDebug;
+  if (&align==this) return *this;
+  new (this) AliACORDEAlign(align);
   return (*this);
 }
 
index c3724c5..b92393f 100644 (file)
@@ -30,7 +30,12 @@ fTHits(0),
 fMultiHits(0),
 fTMultiHits(0)
 {
-  
+       for (Int_t imod = 0; imod < 60; imod++)
+       {
+               fEfficiencies[imod]=0.;
+               fRates[imod]=0.;
+               fModulesActivity[imod]=0.;
+       } 
 }
 
 //________________________________________________________________
index a737dcc..ceb7568 100644 (file)
@@ -117,23 +117,12 @@ fIsProcessed(kFALSE)
 //--------------------------------------------------------------
 AliACORDEDataDCS& AliACORDEDataDCS:: operator=(const AliACORDEDataDCS & data) { 
 
-       
-        this->fRun=data.fRun;
-       this->fStartTime=data.fStartTime;
-       this->fEndTime=data.fEndTime;
-       this->fFunc=data.fFunc;
-       this->fIsProcessed=data.fIsProcessed;
-
+       // use ctor copy
 
-        for(int i=0;i<kNAliases;i++){this->fAliasNames[i] = data.fAliasNames[i];}
+       if (&data == this) return *this;
 
-        for(int i=0;i<kNHistos;i++)
-       {
-               this->fHv[i]=data.fHv[i];
-       }
-
-       
-         return *this;
+       new (this) AliACORDEDataDCS(data);
+       return *this;
  
 }
 //---------------------------------------------------------------
index 12377b7..71479a4 100755 (executable)
@@ -84,10 +84,9 @@ fMin(static_cast<TLine*>(qac.fMin->Clone()))
 //____________________________________________________________________________
 AliACORDEQAChecker& AliACORDEQAChecker::operator = (const AliACORDEQAChecker &qac)
 {
-       
-       fTextDQMShifterInfo = new TPaveText(35,0.5,55,0.7,"T");
-       fMax = static_cast<TLine*>(qac.fMax->Clone());
-       fMin = static_cast<TLine*>(qac.fMin->Clone());
+       // use cpy option from constructor
+       if (&qac==this) return *this;
+       new (this) AliACORDEQAChecker(qac);
        return *this;
 }
 //____________________________________________________________________________