Coverity
authorfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 2 Feb 2011 15:00:56 +0000 (15:00 +0000)
committerfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 2 Feb 2011 15:00:56 +0000 (15:00 +0000)
EVE/EveDet/AliEveListAnalyser.cxx

index cfda70d..4c109e4 100644 (file)
@@ -605,13 +605,15 @@ Bool_t AliEveListAnalyser::ApplyProcessMacros(const TList* selIterator, const TL
     } 
   }  
 
+
+  Int_t selEntries = selIterator->GetEntries();
   // Collect the commands for each selection macro and add them to "data-from-list"
-  if (selIterator->GetEntries() > 0) {
-    selCmds         = new TString[selIterator->GetEntries()];
-    mSelType        = new AliEveListAnalyserMacroType[selIterator->GetEntries()];
-    mSelObjectType  = new TClass*[selIterator->GetEntries()];
-    mSelObjectType2 = new TClass*[selIterator->GetEntries()];
-    for (Int_t i = 0; i < selIterator->GetEntries(); i++){
+  if (selEntries > 0) {
+    selCmds         = new TString[selEntries];
+    mSelType        = new AliEveListAnalyserMacroType[selEntries];
+    mSelObjectType  = new TClass*[selEntries];
+    mSelObjectType2 = new TClass*[selEntries];
+    for (Int_t i = 0; i < selEntries; i++){
       macro = (TGeneralMacroData*)fMacroList->GetValue(selIterator->At(i)->GetTitle());
 
       if (!macro){
@@ -741,7 +743,7 @@ Bool_t AliEveListAnalyser::ApplyProcessMacros(const TList* selIterator, const TL
 
           // Select object by default (so it will be processed, if there are no correlated objects selection macros!)
           selectedByCorrSelMacro = kTRUE;
-          for (Int_t j = 0; j < selIterator->GetEntries(); j++){
+          for (Int_t j = 0; j < selEntries; j++){
             if (mSelType[j] == kCorrelObjectSelect){
           // Check, whether the macro can deal with both objects. If not, skip it.
           // Note: Again, via selCmds[i], the automatic objects are casted to the correct type!
@@ -834,7 +836,7 @@ Bool_t AliEveListAnalyser::ApplyProcessMacros(const TList* selIterator, const TL
 
           // Select object by default (so it will be processed, if there are no correlated objects selection macros!)
           selectedByCorrSelMacro = kTRUE;
-          for (Int_t j = 0; j < selIterator->GetEntries(); j++) {
+          for (Int_t j = 0; j < selEntries; j++) {
             if (mSelType[j] == kCorrelObjectSelect) {
               // Check, whether the macro can deal with both objects. If not, skip it.
               // Note: Again, via selCmds[i], the automatic objects are casted to the correct type!