Cleanup EffC++ warnings
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 15 Sep 2006 14:51:29 +0000 (14:51 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 15 Sep 2006 14:51:29 +0000 (14:51 +0000)
MUON/AliMUONDataInterface.cxx
MUON/AliMUONDigitizer.cxx
MUON/AliMUONDigitizer.h
MUON/AliMUONRegionalTrigger.cxx

index 0ac2213..025fba0 100644 (file)
@@ -67,9 +67,25 @@ ClassImp(AliMUONDataInterface)
 /// \endcond
 
 AliMUONDataInterface::AliMUONDataInterface()
-       : TObject(), fData(NULL, "MUON", "MUON")
+       : TObject(), 
+         fCreatedRunLoader(kFALSE),
+         fHitAddressSet(kFALSE),
+         fSDigitAddressSet(kFALSE),
+         fDigitAddressSet(kFALSE),
+         fClusterAddressSet(kFALSE),
+         fTriggerAddressSet(kFALSE),
+         fRecTracksAddressSet(kFALSE),
+         fRunloader(0x0),
+         fMuonloader(0x0),
+         fData(0x0, "MUON", "MUON"),
+         fFilename(),
+         fFoldername(),
+         fEventnumber(-1),
+         fTrack(-1),
+         fSCathode(-1),
+         fCathode(-1)
 {
-/// Set all internal pointers to NULL and indices to -1.
+/// Set all internal pointers to 0x0 and indices to -1.
 
        Reset();
 }
index d8d2279..fd43d16 100644 (file)
@@ -49,48 +49,39 @@ ClassImp(AliMUONDigitizer)
 //___________________________________________
 AliMUONDigitizer::AliMUONDigitizer() : 
        AliDigitizer(),
+       fRunLoader(0),
+       fGime(0),
+       fMUON(0),
+       fMUONData(0),
+       fTrigDec(0),
        fHitMap(0),
        fTDList(0),
        fTDCounter(0),
        fMask(0),
-       fSignal(0)
+       fSignal(0),
+       fSegmentation(0)
 {
 /// Default constructor.
 /// Initializes all pointers to NULL.
-
-       fRunLoader = NULL;
-       fGime = NULL;
-       fMUON = NULL;
-       fMUONData = NULL;
-       fTrigDec = NULL;
 }
 
 //___________________________________________
 AliMUONDigitizer::AliMUONDigitizer(AliRunDigitizer* manager) : 
        AliDigitizer(manager),
+       fRunLoader(0),
+       fGime(0),
+       fMUON(0),
+       fMUONData(0),
+       fTrigDec(0),
        fHitMap(0),
        fTDList(0),
        fTDCounter(0),
        fMask(0),
-       fSignal(0)
+       fSignal(0),
+       fSegmentation(0)
 {
 /// Constructor which should be used rather than the default constructor.
 /// Initializes all pointers to NULL.
-
-       fRunLoader = NULL;
-       fGime = NULL;
-       fMUON = NULL;
-       fMUONData = NULL;
-       fTrigDec = NULL;
-}
-
-//___________________________________________
-AliMUONDigitizer::AliMUONDigitizer(const AliMUONDigitizer& rhs)
-  : AliDigitizer(rhs)
-{
-/// Protected copy constructor
-
-  AliFatal("Not implemented.");
 }
 
 //___________________________________________
@@ -105,19 +96,6 @@ AliMUONDigitizer::~AliMUONDigitizer()
     delete fTrigDec;
 }
 
-//-------------------------------------------------------------------
-AliMUONDigitizer&  
-AliMUONDigitizer::operator=(const AliMUONDigitizer& rhs)
-{
-/// Protected assignement operator
-
-  if (this == &rhs) return *this;
-
-  AliFatal("Not implemented.");
-    
-  return *this;  
-}    
-         
 //------------------------------------------------------------------------
 Bool_t AliMUONDigitizer::Init()
 {
index 725ebf0..228ab28 100644 (file)
@@ -48,9 +48,6 @@ public:
        virtual void Exec(Option_t* option = 0);
 
 protected:
-        AliMUONDigitizer(const AliMUONDigitizer& rhs);
-        AliMUONDigitizer& operator=(const AliMUONDigitizer& rhs);
-
        /* Digitizers inheriting from AliMUONDigitizer should implement this abstract method 
           so that TransientDigit objects are generated and put onto the fTDList.
           The method would be implemented as some loop over the input stream. The data can be
@@ -189,6 +186,8 @@ protected:
 
 
 private:
+        AliMUONDigitizer(const AliMUONDigitizer& rhs);
+        AliMUONDigitizer& operator=(const AliMUONDigitizer& rhs);
 
        ClassDef(AliMUONDigitizer, 1)   // MUON merging/digitization
 };    
index 0396047..f334538 100644 (file)
@@ -41,15 +41,15 @@ AliMUONRegionalTrigger::AliMUONRegionalTrigger()
 }
 //----------------------------------------------------------------------
 AliMUONRegionalTrigger::AliMUONRegionalTrigger(const AliMUONRegionalTrigger& theMUONRegionalTrig)
-  : TObject(theMUONRegionalTrig)
+  : TObject(theMUONRegionalTrig),
+    fId(theMUONRegionalTrig.fId),
+    fLocalMask(theMUONRegionalTrig.fLocalMask),  
+    fOutput(theMUONRegionalTrig.fOutput)           
 {
 /// Copy constructor (useful for TClonesArray)
 
-  fId             = theMUONRegionalTrig.fId;
-  fLocalMask      = theMUONRegionalTrig.fLocalMask;   
   fLocalOutput[0] = theMUONRegionalTrig.fLocalOutput[0];
   fLocalOutput[1] = theMUONRegionalTrig.fLocalOutput[1];
-  fOutput         = theMUONRegionalTrig.fOutput;           
  
 }
 //----------------------------------------------------------------------