From 8ebd7df4127272d8dc8e711539dd45778cf247f6 Mon Sep 17 00:00:00 2001
From: mtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Date: Thu, 15 May 2008 11:01:48 +0000
Subject: [PATCH] Handle missing rec-point tree in a uniform way: return null
 pointer to TEvePointSet.

---
 EVE/alice-macros/its_clusters.C | 10 ++++++----
 EVE/alice-macros/tpc_clusters.C |  2 +-
 EVE/alice-macros/trd_clusters.C | 15 ++++++++++-----
 3 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/EVE/alice-macros/its_clusters.C b/EVE/alice-macros/its_clusters.C
index a00b056dda1..0ddcb13ef88 100644
--- a/EVE/alice-macros/its_clusters.C
+++ b/EVE/alice-macros/its_clusters.C
@@ -32,14 +32,16 @@ TEvePointSet* its_clusters(TEveElement* cont=0, Float_t maxR=50)
   rl->LoadRecPoints("ITS");
 
   TTree *cTree = rl->GetTreeR("ITS", false);
-
-  TEvePointSet* clusters = new TEvePointSet(10000);
-  clusters->SetOwnIds(kTRUE);
+  if (cTree == 0)
+    return 0;
 
   TClonesArray *cl = NULL;
   TBranch *branch  = cTree->GetBranch("ITSRecPoints");
   branch->SetAddress(&cl);
 
+  TEvePointSet* clusters = new TEvePointSet(10000);
+  clusters->SetOwnIds(kTRUE);
+
   Int_t nentr=(Int_t)cTree->GetEntries();
   for (Int_t i=0; i<nentr; i++) {
     if (!cTree->GetEvent(i)) continue;
@@ -61,7 +63,7 @@ TEvePointSet* its_clusters(TEveElement* cont=0, Float_t maxR=50)
   }
 
   if (clusters->Size() == 0 && gEve->GetKeepEmptyCont() == kFALSE) {
-    Warning("its_clusters", "No ITS clusters");
+    Warning("its_clusters.C", "No ITS clusters");
     delete clusters;
     return 0;
   }
diff --git a/EVE/alice-macros/tpc_clusters.C b/EVE/alice-macros/tpc_clusters.C
index 8f90458f54f..bbb0571ea71 100644
--- a/EVE/alice-macros/tpc_clusters.C
+++ b/EVE/alice-macros/tpc_clusters.C
@@ -74,7 +74,7 @@ TEvePointSet* tpc_clusters(TEveElement* cont=0, Float_t maxR=270)
 
   if (clusters->Size() == 0 && gEve->GetKeepEmptyCont() == kFALSE)
   {
-    Warning("tpc_clusters", "No TPC clusters");
+    Warning("tpc_clusters.C", "No TPC clusters");
     delete clusters;
     return 0;
   }
diff --git a/EVE/alice-macros/trd_clusters.C b/EVE/alice-macros/trd_clusters.C
index 902c46a4a69..b5e0b03473c 100644
--- a/EVE/alice-macros/trd_clusters.C
+++ b/EVE/alice-macros/trd_clusters.C
@@ -23,18 +23,23 @@ class TEveElement;
 TEvePointSet* trd_clusters(TEveElement *cont = 0)
 {
   const Int_t kMaxClusters = 18 * 6 * 24 *10;
-  AliEveEventManager::AssertGeometry();
 
-  TEvePointSet *clusters = new TEvePointSet(kMaxClusters);
-  clusters->SetOwnIds(kTRUE);
+  AliEveEventManager::AssertGeometry();
 
   AliRunLoader *rl = AliEveEventManager::AssertRunLoader();
   rl->LoadRecPoints("TRD");
 
-  TObjArray *TRDcluster = 0x0;
   TTree *recPoints = rl->GetTreeR("TRD", kFALSE);
+  if (recPoints == 0)
+    return 0;
+
+  TObjArray *TRDcluster = 0x0;
   recPoints->SetBranchAddress("TRDcluster", &TRDcluster);
 
+  TEvePointSet *clusters = new TEvePointSet(kMaxClusters);
+  clusters->SetOwnIds(kTRUE);
+
+
   Int_t nentr=(Int_t)recPoints->GetEntries();
   for (Int_t i=0; i<nentr; i++) {
     if (!recPoints->GetEvent(i)) continue;
@@ -53,7 +58,7 @@ TEvePointSet* trd_clusters(TEveElement *cont = 0)
   }
 
   if(clusters->Size() == 0 && gEve->GetKeepEmptyCont() == kFALSE) {
-    Warning("trd_clusters", "No TRD clusters");
+    Warning("trd_clusters.C", "No TRD clusters");
     delete clusters;
     return 0;
   }
-- 
2.31.1