coverity fixes
authorjbook <jbook>
Tue, 29 Apr 2014 10:03:33 +0000 (12:03 +0200)
committerjbook <jbook@cern.ch>
Tue, 29 Apr 2014 10:03:33 +0000 (12:03 +0200)
PWGDQ/dielectron/AliDielectron.cxx
PWGDQ/dielectron/AliDielectronEventCuts.cxx
PWGDQ/dielectron/AliDielectronMixingHandler.cxx
PWGDQ/dielectron/AliDielectronVarManager.h

index b0b1afb..e0a24f5 100644 (file)
@@ -1454,11 +1454,9 @@ void AliDielectron::SetCentroidCorrFunction(TF1 *fun, UInt_t varx, UInt_t vary,
   // clone temporare histogram, otherwise it will not be streamed to file!
   TString key = Form("cntrd%d%d%d",varx,vary,varz);
   fPostPIDCntrdCorr = (TH1*)fun->GetHistogram()->Clone(key.Data());
-  printf("itregal   %f \n", fPostPIDCntrdCorr->Integral());
-  // check if we use a TF* for correction or the histogram
-       fPostPIDCntrdCorr->GetListOfFunctions()->AddAt(fun,0);
-  // check for corrections and add their variables to the fill map
   if(fPostPIDCntrdCorr)  {
+    fPostPIDCntrdCorr->GetListOfFunctions()->AddAt(fun,0);
+    // check for corrections and add their variables to the fill map
     printf("POST TPC PID CORRECTION added for centroids:  ");
     switch(fPostPIDCntrdCorr->GetDimension()) {
     case 3: printf(" %s, ",fPostPIDCntrdCorr->GetZaxis()->GetName());
@@ -1481,7 +1479,6 @@ void AliDielectron::SetCentroidCorrFunction(TH1 *fun, UInt_t varx, UInt_t vary,
   // clone temporare histogram, otherwise it will not be streamed to file!
   TString key = Form("cntrd%d%d%d",varx,vary,varz);
   fPostPIDCntrdCorr = (TH1*)fun->Clone(key.Data());
-
   // check for corrections and add their variables to the fill map
   if(fPostPIDCntrdCorr)  {
     printf("POST TPC PID CORRECTION added for centroids:  ");
@@ -1506,9 +1503,9 @@ void AliDielectron::SetWidthCorrFunction(TF1 *fun, UInt_t varx, UInt_t vary, UIn
   // clone temporare histogram, otherwise it will not be streamed to file!
   TString key = Form("wdth%d%d%d",varx,vary,varz);
   fPostPIDWdthCorr = (TH1*)fun->GetHistogram()->Clone(key.Data());
-  fPostPIDWdthCorr->GetListOfFunctions()->AddAt(fun,0);
-  // check for corrections and add their variables to the fill map
   if(fPostPIDWdthCorr)  {
+    fPostPIDWdthCorr->GetListOfFunctions()->AddAt(fun,0);
+    // check for corrections and add their variables to the fill map
     printf("POST TPC PID CORRECTION added for widths:  ");
     switch(fPostPIDWdthCorr->GetDimension()) {
     case 3: printf(" %s, ",fPostPIDWdthCorr->GetZaxis()->GetName());
index 194eaa3..aadafd0 100644 (file)
@@ -249,22 +249,22 @@ Bool_t AliDielectronEventCuts::IsSelectedAOD(TObject* event)
   if(fUsedVars->CountBits()) {
     AliDielectronVarManager::SetFillMap(fUsedVars);
     AliDielectronVarManager::Fill(ev,values);
-  }
 
-  // correlation cuts
-  for(Int_t i=0; i<5; i++) {
-    if(fCorrCutMin[i]) {
-      Double_t varx = values[fCorrCutMin[i]->GetXaxis()->GetUniqueID()];
-      Double_t vary = values[fCorrCutMin[i]->GetYaxis()->GetUniqueID()];
-      Double_t min  = ((TF1*)fCorrCutMin[i]->GetListOfFunctions()->At(0))->Eval(varx);
-      //      printf("coor cut %d: varx %f -> eval %f > %f \n",i,varx,min,vary);
-      if(vary<min) return kFALSE;
-    }
-    if(fCorrCutMax[i]) {
-      Double_t varx = values[fCorrCutMax[i]->GetXaxis()->GetUniqueID()];
-      Double_t vary = values[fCorrCutMax[i]->GetYaxis()->GetUniqueID()];
-      Double_t max  = ((TF1*)fCorrCutMax[i]->GetListOfFunctions()->At(0))->Eval(varx);
-      if(vary>max) return kFALSE;
+    // correlation cuts
+    for(Int_t i=0; i<5; i++) {
+      if(fCorrCutMin[i]) {
+       Double_t varx = values[fCorrCutMin[i]->GetXaxis()->GetUniqueID()];
+       Double_t vary = values[fCorrCutMin[i]->GetYaxis()->GetUniqueID()];
+       Double_t min  = ((TF1*)fCorrCutMin[i]->GetListOfFunctions()->At(0))->Eval(varx);
+       //      printf("coor cut %d: varx %f -> eval %f > %f \n",i,varx,min,vary);
+       if(vary<min) return kFALSE;
+      }
+      if(fCorrCutMax[i]) {
+       Double_t varx = values[fCorrCutMax[i]->GetXaxis()->GetUniqueID()];
+       Double_t vary = values[fCorrCutMax[i]->GetYaxis()->GetUniqueID()];
+       Double_t max  = ((TF1*)fCorrCutMax[i]->GetListOfFunctions()->At(0))->Eval(varx);
+       if(vary>max) return kFALSE;
+      }
     }
   }
 
index f1d5853..3ddf07b 100644 (file)
@@ -359,12 +359,11 @@ void AliDielectronMixingHandler::Init(const AliDielectron *diele)
   //
   // initialise event buffers
   //
-
   Int_t size=GetNumberOfBins();
 
   AliDebug(10,Form("Creating a pool array with size %d \n",size));
 
-  if(diele->DoEventProcess()) fArrPools.Expand(size);
+  if(diele && diele->DoEventProcess()) fArrPools.Expand(size);
 
   //add statics histogram if we have a histogram manager
   if (diele && diele->fHistos && diele->DoEventProcess()) {
index aad0108..7f5f45a 100644 (file)
@@ -1906,7 +1906,7 @@ inline void AliDielectronVarManager::FillVarVEvent(const AliVEvent *event, Doubl
                                                                     values[AliDielectronVarManager::kv0C3rpH2]) ); 
 
   Double_t ZDCqvec[3][2];
-  memset(ZDCqvec, 999, sizeof(ZDCqvec));
+  memset(ZDCqvec, 255, sizeof(ZDCqvec));
   GetZDCRP(event, ZDCqvec);
 
   values[AliDielectronVarManager::kZDCArpH1] = TMath::ATan2(ZDCqvec[0][1], ZDCqvec[0][0]);