Coverity fixes for :
authorjthaeder <jthaeder@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 14 Dec 2011 14:31:02 +0000 (14:31 +0000)
committerjthaeder <jthaeder@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 14 Dec 2011 14:31:02 +0000 (14:31 +0000)
18405, 18403, 18402, 18401, 18349, 18346, 18345, 18343, 17922, 18249

TPC/AliTPCLaserTracks.cxx
TPC/AliTPCLaserTracks.h
TPC/AliTPCParamCR.cxx
TPC/AliTPCParamCR.h
TPC/AliTPCParamSR.cxx
TPC/AliTPCParamSR.h
TPC/AliTPCcalibTracks.cxx
TPC/AliTPCclustererMI.cxx
TPC/AliTPCclustererMI.h

index fee43ab..bedb8a0 100644 (file)
@@ -115,41 +115,6 @@ AliTPCLaserTracks::AliTPCLaserTracks(Int_t npoints):TObject(),
     fNpoints = npoints;
     InitPoints();
 }
-//______________________________________________________________________
-AliTPCLaserTracks::AliTPCLaserTracks(const AliTPCLaserTracks &param):TObject(),
-    fId(-1),
-    fSide(-1),
-    fRod(-1),
-    fBundle(-1),
-    fBeam(-1),
-    fX(0.),
-    fY(0.),
-    fZ(0.),
-    fTime(0.),
-    fPhi(0.),
-    fTheta(0.),
-    fMaxSize(0),
-    fNpoints(0),
-    fXarr(0x0),
-    fYarr(0x0),
-    fZarr(0x0)
-{
-  //
-  // dummy
-  //
-  fPhi=param.fPhi;
-}
-//______________________________________________________________________
-AliTPCLaserTracks & AliTPCLaserTracks::operator=(const AliTPCLaserTracks & param)
-{
-  //
-  // assignment operator - dummy
-  //
-  if (this == &param) return (*this);
-
-  fPhi=param.fPhi;
-  return (*this);
-}
 
 //_______________________________________________________________________
 Double_t AliTPCLaserTracks::FindBeamLength(TVector3 vF, TVector3 vP)
index 2ddcd7c..f233160 100644 (file)
@@ -23,8 +23,6 @@ public:
     AliTPCLaserTracks();
     AliTPCLaserTracks(Int_t npoints);
     virtual ~AliTPCLaserTracks();
-    AliTPCLaserTracks(const AliTPCLaserTracks &param); // copy constructor
-    AliTPCLaserTracks &operator = (const AliTPCLaserTracks & param);
 
     Int_t GetId()    const {return fId;     }
     Int_t GetSide()  const {return fSide;   }
@@ -89,6 +87,10 @@ protected:
 
     Double_t FindBeamLength(TVector3 vS, TVector3 vP);
 
+private:
+    AliTPCLaserTracks(const AliTPCLaserTracks &param); // copy constructor
+    AliTPCLaserTracks &operator = (const AliTPCLaserTracks & param);
+
     ClassDef(AliTPCLaserTracks,1)
 };
 
index d00cac8..6852426 100644 (file)
@@ -56,29 +56,7 @@ AliTPCParamCR::AliTPCParamCR()
   SetDefault();
   Update();
 }
-AliTPCParamCR::AliTPCParamCR(const AliTPCParamCR &param)
-              :AliTPCParam(),
-              fInnerPRF(0),
-              fOuter1PRF(0),
-              fOuter2PRF(0),
-               fTimeRF(0),
-              fFacSigma(0.)
-{
-  //
-  // copy constructor - dummy
-  //
-  fFacSigma= param.fFacSigma;
-}
-AliTPCParamCR & AliTPCParamCR::operator =(const AliTPCParamCR & param)
-{
-  //
-  // assignment operator - dummy
-  //
-  if (this == &param) return (*this);
 
-  fFacSigma= param.fFacSigma;
-  return (*this); 
-}
 AliTPCParamCR::~AliTPCParamCR()
 {
   //
index bc0ced4..a00d366 100644 (file)
@@ -16,8 +16,6 @@ class AliTPCPRF2D;
 class AliTPCParamCR : public AliTPCParam {
 public:
   AliTPCParamCR();
-  AliTPCParamCR(const AliTPCParamCR &param); // copy constructor
-  AliTPCParamCR &operator = (const AliTPCParamCR & param); //assignment operator
   virtual ~AliTPCParamCR();
   Int_t  CalcResponse(Float_t* x, Int_t * index, Int_t dummy=0);
   //calculate bin response as function of the input position -x 
@@ -51,6 +49,11 @@ protected:
   AliTPCPRF2D * fOuter2PRF;  
   AliTPCRF1D  * fTimeRF;           //!time response function object
   Float_t       fFacSigma;         //factor-how many sigma of response I accept
+
+private:
+  AliTPCParamCR(const AliTPCParamCR &param); // copy constructor
+  AliTPCParamCR &operator = (const AliTPCParamCR & param); //assignment operator
+
   ClassDef(AliTPCParamCR,2)  //parameter  object for set:TPC
 };
 
index 5210557..a112df2 100644 (file)
@@ -65,31 +65,6 @@ AliTPCParamSR::AliTPCParamSR()
   SetDefault();
   Update();
 }
-AliTPCParamSR::AliTPCParamSR(const AliTPCParamSR &param)
-              :AliTPCParam(),
-              fInnerPRF(0),
-              fOuter1PRF(0),
-              fOuter2PRF(0),
-              fTimeRF(0),
-              fFacSigmaPadRow(0),
-              fFacSigmaPad(0),
-              fFacSigmaTime(0)
-{
-  //
-  //  copy constructor - dummy
-  //
-  fFacSigmaPadRow = param.fFacSigmaPadRow;
-}
-AliTPCParamSR & AliTPCParamSR::operator =(const AliTPCParamSR & param)
-{
-  //
-  // assignment operator - dummy
-  //
-  if (this == &param) return (*this);
-
-  fZLength=param.fZLength;
-  return (*this);
-}
 
 AliTPCParamSR::~AliTPCParamSR()
 {
@@ -569,7 +544,9 @@ Int_t  AliTPCParamSR::CalcResponseFast(Float_t* xyz, Int_t * index, Int_t row, F
     if ( (apadrow<0) || (apadrow>=2*kpadrn)) 
       continue;
     // pad angular correction
-    Float_t angle = kTanMax*2.*(cpad+0.5)/Float_t(npads);
+    Float_t angle = 0.;
+    if (npads != 0)
+      angle = kTanMax*2.*(cpad+0.5)/Float_t(npads);
     Float_t dpadangle =0;
     if (index[1]<fNInnerSector){
       dpadangle = angle*dpadrow*fInnerPadPitchLength/fInnerPadPitchWidth;
index 114b008..534b5bd 100644 (file)
@@ -16,8 +16,6 @@ class AliTPCPRF2D;
 class AliTPCParamSR : public AliTPCParam {
 public:
   AliTPCParamSR();
-  AliTPCParamSR(const AliTPCParamSR &param); // copy constructor
-  AliTPCParamSR &operator = (const AliTPCParamSR & param); //assignment operator
   virtual ~AliTPCParamSR();
 
   Int_t  CalcResponse(Float_t* x, Int_t * index, Int_t row);
@@ -75,6 +73,11 @@ protected:
   Float_t      fFacSigmaPadRow;    //factor-how many sigma of response I accept
   Float_t      fFacSigmaPad;       //factor-how many sigma of response I accept
   Float_t      fFacSigmaTime;      //factor-how many sigma of response I accept
+
+private:
+  AliTPCParamSR(const AliTPCParamSR &param); // copy constructor
+  AliTPCParamSR &operator = (const AliTPCParamSR & param); //assignment operator
+
   ClassDef(AliTPCParamSR,2)  //parameter  object for set:TPC
 };
 
index ad5f589..4a40a84 100644 (file)
@@ -1220,7 +1220,6 @@ Long64_t AliTPCcalibTracks::Merge(TCollection *collectionList) {
       histList = new TList;
       while (( objarray =  (TObjArray*)objListIterator->Next() )) { 
          // loop over arrayQDZList, get TObjArray, get object at position i, cast it into TH3F
-         if (!objarray) continue;
          hist = (TH3F*)(objarray->At(i));
          histList->Add(hist);
       }
index bdf28a5..cf1cece 100644 (file)
@@ -165,61 +165,7 @@ AliTPCclustererMI::AliTPCclustererMI(const AliTPCParam* par, const AliTPCRecoPar
     fAllNSigBins[iRow]=0;
   }
 }
-//______________________________________________________________
-AliTPCclustererMI::AliTPCclustererMI(const AliTPCclustererMI &param)
-              :TObject(param),
-  fBins(0),
-  fSigBins(0),
-  fNSigBins(0),
-  fLoop(0),
-  fMaxBin(0),
-  fMaxTime(0),
-  fMaxPad(0),
-  fSector(-1),
-  fRow(-1),
-  fSign(0),
-  fRx(0),
-  fPadWidth(0),
-  fPadLength(0),
-  fZWidth(0),
-  fPedSubtraction(kFALSE),
-  fEventHeader(0),
-  fTimeStamp(0),
-  fEventType(0),
-  fInput(0),
-  fOutput(0),
-  fOutputArray(0),
-  fOutputClonesArray(0),
-  fRowCl(0),
-  fRowDig(0),
-  fParam(0),
-  fNcluster(0),
-  fNclusters(0),
-  fDebugStreamer(0),
-  fRecoParam(0),
-  fBDumpSignal(kFALSE),
-  fBClonesArray(kFALSE),
-  fUseHLTClusters(4),
-  fAllBins(NULL),
-  fAllSigBins(NULL),
-  fAllNSigBins(NULL)
-{
-  //
-  // dummy
-  //
-  fMaxBin = param.fMaxBin;
-}
-//______________________________________________________________
-AliTPCclustererMI & AliTPCclustererMI::operator =(const AliTPCclustererMI & param)
-{
-  //
-  // assignment operator - dummy
-  //
-  if (this == &param) return (*this);
 
-  fMaxBin=param.fMaxBin;
-  return (*this);
-}
 //______________________________________________________________
 AliTPCclustererMI::~AliTPCclustererMI(){
   //
index 89295bf..3dd9c02 100644 (file)
@@ -34,8 +34,6 @@ class AliTPCCalROC;
 class AliTPCclustererMI : public TObject{
 public:
   AliTPCclustererMI(const AliTPCParam* par, const AliTPCRecoParam * recoParam = 0);
-  AliTPCclustererMI(const AliTPCclustererMI &param); // copy constructor
-  AliTPCclustererMI &operator = (const AliTPCclustererMI & param); //assignment
   virtual ~AliTPCclustererMI();
   virtual void Digits2Clusters();
   virtual void Digits2ClustersOld(AliRawReader* rawReader);
@@ -51,6 +49,9 @@ public:
   void SetUseHLTClusters(Int_t useHLTClusters) {fUseHLTClusters = useHLTClusters;} // set usage from HLT clusters from rec.C options
 
 private:
+  AliTPCclustererMI(const AliTPCclustererMI &param); // copy constructor
+  AliTPCclustererMI &operator = (const AliTPCclustererMI & param); //assignment
+
   Bool_t IsMaximum(Float_t k, Int_t max, const Float_t *bins) const; 
   void MakeCluster2(Int_t k,Int_t max,Float_t *bins,UInt_t m,
    AliTPCclusterMI &c);