Changed the return type for operator=(const TIterator&)
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 28 May 2008 09:20:12 +0000 (09:20 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 28 May 2008 09:20:12 +0000 (09:20 +0000)
from TIterator& to concrete class reference to avoid
compiler warnings (suggested by F.Carminati)

14 files changed:
MUON/AliMUON2DMapIterator.cxx
MUON/AliMUON2DMapIterator.h
MUON/AliMUON2DMapIteratorByI.cxx
MUON/AliMUON2DMapIteratorByI.h
MUON/AliMUONClusterStoreV2Iterator.cxx
MUON/AliMUONClusterStoreV2Iterator.h
MUON/AliMUONDigitStoreV1Iterator.cxx
MUON/AliMUONDigitStoreV1Iterator.h
MUON/AliMUONDigitStoreVImplIterator.cxx
MUON/AliMUONDigitStoreVImplIterator.h
MUON/AliMUONTOTCAStoreIterator.cxx
MUON/AliMUONTOTCAStoreIterator.h
MUON/mapping/AliMpExMapIterator.cxx
MUON/mapping/AliMpExMapIterator.h

index 18ca049..a95de14 100644 (file)
@@ -46,16 +46,12 @@ fIter2(NextIterator())
 }
 
 //_____________________________________________________________________________
-TIterator& 
-AliMUON2DMapIterator::operator=(const TIterator& rhs)
+AliMUON2DMapIterator& 
+AliMUON2DMapIterator::operator=(const TIterator& /*rhs*/)
 {
-  /// overriden operator= (imposed by Root's definition of TIterator::operator= ?)
+  // overriden operator= (imposed by Root's definition of TIterator::operator= ?)
   
-  if ( this != &rhs && rhs.IsA() == AliMUON2DMapIterator::Class() ) 
-  {
-//    const AliMUON2DMapIterator& rhs1 = static_cast<const AliMUON2DMapIterator&>(rhs);
-    AliFatalGeneral("operator=(TIterator&)",""); // as in copy ctor
-  }
+  AliFatalGeneral("operator=(TIterator&)",""); // as in copy ctor
   return *this;
 }
 
index 695f77b..1fe82d4 100644 (file)
@@ -41,7 +41,8 @@ private:
   AliMUON2DMapIterator(const AliMUON2DMapIterator& rhs);
   /// Not implemented
   AliMUON2DMapIterator& operator=(const AliMUON2DMapIterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  /// Overriden TIterator virtual operator=
+  AliMUON2DMapIterator& operator=(const TIterator& rhs);
 
   const AliMpExMap* fkMap; ///< Top map we iterate upon
   TIterator* fIter1; ///< first iterator
index 37df45d..a9c9973 100644 (file)
@@ -51,16 +51,12 @@ fCurrentI(-1)
 }
 
 //_____________________________________________________________________________
-TIterator& 
-AliMUON2DMapIteratorByI::operator=(const TIterator& rhs)
+AliMUON2DMapIteratorByI& 
+AliMUON2DMapIteratorByI::operator=(const TIterator& /*rhs*/)
 {
-  /// overriden operator= (imposed by Root's definition of TIterator::operator= ?)
+  // overriden operator= (imposed by Root's definition of TIterator::operator= ?)
   
-  if ( this != &rhs && rhs.IsA() == AliMUON2DMapIteratorByI::Class() ) 
-  {
-    //    const AliMUON2DMapIteratorByI& rhs1 = static_cast<const AliMUON2DMapIteratorByI&>(rhs);
-    AliFatalGeneral("operator=(TIterator&)",""); // as in copy ctor
-  }
+  AliFatalGeneral("operator=(TIterator&)",""); // as in copy ctor
   return *this;
 }
 
index c906e1f..c9318c5 100644 (file)
@@ -43,7 +43,8 @@ private:
   AliMUON2DMapIteratorByI(const AliMUON2DMapIteratorByI& rhs);
   /// Not implemented
   AliMUON2DMapIteratorByI& operator=(const AliMUON2DMapIteratorByI& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  /// Overriden TIterator virtual operator=
+  AliMUON2DMapIteratorByI& operator=(const TIterator& rhs);
 
   const AliMpExMap* fkMap; ///< Top map we iterate upon
   AliMpExMapIterator* fIter1; ///< first iterator
index 379b11c..19907f4 100644 (file)
@@ -55,14 +55,11 @@ AliMUONClusterStoreV2Iterator::AliMUONClusterStoreV2Iterator(const AliMUONCluste
 }
 
 //_____________________________________________________________________________
-TIterator& AliMUONClusterStoreV2Iterator::operator=(const TIterator& /*iter*/)
+AliMUONClusterStoreV2Iterator& 
+AliMUONClusterStoreV2Iterator::operator=(const TIterator& /*iter*/)
 {
-  /// Overriden operator= (imposed by Root's definition of TIterator::operator= ?)
-/*
-  if ( this != &iter && iter.IsA() == AliMUONClusterStoreV2Iterator::Class() ) {
-    (*this) = static_cast<const AliMUONClusterStoreV2Iterator&>(iter);
-  }
-*/
+  // Overriden operator= (imposed by Root's definition of TIterator::operator= ?)
+
   AliFatalGeneral("AliMUONClusterStoreV2Iterator::operator=","reimplement me");
   return *this;
 }
index 1e6e8a5..41f601c 100644 (file)
@@ -41,7 +41,8 @@ private:
   AliMUONClusterStoreV2Iterator(const AliMUONClusterStoreV2Iterator& rhs);
   /// Not implemented
   AliMUONClusterStoreV2Iterator& operator=(const AliMUONClusterStoreV2Iterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  /// Overriden TIterator virtual operator=
+  AliMUONClusterStoreV2Iterator& operator=(const TIterator& rhs);
 
   const AliMUONClusterStoreV2* fStore; ///< store to iterate upon
   Int_t fFirstChamberId; ///< first chamber
index 47c99f4..b849ca6 100644 (file)
@@ -61,7 +61,7 @@ AliMUONDigitStoreV1Iterator::AliMUONDigitStoreV1Iterator(const AliMUONDigitStore
 }
 
 //_____________________________________________________________________________
-TIterator& 
+AliMUONDigitStoreV1Iterator& 
 AliMUONDigitStoreV1Iterator::operator=(const TIterator& rhs)
 {
   /// overriden assignment operator (imposed by Root's definition of TIterator ?)
@@ -70,10 +70,7 @@ AliMUONDigitStoreV1Iterator::operator=(const TIterator& rhs)
     const AliMUONDigitStoreV1Iterator& rhs1 = 
     static_cast<const AliMUONDigitStoreV1Iterator&>(rhs);
     
-    fArray = rhs1.fArray;
-    fFirstDetElemId = rhs1.fFirstDetElemId;
-    fLastDetElemId = rhs1.fLastDetElemId;
-    fCathode = rhs1.fCathode;
+    AliMUONDigitStoreV1Iterator::operator=(rhs1);
   }
   return *this;
 }
index 11edff9..2fa80b3 100644 (file)
@@ -21,7 +21,7 @@ class AliMUONDigitStoreV1Iterator : public AliMUONTOTCAStoreIterator
 {
 public:
   AliMUONDigitStoreV1Iterator(const AliMUONDigitStoreV1Iterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  AliMUONDigitStoreV1Iterator& operator=(const TIterator& rhs);
   AliMUONDigitStoreV1Iterator& operator=(const AliMUONDigitStoreV1Iterator& rhs);
   AliMUONDigitStoreV1Iterator(TObjArray* a,
                               Int_t firstDetElemId,
index 94cfecb..0b5885a 100644 (file)
@@ -69,34 +69,10 @@ fCurrentCalibParamIndex(-1)
 }
 
 //_____________________________________________________________________________
-AliMUONDigitStoreVImplIterator::AliMUONDigitStoreVImplIterator(const AliMUONDigitStoreVImplIterator& rhs)
-: TIterator(rhs),
-fStore(rhs.fStore),
-fFirstDetElemId(rhs.fFirstDetElemId),
-fLastDetElemId(rhs.fLastDetElemId),
-fCathode(rhs.fCathode),
-fStoreIterator(0x0),
-fCurrentCalibParam(0x0),
-fCurrentCalibParamIndex(-1)
-{
-  /// copy ctor
-  Fatal("AliMUONDigitStoreVImplIterator::copy tor","Not implementeable"); // because there's no clone in TIterator :-(
-}
-
-//_____________________________________________________________________________
 AliMUONDigitStoreVImplIterator&
-AliMUONDigitStoreVImplIterator::operator=(const AliMUONDigitStoreVImplIterator&)
-{
-  /// assignment operator
-  Fatal("AliMUONDigitStoreVImplIterator::operator=","Not implementeable"); // because there's no clone in TIterator :-(
-  return *this;
-}
-
-//_____________________________________________________________________________
-TIterator&
 AliMUONDigitStoreVImplIterator::operator=(const TIterator&)
 {
-  /// overriden assignment operator (imposed by Root's declaration of Titerator ?)
+  // overriden assignment operator (imposed by Root's declaration of Titerator ?)
   Fatal("TIterator::operator=","Not implementeable"); // because there's no clone in TIterator :-(
   return *this;
 }
index 9671f82..29d1395 100644 (file)
@@ -27,9 +27,6 @@ public:
                               Int_t firstDetElemId,
                               Int_t lastDetElemId,
                               Int_t cathode=2);
-  AliMUONDigitStoreVImplIterator(const AliMUONDigitStoreVImplIterator& rhs);
-  AliMUONDigitStoreVImplIterator& operator=(const AliMUONDigitStoreVImplIterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
   
   virtual ~AliMUONDigitStoreVImplIterator();
   
@@ -41,6 +38,13 @@ public:
   virtual const TCollection* GetCollection() const { return 0x0; }
   
 private:
+  /// Not implemented
+  AliMUONDigitStoreVImplIterator(const AliMUONDigitStoreVImplIterator& rhs);
+  /// Not implemented
+  AliMUONDigitStoreVImplIterator& operator=(const AliMUONDigitStoreVImplIterator& rhs);
+  /// Overriden TIterator virtual operator=
+  AliMUONDigitStoreVImplIterator& operator=(const TIterator& rhs);
+
   const AliMUONDigitStoreVImpl* fStore; //!< store to iterate upon
   Int_t fFirstDetElemId; //!< first de
   Int_t fLastDetElemId; //!< last de
index 0fd79ed..f07bc60 100644 (file)
@@ -50,7 +50,7 @@ fCurrentChamberId(-1)
 }
 
 //_____________________________________________________________________________
-TIterator& 
+AliMUONTOTCAStoreIterator& 
 AliMUONTOTCAStoreIterator::operator=(const TIterator& rhs)
 {
   /// Overriden operator= (imposed by Root's declaration of TIterator ?)
index 567a2ac..de174f5 100644 (file)
@@ -22,7 +22,7 @@ class AliMUONTOTCAStoreIterator : public TIterator
 public:
   AliMUONTOTCAStoreIterator(const TObjArray* a, Int_t firstChamberId, Int_t lastChamberId);
   AliMUONTOTCAStoreIterator(const AliMUONTOTCAStoreIterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  AliMUONTOTCAStoreIterator& operator=(const TIterator& rhs);
   AliMUONTOTCAStoreIterator& operator=(const AliMUONTOTCAStoreIterator& rhs);
   virtual ~AliMUONTOTCAStoreIterator();
     
index 748fabd..6ea13b3 100644 (file)
@@ -37,14 +37,16 @@ ClassImp(AliMpExMapIterator)
 
 //_____________________________________________________________________________
 AliMpExMapIterator::AliMpExMapIterator(const AliMpExMap& theMap)
-: fIterator(new TExMapIter(&(theMap.fMap)))
+: TIterator(),
+  fIterator(new TExMapIter(&(theMap.fMap)))
 {
 /// Standard constructor
 }
 
 //_____________________________________________________________________________
 AliMpExMapIterator::AliMpExMapIterator(const AliMpExMapIterator& rhs)
-: fIterator(rhs.fIterator)
+: TIterator(rhs),
+  fIterator(rhs.fIterator)
 {
 /// Copy constructor
 }
@@ -63,10 +65,10 @@ AliMpExMapIterator::operator=(const AliMpExMapIterator& rhs)
 }
 
 //_____________________________________________________________________________
-TIterator& 
+AliMpExMapIterator& 
 AliMpExMapIterator::operator=(const TIterator& rhs)
 {
-  /// overriden operator= (imposed by Root's definition of TIterator::operator= ?)
+  /// Overriden operator= (imposed by Root's definition of TIterator::operator= ?)
   
   if ( this != &rhs && rhs.IsA() == AliMpExMapIterator::Class() ) 
   {
index 50884df..5eb697d 100644 (file)
@@ -28,7 +28,7 @@ public:
   AliMpExMapIterator(const AliMpExMap& theMap);
   AliMpExMapIterator(const AliMpExMapIterator& rhs);
   AliMpExMapIterator& operator=(const AliMpExMapIterator& rhs);
-  TIterator& operator=(const TIterator& rhs);
+  AliMpExMapIterator& operator=(const TIterator& rhs);
   
   virtual ~AliMpExMapIterator();