Merge branch 'master' of https://git.cern.ch/reps/AliRoot
[u/mrichter/AliRoot.git] / MUON / AliMUONTOTCAStoreIterator.cxx
index 1c87de4..ef1f3f0 100644 (file)
 ///
 /// An iterator to access TObject stored in a TObjArray of TClonesArray
 ///
-///
 /// \author Laurent Aphecetche, Subatech
 //-----------------------------------------------------------------------------
 
 #include "AliMUONTOTCAStoreIterator.h"
 
+#include "AliLog.h"
 #include <TClonesArray.h>
 #include <TObjArray.h>
 
@@ -39,7 +39,7 @@ AliMUONTOTCAStoreIterator::AliMUONTOTCAStoreIterator(const TObjArray* data,
                                                      Int_t lastChamberId)
 : 
 TIterator(),
-fData(data),
+fkData(data),
 fFirstChamberId(firstChamberId),
 fLastChamberId(lastChamberId),
 fCurrentTCA(0x0),
@@ -51,16 +51,22 @@ fCurrentChamberId(-1)
 }
 
 //_____________________________________________________________________________
-TIterator& 
+AliMUONTOTCAStoreIterator& 
 AliMUONTOTCAStoreIterator::operator=(const TIterator& rhs)
 {
   /// Overriden operator= (imposed by Root's declaration of TIterator ?)
-  if ( this != &rhs && rhs.IsA() == AliMUONTOTCAStoreIterator::Class() )
+  if ( this != &rhs )
   {
-    const AliMUONTOTCAStoreIterator& rhs1 = 
-    static_cast<const AliMUONTOTCAStoreIterator&>(rhs);
-    
-    rhs1.CopyTo(*this);
+    if ( rhs.IsA() != AliMUONTOTCAStoreIterator::Class() )
+    {
+      AliErrorGeneral("AliMUONTOTCAStoreIterator::operator=","Wrong type");
+    }
+    else
+    {
+      const AliMUONTOTCAStoreIterator& rhs1 = 
+      static_cast<const AliMUONTOTCAStoreIterator&>(rhs);
+      rhs1.CopyTo(*this);
+    }
   }
   return *this;
 }
@@ -69,7 +75,7 @@ AliMUONTOTCAStoreIterator::operator=(const TIterator& rhs)
 AliMUONTOTCAStoreIterator::AliMUONTOTCAStoreIterator(const AliMUONTOTCAStoreIterator& rhs)
 : 
 TIterator(rhs),
-fData(0x0),
+fkData(0x0),
 fFirstChamberId(-1),
 fLastChamberId(-1),
 fCurrentTCA(0x0),
@@ -102,7 +108,7 @@ void
 AliMUONTOTCAStoreIterator::CopyTo(AliMUONTOTCAStoreIterator& destination) const
 {
   /// Copy *this to destination
-  destination.fData=fData;
+  destination.fkData=fkData;
   destination.fFirstChamberId=fFirstChamberId;
   destination.fLastChamberId=fLastChamberId;
   destination.fCurrentTCAIndex=fCurrentTCAIndex;
@@ -115,7 +121,7 @@ const TCollection*
 AliMUONTOTCAStoreIterator::GetCollection() const
 {
   /// The top level collection we're iterating upon, i.e. a TObjArray
-  return fData;
+  return fkData;
 }
 
 //_____________________________________________________________________________
@@ -137,7 +143,7 @@ AliMUONTOTCAStoreIterator::Next()
             fCurrentChamberId < fLastChamberId ) 
     {
       ++fCurrentChamberId;
-      fCurrentTCA = static_cast<TClonesArray*>(fData->At(fCurrentChamberId));
+      fCurrentTCA = static_cast<TClonesArray*>(fkData->At(fCurrentChamberId));
     }
   }