Coding conventions, simplifications
authorfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 7 Oct 2009 21:42:30 +0000 (21:42 +0000)
committerfca <fca@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 7 Oct 2009 21:42:30 +0000 (21:42 +0000)
19 files changed:
ACORDE/AliACORDEReconstructor.h
EMCAL/AliEMCALReconstructor.cxx
EMCAL/AliEMCALReconstructor.h
FMD/AliFMDReconstructor.cxx
FMD/AliFMDReconstructor.h
HMPID/AliHMPIDReconstructor.h
ITS/AliITSReconstructor.cxx
ITS/AliITSReconstructor.h
PHOS/AliPHOSReconstructor.h
STEER/AliReconstructor.h
T0/AliT0Reconstructor.cxx
T0/AliT0Reconstructor.h
TOF/AliTOFReconstructor.cxx
TOF/AliTOFReconstructor.h
TPC/AliTPCReconstructor.h
TRD/AliTRDReconstructor.cxx
TRD/AliTRDReconstructor.h
VZERO/AliVZEROReconstructor.h
ZDC/AliZDCReconstructor.h

index 1549873..ff365bb 100644 (file)
@@ -50,8 +50,8 @@ protected:
   AliACORDERecoParam* fAcordeRecoParam; // Pointer to the ACORDE's RecoParam
 
 private:
-  AliACORDEReconstructor(const AliACORDEReconstructor& reconstructor);
-  AliACORDEReconstructor& operator = (const AliACORDEReconstructor& reconstructor);
+  AliACORDEReconstructor(const AliACORDEReconstructor&); //Not implemented
+  AliACORDEReconstructor& operator = (const AliACORDEReconstructor&); //Not implemented
   
   AliACORDECalibData* fCalibData;      //! calibration data
  
index 0152b2c..6b013b5 100644 (file)
@@ -102,17 +102,6 @@ AliEMCALReconstructor::AliEMCALReconstructor()
 } 
 
 //____________________________________________________________________________
-AliEMCALReconstructor::AliEMCALReconstructor(const AliEMCALReconstructor & rec)
-  : AliReconstructor(rec),
-    fDebug(rec.fDebug),
-    fList(rec.fList),
-    fGeom(rec.fGeom),
-       fCalibData(rec.fCalibData) 
-{
-  //copy ctor
-}
-
-//____________________________________________________________________________
 AliEMCALReconstructor::~AliEMCALReconstructor()
 {
   // dtor
index c8df22f..4fd2b64 100644 (file)
@@ -42,7 +42,6 @@ class AliEMCALReconstructor : public AliReconstructor {
 public:
 
   AliEMCALReconstructor() ; //ctor            
-  AliEMCALReconstructor(const AliEMCALReconstructor & rec);
    
   virtual ~AliEMCALReconstructor() ; //dtor
 
@@ -60,13 +59,6 @@ public:
   virtual Bool_t             HasDigitConversion() const {return kTRUE;};
   virtual void               ConvertDigits(AliRawReader* rawReader, TTree* digitsTree) const;
   
-  
-  AliEMCALReconstructor & operator = (const AliEMCALReconstructor & /*rvalue*/)  {
-    // assignement operator requested by coding convention but not needed
-    Fatal("operator =", "not implemented") ;
-    return *this ; 
-  }
-  
   static void   SetRecParam(AliEMCALRecParam * recParam){ fgkRecParam = recParam;}
 
   void   ReadDigitsArrayFromTree(TTree *digitsTree) const;
@@ -82,6 +74,9 @@ public:
 
 private:
   
+  AliEMCALReconstructor(const AliEMCALReconstructor &); //Not implemented
+  AliEMCALReconstructor & operator = (const AliEMCALReconstructor &); //Not implemented
+
   Bool_t fDebug; //! verbosity controller
 
   TList *fList;  //! List of hists (only for trigger now)
index a4eca07..f6493fd 100644 (file)
@@ -95,53 +95,6 @@ AliFMDReconstructor::AliFMDReconstructor()
     fZSFactor[det-1] = 0;
   }
 }
-  
-
-//____________________________________________________________________
-AliFMDReconstructor::AliFMDReconstructor(const AliFMDReconstructor& other) 
-  : AliReconstructor(), 
-    fMult(other.fMult),
-    fNMult(other.fNMult),
-    fTreeR(other.fTreeR),
-    fCurrentVertex(other.fCurrentVertex),
-    fESDObj(other.fESDObj),
-    fNoiseFactor(other.fNoiseFactor),
-    fAngleCorrect(other.fAngleCorrect),
-    fVertexType(other.fVertexType),
-    fESD(other.fESD),
-    fDiagnostics(other.fDiagnostics),
-    fDiagStep1(other.fDiagStep1), 
-    fDiagStep2(other.fDiagStep2),
-    fDiagStep3(other.fDiagStep3),
-    fDiagStep4(other.fDiagStep4),
-    fDiagAll(other.fDiagAll) 
-{
-  // Copy constructor 
-}
-  
-
-//____________________________________________________________________
-AliFMDReconstructor&
-AliFMDReconstructor::operator=(const AliFMDReconstructor& other) 
-{
-  // Assignment operator
-  fMult          = other.fMult;
-  fNMult         = other.fNMult;
-  fTreeR         = other.fTreeR;
-  fCurrentVertex = other.fCurrentVertex;
-  fESDObj        = other.fESDObj;
-  fNoiseFactor   = other.fNoiseFactor;
-  fAngleCorrect  = other.fAngleCorrect;
-  fVertexType    = other.fVertexType;
-  fESD           = other.fESD;
-  fDiagnostics   = other.fDiagnostics;
-  fDiagStep1     = other.fDiagStep1;
-  fDiagStep2     = other.fDiagStep2;
-  fDiagStep3     = other.fDiagStep3;
-  fDiagStep4     = other.fDiagStep4;
-  fDiagAll       = other.fDiagAll;
-  return *this;
-}
 
 //____________________________________________________________________
 AliFMDReconstructor::~AliFMDReconstructor() 
index 499d85f..3b064ba 100644 (file)
@@ -167,7 +167,7 @@ protected:
    *
    * @param other Object to copy from. 
    */
-  AliFMDReconstructor(const AliFMDReconstructor& other);
+  AliFMDReconstructor(const AliFMDReconstructor&); //Not implemented
   /** 
    * Assignment operator 
    *
@@ -175,7 +175,7 @@ protected:
    *
    * @return reference to this object 
    */
-  AliFMDReconstructor& operator=(const AliFMDReconstructor& other);
+  AliFMDReconstructor& operator=(const AliFMDReconstructor&); //Not implemented
   /** 
    * Try to get the vertex from either ESD or generator header.  Sets
    * @c fCurrentVertex to the found Z posistion of the vertex (if 
index 4a5b00c..29351e9 100644 (file)
@@ -45,8 +45,8 @@ public:
   TObjArray *fClu;                     // tmp list of clusters
 //
   private:
-  AliHMPIDReconstructor(const AliHMPIDReconstructor& r);              //dummy copy constructor
-  AliHMPIDReconstructor &operator=(const AliHMPIDReconstructor& r);   //dummy assignment operator
+  AliHMPIDReconstructor(const AliHMPIDReconstructor&);              //Not implemented
+  AliHMPIDReconstructor &operator=(const AliHMPIDReconstructor&);   //Not implemented
 //  
   ClassDef(AliHMPIDReconstructor, 2)        // class for the HMPID reconstruction
 };
index 102b276..1b363e0 100644 (file)
@@ -60,22 +60,6 @@ AliITSReconstructor::~AliITSReconstructor(){
   delete fItsPID;
   if(fDetTypeRec) delete fDetTypeRec;
 } 
-//______________________________________________________________________
-AliITSReconstructor::AliITSReconstructor(const AliITSReconstructor &ob) :AliReconstructor(ob),
-fItsPID(ob.fItsPID),
-fDetTypeRec(ob.fDetTypeRec)
-
-{
-  // Copy constructor
-}
-
-//______________________________________________________________________
-AliITSReconstructor& AliITSReconstructor::operator=(const AliITSReconstructor&  ob ){
-  // Assignment operator
-  this->~AliITSReconstructor();
-  new(this) AliITSReconstructor(ob);
-  return *this;
-}
 
 //______________________________________________________________________
 void AliITSReconstructor::Init() {
index 541c6b2..2025658 100644 (file)
@@ -23,8 +23,6 @@ class AliITSReconstructor: public AliReconstructor {
 public:
   AliITSReconstructor();
   virtual ~AliITSReconstructor();
-  AliITSReconstructor(const AliITSReconstructor &ob); // copy constructor
-  AliITSReconstructor& operator=(const AliITSReconstructor & ob); // ass. op.
   virtual void         Init();
   
   virtual void         Reconstruct(AliRawReader* rawReader, TTree* clustersTree) const;
@@ -43,6 +41,9 @@ public:
   static const AliITSRecoParam* GetRecoParam() { return dynamic_cast<const AliITSRecoParam*>(AliReconstructor::GetRecoParam(0)); }
 
 private:
+  AliITSReconstructor(const AliITSReconstructor &); //Not implemented
+  AliITSReconstructor& operator=(const AliITSReconstructor &); //Not implemented
+
   //data
   AliITSpidESD           *fItsPID;      // Pid for ITS
   AliITSDetTypeRec       *fDetTypeRec;  // reconstructor
index 8fea912..dcca58d 100644 (file)
@@ -69,17 +69,6 @@ class AliPHOSReconstructor : public AliReconstructor {
 public:
 
   AliPHOSReconstructor() ; //ctor            
-  AliPHOSReconstructor(const AliPHOSReconstructor & rec) :
-    AliReconstructor(rec),
-    fGeom(rec.fGeom),
-    fClusterizer(rec.fClusterizer),
-    fTSM(rec.fTSM),
-    fPID(rec.fPID)
-    {
-    // cpy ctor: 
-    // requested by the Coding Convention
-    Fatal("cpy ctor", "not implemented") ;
-  }
   virtual ~AliPHOSReconstructor() ; //dtor            
 
   static void                SetDebug()   { fgDebug = kTRUE ; }
@@ -98,17 +87,13 @@ public:
   virtual Float_t            CalibrateT(Float_t time, Int_t absId) const ;
 
   void FillMisalMatrixes(AliESDEvent* esd)const ;
-
-  AliPHOSReconstructor & operator = (const AliPHOSReconstructor & /*rvalue*/)  {
-    // assignement operator requested by coding convention but not needed
-    Fatal("operator =", "not implemented") ;
-    return *this ; 
-  }
   
   static const AliPHOSRecoParam* GetRecoParam() {
     return dynamic_cast<const AliPHOSRecoParam*>(AliReconstructor::GetRecoParam(4)); }
 
 private:
+  AliPHOSReconstructor(const AliPHOSReconstructor & rec); // Not implemented
+  AliPHOSReconstructor & operator = (const AliPHOSReconstructor &); // Not implemented
   
   static Bool_t fgDebug ; //! verbosity controller
   AliPHOSGeometry          *fGeom;           // pointer to the PHOS geometry
index 6f55b53..aa92b10 100644 (file)
@@ -64,6 +64,10 @@ public:
   static const AliDetectorRecoParam* GetRecoParam(Int_t iDet);
 
 private:
+
+  AliReconstructor(const AliReconstructor &); // Not implemented
+  AliReconstructor& operator=(const AliReconstructor &); // Not implemented
+  
   TString                            fOption;                                       //! option for reconstruction
   static const AliDetectorRecoParam* fgRecoParam[AliReconstruction::kNDetectors]; //! event reconstruction parameters for all detectors
   AliRunInfo*                        fRunInfo;                                    //! pointer to the run info object
index 600ad76..aadb1e7 100644 (file)
@@ -69,41 +69,8 @@ ClassImp(AliT0Reconstructor)
   fCalib = new AliT0Calibrator();
 
 }
-//____________________________________________________________________
-
-AliT0Reconstructor::AliT0Reconstructor(const AliT0Reconstructor &r):
-  AliReconstructor(r),
-                                            fdZonA(0),
-                                            fdZonC(0),
-                                            fZposition(0),
-                                            fParam(NULL),
-                                             fAmpLEDrec(0),
-                                            fQTC(0),
-                                            fAmpLED(0),
-                                            fCalib()
-
- {
-  //
-  // AliT0Reconstructor copy constructor
-  //
-
-  ((AliT0Reconstructor &) r).Copy(*this);
-
-}
-
-//_____________________________________________________________________________
-AliT0Reconstructor &AliT0Reconstructor::operator=(const AliT0Reconstructor &r)
-{
-  //
-  // Assignment operator
-  //
-  if (this != &r) ((AliT0Reconstructor &) r).Copy(*this);
-  return *this;
-
-}
 
 //_____________________________________________________________________________
-
 void AliT0Reconstructor::Reconstruct(TTree*digitsTree, TTree*clustersTree) const
   
 {
index df1172c..ebaccef 100644 (file)
@@ -19,9 +19,6 @@ class AliT0Reconstructor: public AliReconstructor {
  public:
   AliT0Reconstructor();
   virtual ~AliT0Reconstructor() {};
-  AliT0Reconstructor( const AliT0Reconstructor&r );
-  AliT0Reconstructor& operator=(const AliT0Reconstructor&r); 
-
 
   virtual  void   Reconstruct(TTree* fdigits, TTree * frecpoints) const;
   virtual  void   Reconstruct(AliRawReader*rawReader , TTree* recTree) const;
@@ -46,6 +43,10 @@ class AliT0Reconstructor: public AliReconstructor {
   TObjArray           fAmpLED;        // LED-CFD vs #MIPs
   AliT0Calibrator     *fCalib;           //pointer to T0 Calibrator     
 
+ private:
+  AliT0Reconstructor( const AliT0Reconstructor&r ); //Not implemented
+  AliT0Reconstructor& operator=(const AliT0Reconstructor&r); //Not implemented
+
   ClassDef(AliT0Reconstructor, 5)   // class for the T0 reconstruction
 
 };
index 3395725..a1fff1d 100644 (file)
@@ -61,28 +61,6 @@ AliTOFReconstructor::AliTOFReconstructor()
   if(!fTOFcalib->ReadParOfflineFromCDB("TOF/Calib",-1)) {AliFatal("Exiting, no CDB object found!!!");exit(0);}  
 }
 
-//------------------------------------------------------------------------
-AliTOFReconstructor::AliTOFReconstructor(const AliTOFReconstructor &source)
-  : AliReconstructor(source),
-    fTOFcalib(source.fTOFcalib)
-{
-//
-// copy ctor
-//
-}
-
-//------------------------------------------------------------------------
-AliTOFReconstructor & AliTOFReconstructor::operator=(const AliTOFReconstructor &source)
-{
-//
-// assignment op.
-//
-  if (this == &source)
-    return *this;
-
-  fTOFcalib=source.fTOFcalib;
-  return *this;
-}
 //_____________________________________________________________________________
 AliTOFReconstructor::~AliTOFReconstructor() 
 {
index adfa625..4a36137 100644 (file)
@@ -24,8 +24,6 @@ class AliTOFcalib;
 class AliTOFReconstructor: public AliReconstructor {
 public:
   AliTOFReconstructor();
-  AliTOFReconstructor(const AliTOFReconstructor &source); // copy constructor
-  AliTOFReconstructor& operator=(const AliTOFReconstructor &source); // ass. op.
   virtual ~AliTOFReconstructor();
 
   virtual void         Reconstruct(AliRawReader* rawReader,
@@ -43,6 +41,9 @@ public:
   static const AliTOFRecoParam* GetRecoParam() { return dynamic_cast<const AliTOFRecoParam*>(AliReconstructor::GetRecoParam(3)); } // getting RecoParam obj
 
 private:
+  AliTOFReconstructor(const AliTOFReconstructor &); //Not implemented
+  AliTOFReconstructor& operator=(const AliTOFReconstructor &); //Not implemented
+
   AliTOFcalib    *fTOFcalib;    // pointer to TOF calib class
 
   ClassDef(AliTOFReconstructor, 3)   // class for the TOF reconstruction
index b6ecc73..f5b5b6c 100644 (file)
@@ -36,8 +36,8 @@ public:
   static void  SetStreamLevel(Int_t level) { fgStreamLevel = level;}
 
 private:
-  AliTPCReconstructor(const AliTPCReconstructor& /*rec*/);
-  AliTPCReconstructor& operator=(const AliTPCReconstructor&);
+  AliTPCReconstructor(const AliTPCReconstructor&); //Not implemented
+  AliTPCReconstructor& operator=(const AliTPCReconstructor&); //Not implemented
   AliTPCParam*         GetTPCParam() const;
   static Int_t               fgStreamLevel; // flag for streaming      - for TPC reconstruction
   AliTPCclustererMI*         fClusterer;   // TPC clusterer
index 139b05b..69a066e 100644 (file)
@@ -88,20 +88,6 @@ AliTRDReconstructor::AliTRDReconstructor()
 }
 
 //_____________________________________________________________________________
-AliTRDReconstructor::AliTRDReconstructor(const AliTRDReconstructor &r)
-  :AliReconstructor(r)
-  ,fSteerParam(r.fSteerParam)
-{
-  //
-  // Copy constructor
-  //
-
-  memcpy(fDebugStream, r.fDebugStream, sizeof(TTreeSRedirector *) *AliTRDrecoParam::kTRDreconstructionTasks);
-  // ownership of debug streamers is not taken
-  CLRFLG(fSteerParam, kOwner);
-}
-
-//_____________________________________________________________________________
 AliTRDReconstructor::~AliTRDReconstructor()
 {
   //
index 75264ae..3eafd50 100644 (file)
@@ -35,9 +35,7 @@ public:
     ,kNsteer         = 7       // number of tasks
   };
   AliTRDReconstructor();
-  AliTRDReconstructor(const AliTRDReconstructor &r);
   virtual ~AliTRDReconstructor();
-  AliTRDReconstructor& operator = (const AliTRDReconstructor&)          { return *this;}
        
        virtual void        Init();
 
@@ -70,6 +68,9 @@ public:
   void               SetOption(Option_t *opt);
 
 private:
+  AliTRDReconstructor(const AliTRDReconstructor &r); //Not implemented
+  AliTRDReconstructor& operator = (const AliTRDReconstructor&); //Not implemented
+
   static Char_t    *fgSteerNames[kNsteer];//! steering names
   static Char_t    *fgSteerFlags[kNsteer];//! steering flags
   static Char_t    *fgTaskNames[AliTRDrecoParam::kTRDreconstructionTasks]; //! tasks names
index 4a186c6..814cb5b 100644 (file)
@@ -55,8 +55,8 @@ protected:
   AliESDVZEROfriend*  fESDVZEROfriend; // ESD friend object (complete raw data)
   
 private:
-  AliVZEROReconstructor(const AliVZEROReconstructor& reconstructor);
-  AliVZEROReconstructor& operator = (const AliVZEROReconstructor& reconstructor);
+  AliVZEROReconstructor(const AliVZEROReconstructor&); //Not implemented
+  AliVZEROReconstructor& operator = (const AliVZEROReconstructor&); //Not implemented
   
   AliVZEROCalibData* fCalibData;      //! calibration data
 
index 139b2a5..351b019 100644 (file)
@@ -67,8 +67,8 @@ public:
   void WritePbPbRecoParamInOCDB() const;
 
 private:
-  AliZDCReconstructor(const AliZDCReconstructor&);
-  AliZDCReconstructor& operator =(const AliZDCReconstructor&);
+  AliZDCReconstructor(const AliZDCReconstructor&); //Not implemented
+  AliZDCReconstructor& operator =(const AliZDCReconstructor&); //Not implemented
 
   void   ReconstructEventpp(TTree *clustersTree, 
            Float_t* ZN1ADCCorr, Float_t* ZP1ADCCorr, Float_t* ZN2ADCCorr, Float_t* ZP2ADCCorr,