Fixed coverity issues

diff --git a/PWGLF/FORWARD/analysis2/AliForwardCreateResponseMatrices.cxx b/PWGLF/FORWARD/analysis2/AliForwardCreateResponseMatrices.cxx
index e2c82fc85346def1f773e2da3c469bea91b8827d..18dd7275a73020d4dd65c0221cc63bb0584e3680 100644 (file)
--- a/PWGLF/FORWARD/analysis2/AliForwardCreateResponseMatrices.cxx
+++ b/PWGLF/FORWARD/analysis2/AliForwardCreateResponseMatrices.cxx
@@ -125,13 +125,16 @@ void AliForwardCreateResponseMatrices::UserExec(Option_t */*option*/)
   TH1D* dndetaSumForward    = (TH1D*)fOutput->FindObject("dndetaSumForward");
   TH1D* dndetaSumCentral    = (TH1D*)fOutput->FindObject("dndetaSumCentral");
   TH1D* dndetaSumMC         = (TH1D*)fOutput->FindObject("dndetaSumMC");
-  TH1D* dndetaEventForward  = (TH1D*)fOutput->FindObject("dndetaEventForward");
-  TH1D* dndetaEventCentral  = (TH1D*)fOutput->FindObject("dndetaEventCentral");
-  TH1D* dndetaEventMC       = (TH1D*)fOutput->FindObject("dndetaEventMC");
-
-  dndetaEventForward = forward.ProjectionX("dndetaForward",1,forward.GetNbinsY(),"");
-  dndetaEventCentral = central.ProjectionX("dndetaCentral",1,central.GetNbinsY(),"");
-  dndetaEventMC      = primHist->ProjectionX("dndetaMC",1,primHist->GetNbinsY(),"");
+  // TH1D* dndetaEventForward= (TH1D*)fOutput->FindObject("dndetaEventForward");
+  // TH1D* dndetaEventCentral= (TH1D*)fOutput->FindObject("dndetaEventCentral");
+  // TH1D* dndetaEventMC     = (TH1D*)fOutput->FindObject("dndetaEventMC");
+
+  TH1D* dndetaEventForward = forward.ProjectionX("dndetaForward",1,
+                                                forward.GetNbinsY(),"");
+  TH1D* dndetaEventCentral = central.ProjectionX("dndetaCentral",1,
+                                                central.GetNbinsY(),"");
+  TH1D* dndetaEventMC      = primHist->ProjectionX("dndetaMC",1,
+                                                  primHist->GetNbinsY(),"");
    
   // underflow eta bin of forward/central carry information on whether
   // there is acceptance. 1= acceptance, 0= no acceptance

diff --git a/PWGLF/FORWARD/analysis2/AliForwardMCFlowTaskQC.cxx b/PWGLF/FORWARD/analysis2/AliForwardMCFlowTaskQC.cxx
index 84a0cd44b1b993c241a13f89824296bd6db52fff..5f944101e3a0b0626f2c13e5a0c081416eb53105 100644 (file)
--- a/PWGLF/FORWARD/analysis2/AliForwardMCFlowTaskQC.cxx
+++ b/PWGLF/FORWARD/analysis2/AliForwardMCFlowTaskQC.cxx
@@ -300,14 +300,13 @@ Bool_t AliForwardMCFlowTaskQC::LoopAODMC()
     return kFALSE;
   }
 
-  Double_t rp = 0;
   AliAODMCHeader* header = 
     dynamic_cast<AliAODMCHeader*>(fAOD->FindListObject(
                                   AliAODMCHeader::StdBranchName()));
   if (!header) 
     AliWarning("No header file found.");
   
-  rp = header->GetReactionPlaneAngle();
+  // Double_t rp = header->GetReactionPlaneAngle();
 
   Int_t ntracks = mcArray->GetEntriesFast();
   // TODO: Make this bit smarter...
@@ -322,8 +321,7 @@ Bool_t AliForwardMCFlowTaskQC::LoopAODMC()
     if (fAddFlow.Contains("eta"))  flowFlags |= AliForwardFlowWeights::kEta;
     if (fAddFlow.Contains("cent")) flowFlags |= AliForwardFlowWeights::kCent;
   }
-  Double_t b = -1.;
-  b = header->GetImpactParameter();
+  // Double_t b = header->GetImpactParameter();
 
   // Track loop: chek how many particles will be accepted
   Double_t weight = 0;
@@ -366,12 +364,11 @@ Double_t AliForwardMCFlowTaskQC::GetCentFromB() const
   // Get centrality from MC impact parameter.
   //
   Double_t cent = -1.;
-  Double_t b = -1.;
   AliAODMCHeader* header = 
     static_cast<AliAODMCHeader*>(fAOD->FindListObject(AliAODMCHeader::
                                                      StdBranchName()));
   if (!header) return cent;
-  b = header->GetImpactParameter();
+  Double_t b = header->GetImpactParameter();
 
   cent = fImpactParToCent->Eval(b);
 
@@ -390,9 +387,12 @@ void AliForwardMCFlowTaskQC::PrintFlowSetup() const
   printf("Doing flow analysis for      :\t");
   for (Int_t n  = 0; n < fV.GetSize(); n++) printf("v%d ", fV.At(n));
   printf("\n");
-  Printf("Satellite vertex flag           :\t%s", ((fFlowFlags & kSatVtx) ? "true" : "false"));
-  Printf("Symmetrize ref. flow wrt eta = 0:\t%s", ((fFlowFlags & kSymEta) ? "true" : "false"));
-  Printf("Use an eta-gap for ref. flow    :\t%s", ((fFlowFlags & kEtaGap) ? "true" : "false"));
+  Printf("Satellite vertex flag           :\t%s", ((fFlowFlags & kSatVtx) ? 
+                                                  "true" : "false"));
+  Printf("Symmetrize ref. flow wrt eta = 0:\t%s", ((fFlowFlags & kSymEta) ? 
+                                                  "true" : "false"));
+  Printf("Use an eta-gap for ref. flow    :\t%s", ((fFlowFlags & kEtaGap) ? 
+                                                  "true" : "false"));
   Printf("FMD sigma cut:               :\t%f", fFMDCut);
   Printf("SPD sigma cut:               :\t%f", fSPDCut);
   if ((fFlowFlags & kEtaGap)) 

diff --git a/PWGLF/FORWARD/trains/ParUtilities.C b/PWGLF/FORWARD/trains/ParUtilities.C
index 2d6d57b9ab1509289825aae1ddaafece95acf53a..817596b88447589692a26cc7affe143e4b6fb25b 100644 (file)
--- a/PWGLF/FORWARD/trains/ParUtilities.C
+++ b/PWGLF/FORWARD/trains/ParUtilities.C
@@ -296,7 +296,7 @@ struct ParUtilities
       retVal = true;
     }
     catch (TString& e) {
-      Error("ParUtilities::MakeScriptPAR", e.Data());
+      Error("ParUtilities::MakeScriptPAR", "%s", e.Data());
       retVal = false;
     }
     

diff --git a/PWGLF/FORWARD/trains/trainMain.cxx b/PWGLF/FORWARD/trains/trainMain.cxx
index 2aa4ad893ff59b22db8a82c567d5a40da18776c3..e7480dbe96c4546a8394d15fe119c4eb8f2a4c87 100644 (file)
--- a/PWGLF/FORWARD/trains/trainMain.cxx
+++ b/PWGLF/FORWARD/trains/trainMain.cxx
@@ -152,7 +152,7 @@ main(int argc, char** argv)
   Bool_t  batch   = false;
   Bool_t  help    = false;
   Bool_t  urlSeen = false;
-  Bool_t  spawn   = true;
+  Bool_t  spawn   = false;
   // --- Parse options -----------------------------------------------
   for (int i = 1; i < argc; i++) { 
     if (argv[i][0] == '-' && argv[i][1] == '-') {