Coverity fixes
authorshahoian <shahoian@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 16 Dec 2011 09:54:27 +0000 (09:54 +0000)
committershahoian <shahoian@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 16 Dec 2011 09:54:27 +0000 (09:54 +0000)
STEER/ESD/AliESDtrack.cxx
STEER/ESD/AliESDtrack.h
STEER/ESD/AliTrackPointArray.cxx
STEER/ESD/AliVertexerTracks.cxx
STEER/STEER/AliMatrixSq.cxx
STEER/STEER/AliMatrixSq.h
STEER/STEERBase/AliMagF.cxx

index 1856677..8fe37bd 100644 (file)
@@ -2721,3 +2721,15 @@ void AliESDtrack::ReleaseESDfriendTrackGently()
 void AliESDtrack::PrintTmp()
 {
 }
+
+void AliESDtrack::SetFriendTrack(const AliESDfriendTrack *t)
+{
+  //attach fr. track
+  if (fFriendTrack) {
+    AliClonesPool* poolFr = fgPools ? fgPools->GetPoolTrFriend() : 0;
+    if (poolFr) poolFr->MarkSlotFree(fFriendTrack);
+    else        delete fFriendTrack;
+    fFriendTrack = 0;
+  }
+  if (t) fFriendTrack=new AliESDfriendTrack(*t);
+}
index 9f2aac3..a82fc0d 100644 (file)
@@ -55,9 +55,7 @@ public:
   virtual ~AliESDtrack();
   virtual void Copy(TObject &obj) const;
   const AliESDfriendTrack *GetFriendTrack() const {return fFriendTrack;}
-  void SetFriendTrack(const AliESDfriendTrack *t) {
-    delete fFriendTrack; fFriendTrack=new AliESDfriendTrack(*t);
-  }
+  void SetFriendTrack(const AliESDfriendTrack *t);
   void ReleaseESDfriendTrackGently();
   //  void ReleaseESDfriendTrack() {delete fFriendTrack; fFriendTrack=0;}
   void AddCalibObject(TObject * object);     // add calib object to the list
index 84fca6f..796e238 100644 (file)
@@ -290,7 +290,7 @@ void AliTrackPointArray::Sort(Bool_t down)
     GetPoint(p1,i);       // and remember where point i was moved
     AddPoint(i,&p);
     AddPoint(index[i],&p1);
-    for (int j=i;j<=fNPoints;j++) {if (index[j]==i) index[j]=index[i]; break;}
+    for (int j=i;j<fNPoints;j++) if (index[j]==i) {index[j]=index[i]; break;}
   }
   fSorted=kTRUE;
 }
index 54ec215..38898fb 100644 (file)
@@ -881,7 +881,7 @@ AliESDVertex* AliVertexerTracks::RemoveTracksFromVertex(AliESDVertex *inVtx,
   for(Int_t k=0; k<inVtx->GetNIndices(); k++) {
     Bool_t copyindex=kTRUE;
     for(Int_t l=0; l<ntrks; l++) {
-      if(inindices[k]==id[l]) copyindex=kFALSE;
+      if(inindices[k]==id[l]) {copyindex=kFALSE; break;}
     }
     if(copyindex) {
       outindices[j] = inindices[k]; j++;
index c34ba27..cd90702 100644 (file)
@@ -17,7 +17,13 @@ using namespace std;
 
 ClassImp(AliMatrixSq)
 
-
+AliMatrixSq & AliMatrixSq::operator=(const AliMatrixSq &src)
+{
+  // = operator
+  if (this == &src) return *this;
+  TMatrixDBase::operator=(src);
+  fSymmetric = src.fSymmetric; 
+}
 
 //___________________________________________________________
 void AliMatrixSq::MultiplyByVec(const Double_t *vecIn,Double_t *vecOut) const
index f621d46..b4b5ae8 100644 (file)
@@ -15,6 +15,7 @@ class AliMatrixSq : public TMatrixDBase {
   AliMatrixSq(): fSymmetric(kFALSE) {}
   AliMatrixSq(const AliMatrixSq &src) : TMatrixDBase(src), fSymmetric(src.fSymmetric) {}
   virtual ~AliMatrixSq() {}
+  AliMatrixSq& operator=(const AliMatrixSq &src);
   virtual Int_t   GetSize()                            const {return fNcols;}
   virtual Float_t GetDensity()                         const     = 0;
   //
index 6c818c4..adc092c 100644 (file)
@@ -261,7 +261,7 @@ void AliMagF::InitMachineField(BeamType_t btype, Double_t benergy)
   //
   double rigScale = benergy/7000.;   // scale according to ratio of E/Enominal
   // for ions assume PbPb (with energy provided per nucleon) and account for A/Z
-  if (btype == kBeamTypeAA || kBeamTypepA || kBeamTypeAp) rigScale *= 208./82.;
+  if (btype==kBeamTypeAA || btype==kBeamTypepA || btype==kBeamTypeAp) rigScale *= 208./82.;
   //
   fQuadGradient = 22.0002*rigScale;
   fDipoleField  = 37.8781*rigScale;