Fix some coverity compilation warnings (-Wunused-but-set-variable)
authorlaphecet <laurent.aphecetche@subatech.in2p3.fr>
Thu, 3 Jul 2014 16:23:01 +0000 (18:23 +0200)
committerlaphecet <laurent.aphecetche@subatech.in2p3.fr>
Thu, 3 Jul 2014 16:35:44 +0000 (18:35 +0200)
MUON/AliMUONCheck.cxx
MUON/AliMUONClusterFinderMLEM.cxx
MUON/AliMUONClusterFinderPeakCOG.cxx
MUON/AliMUONClusterFinderPeakFit.cxx
MUON/AliMUONClusterSplitterMLEM.cxx
MUON/AliMUONRawWriter.cxx

index af09620..fd8c8dd 100644 (file)
@@ -160,7 +160,6 @@ AliMUONCheck::CheckESD(Bool_t pdc06TriggerResponse)
   Float_t muonMass = 0.105658389;
   Double_t thetaX, thetaY, pYZ;
   Double_t fPxRec1, fPyRec1, fPzRec1, fE1;
-  Int_t fZ1;
   
   AliESDEvent* fESD = new AliESDEvent();
   TTree* tree = (TTree*) esdFile->Get("esdTree");
@@ -287,7 +286,6 @@ AliMUONCheck::CheckESD(Bool_t pdc06TriggerResponse)
       fPzRec1  = - pYZ / TMath::Sqrt(1.0 + TMath::Tan(thetaY)*TMath::Tan(thetaY));
       fPxRec1  = fPzRec1 * TMath::Tan(thetaX);
       fPyRec1  = fPzRec1 * TMath::Tan(thetaY);
-      fZ1 = Int_t(TMath::Sign(1.,muonTrack->GetInverseBendingMomentum()));
       fE1 = TMath::Sqrt(muonMass * muonMass + fPxRec1 * fPxRec1 + fPyRec1 * fPyRec1 + fPzRec1 * fPzRec1);
       fV1.SetPxPyPzE(fPxRec1, fPyRec1, fPzRec1, fE1);
       // -----------> transverse momentum
index 58019ef..c8d11f9 100644 (file)
@@ -1744,7 +1744,7 @@ void AliMUONClusterFinderMLEM::Simple(AliMUONCluster& cluster)
 {
 /// Process simple cluster (small number of pads) without EM-procedure
 
-  Int_t nForFit = 1, clustFit[1] = {0}, nfit;
+  Int_t nForFit = 1, clustFit[1] = {0};
   Double_t parOk[3] = {0.}; 
   TObjArray *clusters[1]; 
   clusters[0] = fPixArray;
@@ -1767,7 +1767,7 @@ void AliMUONClusterFinderMLEM::Simple(AliMUONCluster& cluster)
     */
     if (!pad->IsSaturated()) pad->SetStatus(fgkUseForFit);
   }
-  nfit = fSplitter->Fit(cluster,1, nForFit, clustFit, clusters, parOk, fClusterList, fHistMlem);
+  fSplitter->Fit(cluster,1, nForFit, clustFit, clusters, parOk, fClusterList, fHistMlem);
 }
 
 //_____________________________________________________________________________
index c0fbb5a..67b9980 100644 (file)
@@ -815,9 +815,6 @@ void AliMUONClusterFinderPeakCOG::FindCluster(AliMUONCluster& cluster,
     nonb[0] = 0;
     nonb[1] = 1;
   }
-  Bool_t samex = kFALSE, samey = kFALSE;
-  if (TMath::Abs(dim0.X()-dim1.X()) < fgkDistancePrecision) samex = kTRUE; // the same X pad size on both planes 
-  if (TMath::Abs(dim0.Y()-dim1.Y()) < fgkDistancePrecision) samey = kTRUE; // the same Y pad size on both planes 
 
   // Drop all pixels from the array - pick up only the ones from the cluster
   //fPixArray->Delete();
index 21b011d..fe2d4c5 100644 (file)
@@ -1023,9 +1023,6 @@ void AliMUONClusterFinderPeakFit::FindClusterCOG(AliMUONCluster& cluster,
     nonb[0] = 0;
     nonb[1] = 1;
   }
-  Bool_t samex = kFALSE, samey = kFALSE;
-  if (TMath::Abs(dim0.X()-dim1.X()) < fgkDistancePrecision) samex = kTRUE; // the same X pad size on both planes 
-  if (TMath::Abs(dim0.Y()-dim1.Y()) < fgkDistancePrecision) samey = kTRUE; // the same Y pad size on both planes 
 
   // Drop all pixels from the array - pick up only the ones from the cluster
   //fPixArray->Delete();
index ee38302..42ff82b 100644 (file)
@@ -271,7 +271,6 @@ AliMUONClusterSplitterMLEM::Fit(const AliMUONCluster& cluster,
   Double_t xmax = mlem->GetXaxis()->GetXmax() + mlem->GetXaxis()->GetBinWidth(1);
   Double_t ymin = mlem->GetYaxis()->GetXmin() - mlem->GetYaxis()->GetBinWidth(1);
   Double_t ymax = mlem->GetYaxis()->GetXmax() + mlem->GetYaxis()->GetBinWidth(1);
-  Double_t xPad = 0, yPad = 99999;
   
   // Number of pads to use and number of virtual pads
   Int_t npads = 0, nVirtual = 0, nfit0 = nfit;
@@ -286,22 +285,6 @@ AliMUONClusterSplitterMLEM::Fit(const AliMUONCluster& cluster,
     if ( pad->IsReal() )
     {
       ++npads;
-      if (yPad > 9999) 
-      { 
-        xPad = pad->X();
-        yPad = pad->Y();
-      } 
-      else 
-      {
-        if (pad->DY() < pad->DX() ) 
-        {
-          yPad = pad->Y();
-        }
-        else 
-        {
-          xPad = pad->X();
-        }
-      }
     }
   }
   
@@ -429,7 +412,7 @@ AliMUONClusterSplitterMLEM::Fit(const AliMUONCluster& cluster,
   Double_t param0[2][8]={{0},{0}}, deriv[2][8]={{0},{0}}; 
   Double_t shift[8]={0}, stepMax, derMax, parmin[8]={0}, parmax[8]={0}, func2[2]={0}, shift0;
   Double_t delta[8]={0}, scMax, dder[8], estim, shiftSave = 0;
-  Int_t min, max, nCall = 0, nLoop, idMax = 0, iestMax = 0, nFail;
+  Int_t min, max, nCall = 0, nLoop, idMax = 0, nFail;
   Double_t rad, dist[3] = {0};
     
   // Try to fit with one-track hypothesis, then 2-track. If chi2/dof is 
@@ -555,7 +538,6 @@ AliMUONClusterSplitterMLEM::Fit(const AliMUONCluster& cluster,
        if (es > estim) 
         { 
          estim = es;
-         iestMax = j;
        }
           
        // Too big step
@@ -1082,7 +1064,7 @@ AliMUONClusterSplitterMLEM::Merge(const AliMUONCluster& cluster,
 {
   /// Merge the group of clusters with the one having the strongest coupling with them
   
-  Int_t indx, indx1, npxclu, npxclu1, imax=0;
+  Int_t indx, indx1, npxclu, imax=0;
   TObjArray *pix, *pix1;
   Double_t couplMax;
   
@@ -1104,8 +1086,7 @@ AliMUONClusterSplitterMLEM::Merge(const AliMUONCluster& cluster,
     } // for (Int_t icl1=0;
       // Add to it
     pix1 = clusters[imax];
-    npxclu1 = pix1->GetEntriesFast();
-    // Add pixels 
+    // Add pixels
     for (Int_t i = 0; i < npxclu; ++i) 
     { 
       pix1->Add(pix->UncheckedAt(i)); 
index 7f378cd..8b744b3 100644 (file)
@@ -480,7 +480,6 @@ Int_t AliMUONRawWriter::WriteTriggerDDL(const AliMUONVTriggerStore& triggerStore
   UInt_t word;
   Int_t* buffer = 0;
   Int_t index;
-  Int_t locCard;
   UChar_t locDec, trigY, posY, posX, regOut;
   UInt_t regInpLpt;
   UInt_t regInpHpt;
@@ -646,7 +645,6 @@ Int_t AliMUONRawWriter::WriteTriggerDDL(const AliMUONVTriggerStore& triggerStore
 
       if (locTrg)
       {
-        locCard = locTrg->LoCircuit();
         locDec  = locTrg->GetLoDecision();
         trigY   = locTrg->LoTrigY();
         posY    = locTrg->LoStripY();