Fix for RETURN_LOCAL problem reported by Coverity

[u/mrichter/AliRoot.git] / PWG0 / CreateStandardCuts.C

diff --git a/PWG0/CreateStandardCuts.C b/PWG0/CreateStandardCuts.C
index c801069d015c9b99292f623bef47a21094c05104..2636ca35356380d3af409ae20d07d4cdd857521b 100644 (file)
--- a/PWG0/CreateStandardCuts.C
+++ b/PWG0/CreateStandardCuts.C
@@ -1,54 +1,47 @@
 /* $Id: CreateCuts.C,v 1.5 2008/01/11 08:28:52 jgrosseo Exp $ */
 
-// this macro creates the track and event cuts used in this analysis
+// this macro creates the track cuts used in this analysis
 
-AliESDtrackCuts* CreateTrackCuts(AliPWG0Helper::AnalysisMode analysisMode, Bool_t fieldOn = kTRUE, Bool_t hists = kTRUE)
+AliESDtrackCuts* CreateTrackCuts(AliPWG0Helper::AnalysisMode analysisMode, Bool_t hists = kTRUE)
 {
-  AliESDtrackCuts* esdTrackCuts = new AliESDtrackCuts("AliESDtrackCuts");
-
-  if (hists)
-    esdTrackCuts->DefineHistograms(1);
+  AliESDtrackCuts* esdTrackCuts = 0;
+  
+  // see https://twiki.cern.ch/twiki/bin/view/ALICE/SelectionOfPrimaryTracksForPp2009DataAnalysis
+  
+  if (analysisMode & AliPWG0Helper::kTPC)
+  {
+    esdTrackCuts = new AliESDtrackCuts("AliESDtrackCuts");
 
-  // default cuts for ITS+TPC
-  Double_t cov1 = 2;
-  Double_t cov2 = 2;
-  Double_t cov3 = 0.5;
-  Double_t cov4 = 0.5;
-  Double_t cov5 = 2;
-  Double_t nSigma = 3;
+    TString tag("TPC-only tracking");
 
-  TString tag("Global tracking");
+    esdTrackCuts->SetMaxDCAToVertexZ(3.2);
+    esdTrackCuts->SetMaxDCAToVertexXY(2.4);
+    esdTrackCuts->SetDCAToVertex2D(kTRUE);
+  
+    esdTrackCuts->SetRequireTPCRefit(kTRUE);
+    esdTrackCuts->SetAcceptKinkDaughters(kFALSE);
+    esdTrackCuts->SetMinNClustersTPC(70);
+    esdTrackCuts->SetMaxChi2PerClusterTPC(4);
+  }
 
-  // TPC-only cuts
-  if (analysisMode == AliPWG0Helper::kTPC) 
+  if (analysisMode & AliPWG0Helper::kTPCITS)
   {
-    // beta cuts (still under investigation)
-    cov1 = 4;
-    cov2 = 4;
-    nSigma = 4;
-    
-    tag = "TPC-only tracking";
+    esdTrackCuts = AliESDtrackCuts::GetStandardITSTPCTrackCuts2009(kTRUE);
+  
+    TString tag("Global tracking");
   }
 
+  if (hists)
+    esdTrackCuts->DefineHistograms(1);
+
   // cuts for data without field
-  if (!fieldOn)
+  if (!(analysisMode & AliPWG0Helper::kFieldOn))
   {
-    cov5 = 1e10;
     tag += " without field";
   }
-  
-  esdTrackCuts->SetMaxCovDiagonalElements(cov1, cov2, cov3, cov4, cov5);
-
-  esdTrackCuts->SetMinNsigmaToVertex(nSigma);
-  esdTrackCuts->SetRequireSigmaToVertex(kTRUE);
-
-  esdTrackCuts->SetRequireTPCRefit(kTRUE);
-  esdTrackCuts->SetAcceptKingDaughters(kFALSE);
-
-  esdTrackCuts->SetMinNClustersTPC(50);
-  esdTrackCuts->SetMaxChi2PerClusterTPC(3.5);
 
   Printf("Created track cuts for: %s", tag.Data());
 
   return esdTrackCuts;
 }
+