Effective c++ corrections (T.Pocheptsov)
authorkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 7 Sep 2006 18:31:08 +0000 (18:31 +0000)
committerkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 7 Sep 2006 18:31:08 +0000 (18:31 +0000)
22 files changed:
PHOS/AliPHOSCalibData.cxx
PHOS/AliPHOSCalibData.h
PHOS/AliPHOSCalibrManager.cxx
PHOS/AliPHOSCalibrManager.h
PHOS/AliPHOSCalibrationDB.cxx
PHOS/AliPHOSCalibrator.cxx
PHOS/AliPHOSCalibrator.h
PHOS/AliPHOSEvalRecPoint.cxx
PHOS/AliPHOSFastRecParticle.cxx
PHOS/AliPHOSJet.cxx
PHOS/AliPHOSJet.h
PHOS/AliPHOSJetFinder.cxx
PHOS/AliPHOSJetFinder.h
PHOS/AliPHOSLoader.cxx
PHOS/AliPHOSPIDv0.cxx
PHOS/AliPHOSPIDv0.h
PHOS/AliPHOSPIDv1.cxx
PHOS/AliPHOSRaw2Digits.cxx
PHOS/AliPHOSRawStream2004.cxx
PHOS/AliPHOSRawStream2004.h
PHOS/AliPHOSRecCpvManager.cxx
PHOS/AliPHOSRecEmcManager.cxx

index 8b7ba83..275fdbb 100644 (file)
@@ -35,13 +35,14 @@ ClassImp(AliPHOSCalibData)
 
 //________________________________________________________________
   AliPHOSCalibData::AliPHOSCalibData(): 
-    TNamed(), fCalibDataEmc(0x0), fCalibDataCpv(0x0)
+    TNamed(), 
+    fCalibDataEmc(0x0), 
+    fCalibDataCpv(0x0),
+    fEmcDataPath("PHOS/Calib/EmcGainPedestals"),
+    fCpvDataPath("PHOS/Calib/CpvGainPedestals")
 {
-  // Default constructor  
-
-  fEmcDataPath="PHOS/Calib/EmcGainPedestals";
-  fCpvDataPath="PHOS/Calib/CpvGainPedestals";
-
+  // Default constructor
+  
   AliCDBEntry* entryEmc = AliCDBManager::Instance()->Get(fEmcDataPath.Data());
   if(entryEmc)
     fCalibDataEmc = (AliPHOSEmcCalibData*)entryEmc->GetObject();
@@ -55,13 +56,11 @@ ClassImp(AliPHOSCalibData)
 //________________________________________________________________
 AliPHOSCalibData::AliPHOSCalibData(Int_t runNumber) :
   TNamed("phosCalib","PHOS Calibration Data Manager"),
-  fCalibDataEmc(0x0), fCalibDataCpv(0x0)
+  fCalibDataEmc(0x0), fCalibDataCpv(0x0),
+  fEmcDataPath("PHOS/Calib/EmcGainPedestals"),
+  fCpvDataPath("PHOS/Calib/CpvGainPedestals")
 {
   // Constructor
-  
-  fEmcDataPath="PHOS/Calib/EmcGainPedestals";
-  fCpvDataPath="PHOS/Calib/CpvGainPedestals";
-
   AliCDBEntry* entryEmc = AliCDBManager::Instance()->Get(fEmcDataPath.Data(),runNumber);
   if(entryEmc)
     fCalibDataEmc = (AliPHOSEmcCalibData*)entryEmc->GetObject();
@@ -74,15 +73,13 @@ AliPHOSCalibData::AliPHOSCalibData(Int_t runNumber) :
 
 //________________________________________________________________
 AliPHOSCalibData::AliPHOSCalibData(AliPHOSCalibData & phosCDB) :
-  TNamed(phosCDB)
+  TNamed(phosCDB),
+  fCalibDataEmc(phosCDB.fCalibDataEmc),
+  fCalibDataCpv(phosCDB.fCalibDataCpv),
+  fEmcDataPath(phosCDB.fEmcDataPath),
+  fCpvDataPath(phosCDB.fCpvDataPath)
 {
   // Copy constructor
-
-  fCalibDataEmc = phosCDB.fCalibDataEmc;
-  fCalibDataCpv = phosCDB.fCalibDataCpv;
-  
-  fEmcDataPath  = phosCDB.fEmcDataPath;
-  fCpvDataPath  = phosCDB.fCpvDataPath;
 }
 //________________________________________________________________
 AliPHOSCalibData::~AliPHOSCalibData()
@@ -91,6 +88,20 @@ AliPHOSCalibData::~AliPHOSCalibData()
  
 }
 
+AliPHOSCalibData & AliPHOSCalibData::operator = (const AliPHOSCalibData & rhs)
+{
+  //Copy-assignment. Does not delete anything (see destructor)
+  //compiler generated is ok, but ... because -Weffc++ and pointer
+  //members we have to define it explicitly.
+  TNamed::operator=(rhs);
+  fCalibDataEmc = rhs.fCalibDataEmc;
+  fCalibDataCpv = rhs.fCalibDataCpv;
+  fEmcDataPath  = rhs.fEmcDataPath;
+  fCpvDataPath  = rhs.fCpvDataPath;
+  
+  return *this;
+}
+
 //________________________________________________________________
 void AliPHOSCalibData::Reset()
 {
index bf7173e..a4118da 100644 (file)
@@ -24,6 +24,9 @@ class AliPHOSCalibData: public TNamed {
   AliPHOSCalibData(Int_t runNumber);
   AliPHOSCalibData(AliPHOSCalibData & phosCDB);
   virtual ~AliPHOSCalibData();
+
+  AliPHOSCalibData & operator = (const AliPHOSCalibData & rhs);
+
   void Reset();
   virtual void Print(Option_t *option = "") const; 
   
index a9be865..26eb334 100644 (file)
@@ -54,26 +54,41 @@ ClassImp(AliPHOSCalibrManager)
   
   AliPHOSCalibrManager * AliPHOSCalibrManager::fgCaMa = 0 ;
 //____________________________________________________________________________ 
-AliPHOSCalibrManager::AliPHOSCalibrManager():TNamed() 
+AliPHOSCalibrManager::AliPHOSCalibrManager() : 
+  fFileName(""),
+  fInputKind(0),
+  fctdb(0)
 {
   // default ctor: not to be used
-  fInputKind = 0 ;
-  fFileName= "" ;
-  fctdb = 0 ;
   AliFatal(Form("Should not be used")) ;
 }
 //____________________________________________________________________________ 
 AliPHOSCalibrManager::AliPHOSCalibrManager(const char* filename,const char * kind ):
-  TNamed("AliPHOSCalibrManager",filename){
+  TNamed("AliPHOSCalibrManager",filename),
+  fFileName(filename),
+  fInputKind(0),
+  fctdb(0)
+{
   if(strcmp(kind,"root")==0){
     fInputKind = 0 ;
   }
   else{
     fInputKind = 1 ;
   }
-  fFileName = filename;
-  fctdb = 0 ;
 } 
+
+//____________________________________________________________________________ 
+AliPHOSCalibrManager::AliPHOSCalibrManager(const AliPHOSCalibrManager & manager) : 
+  TNamed(manager), 
+  fFileName(""),
+  fInputKind(0),
+  fctdb(0)
+{
+  // cpy ctor: no need
+  // requested by the Coding Convention
+  Fatal("cpy ctor", "not implemented") ;
+}
+
 //____________________________________________________________________________ 
 AliPHOSCalibrManager::~AliPHOSCalibrManager()
 {
index 51d207c..ba50660 100644 (file)
@@ -24,11 +24,7 @@ class AliPHOSCalibrManager:public TNamed {
 
 public:
   AliPHOSCalibrManager() ;          // ctor
-  AliPHOSCalibrManager(const AliPHOSCalibrManager & manager) : TNamed(manager) {
-    // cpy ctor: no need
-    // requested by the Coding Convention
-    Fatal("cpy ctor", "not implemented") ;
-  }
+  AliPHOSCalibrManager(const AliPHOSCalibrManager & manager);
  
   virtual ~AliPHOSCalibrManager() ; // dtor
   static AliPHOSCalibrManager * GetInstance() ;
index 0557fcd..780db78 100644 (file)
@@ -33,7 +33,7 @@ ClassImp(AliPHOSCalibrationDB)
 
 
 //____________________________________________________________________________ 
-  AliPHOSCalibrationDB::AliPHOSCalibrationDB():TNamed() 
+  AliPHOSCalibrationDB::AliPHOSCalibrationDB():fPedestals(), fGains()
 {
 }
 //____________________________________________________________________________ 
index f6ad49f..950dc9a 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.11  2006/01/13 16:59:39  kharlov
+ * Rename classes to read raw data 2004
+ *
  * Revision 1.10  2005/05/28 14:19:04  schutz
  * Compilation warnings fixed by T.P.
  *
@@ -82,48 +85,82 @@ ClassImp(AliPHOSCalibrator)
 
 
 //____________________________________________________________________________ 
-  AliPHOSCalibrator::AliPHOSCalibrator():TTask("AliPHOSCalibrator","Default") 
+AliPHOSCalibrator::AliPHOSCalibrator() :
+  TTask("AliPHOSCalibrator","Default"),
+  fRunList(0),
+  fPedHistos(0),
+  fGainHistos(0),
+  fhPedestals(0),
+  fhPedestalsWid(0),
+  fhGains(0),
+  fhGainsWid(0),
+  fctdb(0),
+  fConTableDB("Beamtest2002"),
+  fConTableDBFile("ConTableDB.root"),
+  fBeamEnergy(0),
+  fGainAcceptCorr(0),
+  fAcceptCorr(0),
+  fGainMax(0),
+  fNGainBins(0),
+  fNch(0),
+  fNChan(0)  
 {
   //Default constuctor for root. Normally should not be used
-  fRunList=0 ;
-  fBeamEnergy = 0. ;
-  fNch = 0 ;
-  fPedHistos = 0 ;
-  fGainHistos = 0 ;
-  fhPedestals = 0 ;
-  fhPedestalsWid = 0 ;
-  fctdb = 0 ;
-  fConTableDB = "Beamtest2002" ;
-  fConTableDBFile = "ConTableDB.root" ;
 }
+
 //____________________________________________________________________________ 
 AliPHOSCalibrator::AliPHOSCalibrator(const char* file, const char* title):
-  TTask("AliPHOSCalibrator",title) 
+  TTask("AliPHOSCalibrator",title),
+  fRunList(new TList),
+  fPedHistos(0),
+  fGainHistos(0),
+  fhPedestals(0),
+  fhPedestalsWid(0),
+  fhGains(0),
+  fhGainsWid(0),
+  fctdb(0),
+  fConTableDB("Beamtest2002"),
+  fConTableDBFile("ConTableDB.root"),
+  fBeamEnergy(10.),
+  fGainAcceptCorr(5),
+  fAcceptCorr(10),
+  fGainMax(0.1),
+  fNGainBins(100),
+  fNch(0),
+  fNChan(100)
+
 { 
   //Constructor which should normally be used.
   //file: path/galice.root  - header file
   //title: branch name of PHOS reconstruction (e.g. "Default")
-
-  fRunList = new TList() ;
-  fRunList->SetOwner() ;
-  fRunList->Add(new TObjString(file)) ;
-  fNch = 0 ;
-  fBeamEnergy = 10. ;
-
-  fNChan  = 100 ;  
-  fGainMax = 0.1 ;
-  fNGainBins= 100 ;
-  fAcceptCorr = 10 ;     //Maximal deviation from mean, considered as normal 
-
-  fGainAcceptCorr = 5 ;  //Factor for gain deviation
-  fPedHistos = 0 ;
-  fGainHistos = 0 ;
-  fhPedestals = 0 ;
-  fhPedestalsWid = 0 ;
-  fctdb = 0 ;
-  fConTableDB = "Beamtest2002" ;
-  fConTableDBFile = "ConTableDB.root" ;
+  fRunList->SetOwner();
+  fRunList->Add(new TObjString(file));
+}
+
+//____________________________________________________________________________ 
+AliPHOSCalibrator::AliPHOSCalibrator(const AliPHOSCalibrator & ctor) : 
+  TTask(ctor),
+  fRunList(0),
+  fPedHistos(0),
+  fGainHistos(0),
+  fhPedestals(0),
+  fhPedestalsWid(0),
+  fhGains(0),
+  fhGainsWid(0),
+  fctdb(0),
+  fConTableDB("Beamtest2002"),
+  fConTableDBFile("ConTableDB.root"),
+  fBeamEnergy(0),
+  fGainAcceptCorr(0),
+  fAcceptCorr(0),
+  fGainMax(0),
+  fNGainBins(0),
+  fNch(0),
+  fNChan(0)  
+{
+  // cpy ctor: no implementation yet
+  // requested by the Coding Convention
+  Fatal("cpy ctor", "not implemented") ;
 }
 
 //____________________________________________________________________________ 
index dcc47c1..26a9d31 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.10  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 
@@ -33,11 +36,7 @@ class AliPHOSCalibrator: public TTask{
 public:
   AliPHOSCalibrator() ;          // ctor
   AliPHOSCalibrator(const char* run, const char * title = "Default") ;
-  AliPHOSCalibrator(const AliPHOSCalibrator & ctor) : TTask(ctor) {
-    // cpy ctor: no implementation yet
-    // requested by the Coding Convention
-    Fatal("cpy ctor", "not implemented") ;
-  }
+  AliPHOSCalibrator(const AliPHOSCalibrator & ctor);
  
   virtual ~AliPHOSCalibrator() ; // dtor
 
index 82ee818..4477d8c 100644 (file)
 
 ClassImp(AliPHOSEvalRecPoint)
 
-  AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(): fEventFolderName(AliConfig::GetDefaultEventFolderName())
+AliPHOSEvalRecPoint::AliPHOSEvalRecPoint() :
+  fIsEmc(kFALSE),
+  fIsCpv(kTRUE),
+  fParent(-333),
+  fChi2Dof(-111),
+  fEventFolderName(AliConfig::GetDefaultEventFolderName())
 {
   // default ctor
-  fParent=-333;
-  fChi2Dof=-111;
-  fIsCpv = kTRUE;
-  fIsEmc = kFALSE;
 }
 
-AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(Bool_t cpv, AliPHOSEvalRecPoint* parent) : AliPHOSCpvRecPoint()
+AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(Bool_t cpv, AliPHOSEvalRecPoint* parent) : 
+  fIsEmc(kFALSE),
+  fIsCpv(kFALSE),
+  fParent(-333),
+  fChi2Dof(-111),
+  fEventFolderName("")
 {
   // ctor
   fParent=-333;
index 0c5c366..6c39d45 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.39  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 ClassImp(AliPHOSFastRecParticle) 
 
 //____________________________________________________________________________
-AliPHOSFastRecParticle::AliPHOSFastRecParticle() : TParticle()
+AliPHOSFastRecParticle::AliPHOSFastRecParticle() :
+  fIndexInList(0),
+  fTof(0.f),
+  fType(0)
 {
   // ctor
-  fType = 0 ; 
 }
 
 //____________________________________________________________________________
  AliPHOSFastRecParticle::AliPHOSFastRecParticle(const AliPHOSFastRecParticle & rp)
-   : TParticle(rp)
+   : TParticle(rp),
+     fIndexInList(rp.fIndexInList),//?
+     fTof(rp.fTof),//?
+     fType(rp.fType)
 {
   // copy ctor
-
-  fType        = rp.fType ;
   fPdgCode     = rp.fPdgCode;
   fStatusCode  = rp.fStatusCode;
   fMother[0]   = rp.fMother[0];
@@ -77,13 +83,14 @@ AliPHOSFastRecParticle::AliPHOSFastRecParticle() : TParticle()
 }
 
 //____________________________________________________________________________
- AliPHOSFastRecParticle::AliPHOSFastRecParticle(const TParticle & pp)
+ AliPHOSFastRecParticle::AliPHOSFastRecParticle(const TParticle & pp) :
+   fIndexInList(0),
+   fTof(0.f),
+   fType(0)
 {
   // ctor from a TParticle (crummy?!)
   TParticle & pnoconst = (TParticle &)(pp) ;
   AliPHOSFastRecParticle & p = (AliPHOSFastRecParticle &)(pnoconst) ;
-  fType        = 0  ;
   fPdgCode     = p.fPdgCode;
   fStatusCode  = p.fStatusCode;
   fMother[0]   = p.fMother[0];
index b588077..99c8cab 100644 (file)
 ClassImp(AliPHOSJet)
   
 //____________________________________________________________________________ 
-AliPHOSJet::AliPHOSJet():TObject() {
+AliPHOSJet::AliPHOSJet():
+  fNpart(0),
+  fList(0),
+  fConeRad(0),
+  fMaxConeMove(0),
+  fMinConeMove(0),
+  fSumEnergy(0),
+  fSumEta(0),
+  fSumPhi(0),
+  fEnergy(0),
+  fEta(0),
+  fPhi(0),
+  fLEnergy(0),
+  fLEta(0),
+  fLPhi(0) 
+{
   //Initialize members
-  fNpart = 0 ;
-  fList = 0 ;
-  //  fMode = 0 ;
-  fConeRad = 0 ;
-  fMaxConeMove = 0 ;
-  fMinConeMove = 0 ;
-  
-  fSumEnergy = 0 ;
-  fSumEta = 0 ;
-  fSumPhi = 0 ; 
-  fEnergy = 0 ;
-  fEta = 0 ;
-  fPhi = 0 ; 
-  fLEnergy = 0 ;
-  fLEta = 0 ;
-  fLPhi = 0 ; 
 }
 
 //____________________________________________________________________________ 
+AliPHOSJet::AliPHOSJet(const AliPHOSJet & jet) : 
+  TObject(jet),
+  fNpart(0),
+  fList(0),
+  fConeRad(0),
+  fMaxConeMove(0),
+  fMinConeMove(0),
+  fSumEnergy(0),
+  fSumEta(0),
+  fSumPhi(0),
+  fEnergy(0),
+  fEta(0),
+  fPhi(0),
+  fLEnergy(0),
+  fLEta(0),
+  fLPhi(0) 
+{
+  // copy ctor: no implementation yet
+  Fatal("cpy ctor", "not implemented") ;
+}
+
+
+//____________________________________________________________________________ 
 AliPHOSJet::~AliPHOSJet(){
   //dtor
   if(fList){
index 42246df..dd178a3 100644 (file)
@@ -24,10 +24,8 @@ class AliPHOSJet : public TObject {
 
 public:
   AliPHOSJet() ;          // ctor
-  AliPHOSJet(const AliPHOSJet & jet) : TObject(jet){
-    // copy ctor: no implementation yet
-    Fatal("cpy ctor", "not implemented") ;
-  }
+  AliPHOSJet(const AliPHOSJet & jet);
+  
   virtual ~AliPHOSJet() ; 
 
   void AddDigit(Double_t e, Double_t eta, Double_t phi, Int_t index) ;
index dd14f16..0297f6c 100644 (file)
@@ -26,6 +26,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.8  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 // --- ROOT system ---
@@ -48,25 +51,44 @@ ClassImp(AliPHOSJetFinder)
 
 
 //____________________________________________________________________________ 
-  AliPHOSJetFinder::AliPHOSJetFinder():TNamed("AliPHOSJetFinder","") 
+AliPHOSJetFinder::AliPHOSJetFinder():
+  TNamed("AliPHOSJetFinder",""),
+  fNJets(0),
+  fStatusCode(-999),
+  fMode(0),
+  fConeRad(1.),
+  fMaxConeMove(0.15),
+  fMinConeMove(0.05),
+  fEtSeed(4.),
+  fEtMin(5.),
+  fPrecBg(0.00035),
+  fSimGain(0.),
+  fSimPedestal(0.),
+  fParticles(0),
+  fJets(0)
 {
   //Initialize jet parameters
-  fNJets = 0 ; 
-  fMode  = 0 ;   //No iterations 
-  fStatusCode = -999 ; //no selection
-
-  fConeRad = 1.;   //Radius of jet value?????????
-  fMaxConeMove = 0.15 ; //value???????
-  fMinConeMove = 0.05 ; //value???????
-  fEtSeed = 4. ; //Energy of seed particle value??????????
-  fEtMin = 5.;   //minimal energy of jet   value??????????
-  fPrecBg = 0.00035 ; //value????????
-  fSimGain = 0.;
-  fSimPedestal = 0.;
+}
 
-  fParticles = 0;
-  fJets = 0 ;
-  
+//____________________________________________________________________________ 
+AliPHOSJetFinder::AliPHOSJetFinder(const AliPHOSJetFinder & jet) : 
+  TNamed(jet),
+  fNJets(0),
+  fStatusCode(-999),
+  fMode(0),
+  fConeRad(1.),
+  fMaxConeMove(0.15),
+  fMinConeMove(0.05),
+  fEtSeed(4.),
+  fEtMin(5.),
+  fPrecBg(0.00035),
+  fSimGain(0.),
+  fSimPedestal(0.),
+  fParticles(0),
+  fJets(0)
+{
+  // copy ctor: no implementation yet
+  Fatal("cpy ctor", "not implemented");
 }
 
 //____________________________________________________________________________ 
index bab15b1..4d7e48d 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.6  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -29,10 +32,7 @@ class AliPHOSJetFinder : public TNamed {
 
 public:
   AliPHOSJetFinder() ;          // ctor
-  AliPHOSJetFinder(const AliPHOSJetFinder & jet) : TNamed(jet) {
-    // copy ctor: no implementation yet
-    Fatal("cpy ctor", "not implemented") ;
-  }
+  AliPHOSJetFinder(const AliPHOSJetFinder & jet);
   virtual ~AliPHOSJetFinder() ; // dtor
 
   void FindJetsFromParticles(const TClonesArray * plist,TObjArray * jetslist) ; //Do the job
index 690e79b..5e63c93 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.18  2006/08/28 10:01:56  kharlov
+ * Effective C++ warnings fixed (Timur Pocheptsov)
+ *
  * Revision 1.17  2006/08/25 16:00:53  kharlov
  * Compliance with Effective C++AliPHOSHit.cxx
  *
@@ -650,6 +653,8 @@ Int_t AliPHOSLoader::ReadTracks()
    }
   
   TBranch * branch = treeT->GetBranch(fgkTrackSegmentsBranchName);
+//   AliInfo(Form("Branch named %s is opened: 0x%z",
+//               fgkTrackSegmentsBranchName.Data(),branch));
   if (branch == 0) 
    {//easy, maybe just a new tree
     AliError(Form("Cannot find branch named %s",
index 2516937..f7957bf 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.13  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 ClassImp( AliPHOSPIDv0) 
 
 //____________________________________________________________________________
-AliPHOSPIDv0::AliPHOSPIDv0():AliPHOSPID()
+AliPHOSPIDv0::AliPHOSPIDv0():
+  fTrackSegmentsTitle(""), 
+  fRecPointsTitle(""),
+  fRecParticlesTitle(""),
+  fIDOptions("dis time"),
+  fNEvent(0),
+  fClusterizer(0),
+  fTSMaker(0),
+  fFormula(0),
+  fDispersion(0.f),
+  fCpvEmcDistance(0.f),
+  fTimeGate(2.e-9f),
+  fRecParticlesInRun(0)
 { 
   // default ctor
-  fFormula           = 0 ;
-  fDispersion        = 0. ; 
-  fCpvEmcDistance    = 0 ; 
-  fTimeGate          = 2.e-9 ;
-  fEventFolderName    = "" ; 
-  fTrackSegmentsTitle= "" ; 
-  fRecPointsTitle    = "" ; 
-  fRecParticlesTitle = "" ; 
-  fIDOptions         = "dis time" ; 
-  fRecParticlesInRun = 0 ;
-  fClusterizer = 0;
-  fTSMaker = 0;
+  fEventFolderName = ""; 
 }
 
 //____________________________________________________________________________
-AliPHOSPIDv0::AliPHOSPIDv0(const char * evFolderName,const char * name) : AliPHOSPID(evFolderName, name)
+AliPHOSPIDv0::AliPHOSPIDv0(const char * evFolderName,const char * name) : 
+  AliPHOSPID(evFolderName, name),
+  fTrackSegmentsTitle(GetName()), 
+  fRecPointsTitle(GetName()),
+  fRecParticlesTitle(GetName()),
+  fIDOptions("dis time"),
+  fNEvent(0),
+  fClusterizer(0),
+  fTSMaker(0),
+  fFormula(new TFormula("LambdaCuts","(x>1)*(x<2.5)*(y>0)*(y<x)")),
+  fDispersion(2.f),
+  fCpvEmcDistance(3.f),
+  fTimeGate(2.e-9f),
+  fRecParticlesInRun(0)
 { 
   //ctor with the indication on where to look for the track segments
-
-  fFormula        = new TFormula("LambdaCuts","(x>1)*(x<2.5)*(y>0)*(y<x)") ;   
-  fDispersion     = 2.0 ; 
-  fCpvEmcDistance = 3.0 ;
-  fTimeGate          = 2.e-9 ;
-  fEventFolderName     = GetTitle() ; 
-  fTrackSegmentsTitle = GetName();
-  fRecPointsTitle     = GetName();
-  fRecParticlesTitle  = GetName();
-  fIDOptions          = "dis time" ;
-    
-  fRecParticlesInRun = 0 ; 
-
+  fEventFolderName    = GetTitle() ; 
   Init() ;
+}
 
+//____________________________________________________________________________
+AliPHOSPIDv0::AliPHOSPIDv0(const AliPHOSPIDv0 & rhs) :
+  AliPHOSPID(rhs),
+  fTrackSegmentsTitle(rhs.fTrackSegmentsTitle), 
+  fRecPointsTitle(rhs.fRecPointsTitle),
+  fRecParticlesTitle(rhs.fRecParticlesTitle),
+  fIDOptions(rhs.fIDOptions),
+  fNEvent(rhs.fNEvent),
+  fClusterizer(rhs.fClusterizer),
+  fTSMaker(rhs.fTSMaker),
+  fFormula(rhs.fFormula),
+  fDispersion(rhs.fDispersion),
+  fCpvEmcDistance(rhs.fCpvEmcDistance),
+  fTimeGate(rhs.fTimeGate),
+  fRecParticlesInRun(rhs.fRecParticlesInRun)
+{
+  //Copy ctor, the same as compiler-generated, possibly wrong if
+  //someone implements dtor correctly.
+}
+  
+//____________________________________________________________________________
+AliPHOSPIDv0 & AliPHOSPIDv0::operator = (const AliPHOSPIDv0 & rhs)
+{
+  //Copy-assignment, emulates compiler generated, possibly wrong.
+  AliPHOSPID::operator = (rhs);
+  fTrackSegmentsTitle = rhs.fTrackSegmentsTitle;
+  fRecPointsTitle = rhs.fRecPointsTitle;
+  fRecParticlesTitle = rhs.fRecParticlesTitle;
+  fIDOptions = rhs.fIDOptions;
+  fNEvent = rhs.fNEvent;
+  fClusterizer = rhs.fClusterizer;
+  fTSMaker = rhs.fTSMaker;
+  fFormula = rhs.fFormula;
+  fDispersion = rhs.fDispersion;
+  fCpvEmcDistance = rhs.fCpvEmcDistance;
+  fTimeGate = rhs.fTimeGate;
+  fRecParticlesInRun = rhs.fRecParticlesInRun;
+
+  return *this;
 }
 
 //____________________________________________________________________________
 AliPHOSPIDv0::~AliPHOSPIDv0()
-{ 
+{
+  //Empty dtor, fFormula leaks 
 }
 
 //____________________________________________________________________________
index 3918fc0..9563acd 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.10  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -34,7 +37,11 @@ public:
 
   AliPHOSPIDv0() ;          // ctor            
   AliPHOSPIDv0(const char* evFolderName, const char * tsBranch = "Default");
-  virtual ~AliPHOSPIDv0() ; // dtor
+  virtual ~AliPHOSPIDv0() ; // dtor, empty, memory leak in fFormula member
+
+  //Compiler generated should be ok, because destructor is empty.
+  AliPHOSPIDv0(const AliPHOSPIDv0 & rhs);
+  AliPHOSPIDv0 & operator = (const AliPHOSPIDv0 & rhs);
 
   virtual void Exec(Option_t * option);
   //  virtual char * GetRecParticlesBranch()const {return (char*) fRecParticlesTitle.Data() ;}      
index 0348eff..3448da9 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.102  2006/01/23 17:51:48  hristov
+ * Using the recommended way of forward declarations for TVector and TMatrix (see v5-08-00 release notes). Additional clean-up
+ *
  * Revision 1.101  2005/05/28 14:19:04  schutz
  * Compilation warnings fixed by T.P.
  *
 ClassImp( AliPHOSPIDv1) 
 
 //____________________________________________________________________________
-AliPHOSPIDv1::AliPHOSPIDv1():AliPHOSPID()
+AliPHOSPIDv1::AliPHOSPIDv1() :
+  fBayesian(kFALSE),
+  fDefaultInit(kFALSE),
+  fWrite(kFALSE),
+  fNEvent(0),
+  fFileNamePrincipalPhoton(),
+  fFileNamePrincipalPi0(),
+  fFileNameParameters(),
+  fPrincipalPhoton(0),
+  fPrincipalPi0(0),
+  fX(0),
+  fPPhoton(0),
+  fPPi0(0),
+  fRecParticlesInRun(0),
+  fParameters(0),
+  fTFphoton(0),
+  fTFpiong(0),
+  fTFkaong(0),
+  fTFkaonl(0),
+  fTFhhadrong(0),
+  fTFhhadronl(0),
+  fDFmuon(0),
+  fERecWeight(0),
+  fChargedNeutralThreshold(0.),
+  fTOFEnThreshold(0),
+  fDispEnThreshold(0),
+  fDispMultThreshold(0)
 { 
   // default ctor
  
@@ -116,7 +145,35 @@ AliPHOSPIDv1::AliPHOSPIDv1():AliPHOSPID()
 }
 
 //____________________________________________________________________________
-AliPHOSPIDv1::AliPHOSPIDv1(const AliPHOSPIDv1 & pid ):AliPHOSPID(pid)
+AliPHOSPIDv1::AliPHOSPIDv1(const AliPHOSPIDv1 & pid ) : 
+  AliPHOSPID(pid),
+  fBayesian(kFALSE),
+  fDefaultInit(kFALSE),
+  fWrite(kFALSE),
+  fNEvent(0),
+  fFileNamePrincipalPhoton(),
+  fFileNamePrincipalPi0(),
+  fFileNameParameters(),
+  fPrincipalPhoton(0),
+  fPrincipalPi0(0),
+  fX(0),
+  fPPhoton(0),
+  fPPi0(0),
+  fRecParticlesInRun(0),
+  fParameters(0),
+  fTFphoton(0),
+  fTFpiong(0),
+  fTFkaong(0),
+  fTFkaonl(0),
+  fTFhhadrong(0),
+  fTFhhadronl(0),
+  fDFmuon(0),
+  fERecWeight(0),
+  fChargedNeutralThreshold(0.),
+  fTOFEnThreshold(0),
+  fDispEnThreshold(0),
+  fDispMultThreshold(0)
+
 { 
   // ctor
   InitParameters() ; 
@@ -125,7 +182,35 @@ AliPHOSPIDv1::AliPHOSPIDv1(const AliPHOSPIDv1 & pid ):AliPHOSPID(pid)
 }
 
 //____________________________________________________________________________
-AliPHOSPIDv1::AliPHOSPIDv1(const TString alirunFileName, const TString eventFolderName):AliPHOSPID(alirunFileName, eventFolderName)
+AliPHOSPIDv1::AliPHOSPIDv1(const TString alirunFileName, const TString eventFolderName) :
+  AliPHOSPID(alirunFileName, eventFolderName),
+  fBayesian(kFALSE),
+  fDefaultInit(kFALSE),
+  fWrite(kFALSE),
+  fNEvent(0),
+  fFileNamePrincipalPhoton(),
+  fFileNamePrincipalPi0(),
+  fFileNameParameters(),
+  fPrincipalPhoton(0),
+  fPrincipalPi0(0),
+  fX(0),
+  fPPhoton(0),
+  fPPi0(0),
+  fRecParticlesInRun(0),
+  fParameters(0),
+  fTFphoton(0),
+  fTFpiong(0),
+  fTFkaong(0),
+  fTFkaonl(0),
+  fTFhhadrong(0),
+  fTFhhadronl(0),
+  fDFmuon(0),
+  fERecWeight(0),
+  fChargedNeutralThreshold(0.),
+  fTOFEnThreshold(0),
+  fDispEnThreshold(0),
+  fDispMultThreshold(0)
+
 { 
   //ctor with the indication on where to look for the track segments
  
index 219797a..1325a7f 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.12  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 
 ClassImp(AliPHOSRaw2Digits)
   
-  
 //____________________________________________________________________________ 
-  AliPHOSRaw2Digits::AliPHOSRaw2Digits():TTask() 
+AliPHOSRaw2Digits::AliPHOSRaw2Digits() : 
+  fDigits(0),
+  fPHOSHeader(0),
+  fctdb(0),
+  fHeaderFile(0),
+  fDigitsFile(0),
+  fBeamEnergy(0.f),
+  fMaxPerFile(20000),
+  fEvent(0),
+  fStatus(0),
+  fInName(""),
+  fDebug(kFALSE),
+  fIsInitialized(kFALSE),
+  fMK1(0x0123CDEF),
+  fMK2(0x80708070),
+  fMK3(0x4321ABCD),
+  fMK4(0x80618061),
+  fCKW(0x4640E400)
 {
   //As one can easily see, this is constructor.
-  fInName="";  
-  fMK1 = 0x0123CDEF ;
-  fMK2 = 0x80708070 ;
-  fMK3 = 0x4321ABCD ;
-  fMK4 = 0x80618061 ;
-  fCKW = 0x4640E400 ;
-  fDebug = kFALSE;             //  Debug flag
-  fIsInitialized = kFALSE ;
   fTarget[0] = 0 ;
   fTarget[1] = 0 ;
   fTarget[2] = 0 ;
-  fDigits = 0 ;
-  fHeaderFile = 0 ;
-  fDigitsFile = 0 ;
-  fPHOSHeader =0 ;
-  fMaxPerFile = 20000 ;  //Maximal number of events in root file.
-  fEvent = 0 ;
-  fctdb = 0;
 }
+
 //____________________________________________________________________________ 
-  AliPHOSRaw2Digits::AliPHOSRaw2Digits(const char * filename):TTask("Default","") 
+AliPHOSRaw2Digits::AliPHOSRaw2Digits(const char * filename) : 
+  TTask("Default",""),
+  fDigits(0),
+  fPHOSHeader(0),
+  fctdb(0),
+  fHeaderFile(0),
+  fDigitsFile(0),
+  fBeamEnergy(0.f),
+  fMaxPerFile(20000),
+  fEvent(0),
+  fStatus(0),
+  fInName(filename),
+  fDebug(kFALSE),
+  fIsInitialized(kFALSE),
+  fMK1(0x0123CDEF),
+  fMK2(0x80708070),
+  fMK3(0x4321ABCD),
+  fMK4(0x80618061),
+  fCKW(0x4640E400)
 {
-  //this constructor should be normally used. Parameters: imput file 
-  fInName=filename;
-  TString outname("") ;
-  outname =fInName ;
+  //this constructor should be normally used. Parameters: input file 
+  TString outname(fInName) ;
   outname.ToLower() ;
   outname.ReplaceAll(".fz",".root") ;
   outname.ReplaceAll(".gz","") ;
-  
-  SetTitle(outname) ;
-
-  fMK1 = 0x0123CDEF ;
-  fMK2 = 0x80708070 ;
-  fMK3 = 0x4321ABCD ;
-  fMK4 = 0x80618061 ;
-  fCKW = 0x4640E400 ;
-  fDebug = kFALSE;             //  Debug flag
-  fIsInitialized = kFALSE ;
+  SetTitle(outname);
+
   fTarget[0] = 0 ;
   fTarget[1] = 0 ;
   fTarget[2] = 0 ;
-  fDigits = 0 ;
-  fPHOSHeader =0 ;
-  fHeaderFile = 0 ;
-  fDigitsFile = 0 ;
-  fMaxPerFile = 20000 ;
-  fEvent = 0 ;
-  fctdb = 0;
 }
+
 //____________________________________________________________________________ 
-AliPHOSRaw2Digits::AliPHOSRaw2Digits(AliPHOSRaw2Digits & r2d):TTask(r2d.GetName(), r2d.GetTitle()) 
+AliPHOSRaw2Digits::AliPHOSRaw2Digits(AliPHOSRaw2Digits & r2d) :
+  TTask(r2d.GetName(), r2d.GetTitle()),
+  fDigits(r2d.fDigits),
+  fPHOSHeader(r2d.fPHOSHeader),
+  fctdb(new AliPHOSConTableDB(*r2d.fctdb)),
+  fHeaderFile(new TFile(r2d.fHeaderFile->GetName(), "new" )),
+  fDigitsFile(new TFile(r2d.fDigitsFile->GetName(), "new" )),
+  fBeamEnergy(r2d.fBeamEnergy),
+  fMaxPerFile(r2d.fMaxPerFile),
+  fEvent(r2d.fEvent),
+  fStatus(r2d.fStatus),
+  fInName(r2d.fInName),
+  fDebug(kFALSE),
+  fIsInitialized(kFALSE),
+  fMK1(r2d.fMK1),
+  fMK2(r2d.fMK2),
+  fMK3(r2d.fMK3),
+  fMK4(r2d.fMK4),
+  fCKW(r2d.fCKW)
 {
-  // cpy ctor
-  fInName=r2d.fInName ;
-
-  fMK1 =  r2d.fMK1 ;
-  fMK2 =  r2d.fMK2 ;
-  fMK3 =  r2d.fMK3 ;
-  fMK4 =  r2d.fMK4 ;
-  fCKW =  r2d.fCKW ;
-  fDebug =  kFALSE;             //  Debug flag
-  fIsInitialized =  kFALSE ;
+  // cpy ctor. wrong. because dtor can delete fDigits twice (or n times you copy AliPHOSRaw2Digits)
+  //because fHeaderFile and fDigitsFile will recreate existing files etc.
   fTarget[0] = r2d.fTarget[0] ;
   fTarget[1] = r2d.fTarget[1] ;
   fTarget[2] = r2d.fTarget[2] ;
-  fDigits = r2d.fDigits ;
-  fPHOSHeader = r2d.fPHOSHeader  ;
-  fHeaderFile = new TFile( (r2d.fHeaderFile)->GetName(), "new" ) ;
-  fDigitsFile = new TFile( (r2d.fDigitsFile)->GetName(), "new" ) ;
-  fMaxPerFile = r2d.fMaxPerFile ;
-  fEvent = r2d.fEvent ;
-  fctdb =  new AliPHOSConTableDB( *(r2d.fctdb) ) ;
 }
 
 //____________________________________________________________________________ 
index d4b0681..e90512e 100644 (file)
@@ -32,8 +32,6 @@
 #include "TClonesArray.h"
 #include "TClass.h"
 
-
-
 #include "AliPHOSRawStream2004.h"
 #include "AliRawReader.h"
 #include "AliRawEventHeaderBase.h"
 ClassImp(AliPHOSRawStream2004)
 
 //_____________________________________________________________________________
-AliPHOSRawStream2004::AliPHOSRawStream2004(AliRawReader* rawReader) : TObject()
+AliPHOSRawStream2004::AliPHOSRawStream2004(AliRawReader* rawReader) :
+  fRawReader(rawReader),
+  fData(0),
+  fctdb(0),
+  fTrig(0)
+{
+  //Ctor
+}
+
+//_____________________________________________________________________________
+AliPHOSRawStream2004::AliPHOSRawStream2004(const AliPHOSRawStream2004 & rhs) :
+  TObject(rhs),
+  fRawReader(rhs.fRawReader),
+  fData(rhs.fData),
+  fctdb(rhs.fctdb),
+  fTrig(rhs.fTrig)
 {
-  fRawReader = rawReader ;
-  fctdb = 0 ;
+  //Copy ctor
 }
+
+//_____________________________________________________________________________
+AliPHOSRawStream2004 & AliPHOSRawStream2004::operator = (const AliPHOSRawStream2004 &rhs)
+{
+  TObject::operator = (rhs);
+  fRawReader = rhs.fRawReader;
+  fData = rhs.fData;
+  fctdb = rhs.fctdb;
+  fTrig = rhs.fTrig;
+  
+  return *this;
+}
+
 //_____________________________________________________________________________
 Bool_t AliPHOSRawStream2004::ReadDigits(TClonesArray * digits){
 
index 7cfa60b..c45c491 100644 (file)
@@ -26,6 +26,12 @@ public :
   
   AliPHOSRawStream2004(AliRawReader* rawReader);
   
+  //Class does not own/free any resources, compiler generated version
+  //are OK, but to supress -Weffc++ we define them explicitly.
+  AliPHOSRawStream2004(const AliPHOSRawStream2004 & rhs);
+  AliPHOSRawStream2004 & operator = (const AliPHOSRawStream2004 & rhs);
+
+  
 
  Bool_t ReadDigits(TClonesArray * digits) ;
  
index fba2e8a..4947b41 100644 (file)
 ClassImp(AliPHOSRecCpvManager) 
 
 //____________________________________________________________________________
-
-  AliPHOSRecCpvManager::AliPHOSRecCpvManager()
+AliPHOSRecCpvManager::AliPHOSRecCpvManager() :
+  fOneGamChisqCut(3.),
+  fOneGamInitialStep(0.00005),
+  fOneGamChisqMin(1.),
+  fOneGamStepMin(0.0005),
+  fOneGamNumOfIterations(50),
+  fTwoGamInitialStep(0.00005),
+  fTwoGamChisqMin(1.),
+  fTwoGamEmin(0.1),
+  fTwoGamStepMin(0.00005),
+  fTwoGamNumOfIterations(50),  
+  fThr0(0.),
+  fSqdCut(0.) 
 {
   // Put a comment here
-
-  fOneGamChisqCut = 3.; // If Chi2/dof > fOneGamChisqCut, split point.
-
-  fOneGamInitialStep = 0.00005;
-  fOneGamChisqMin = 1.;
-  fOneGamStepMin = 0.0005;
-  fOneGamNumOfIterations = 50;
-
-  fTwoGamInitialStep = 0.00005;
-  fTwoGamChisqMin = 1.;
-  fTwoGamEmin = 0.1;
-  fTwoGamStepMin = 0.00005;
-  fTwoGamNumOfIterations = 50;  
-
-//    fThr0 = 0.0285; // Min. energy of rec. point. If E<fThr0, point deleted.
-//    fSqdCut = 3.; // Min. distance (in cm) between two rec points.
-
-  fThr0 = 0.; 
-  fSqdCut = 0.; 
-  
   SetTitle("Cpv Reconstruction Manager");
 }
 
index 907f8ca..9f69651 100644 (file)
@@ -31,31 +31,22 @@ ClassImp(AliPHOSRecEmcManager)
 
 //____________________________________________________________________________
 
-  AliPHOSRecEmcManager::AliPHOSRecEmcManager()
+AliPHOSRecEmcManager::AliPHOSRecEmcManager():
+  fOneGamChisqCut(1.3f),
+  fOneGamInitialStep(0.00005f),
+  fOneGamChisqMin(1.f),
+  fOneGamStepMin(0.0005f),
+  fOneGamNumOfIterations(50),
+  fTwoGamInitialStep(0.00005f),
+  fTwoGamChisqMin(1.f),
+  fTwoGamEmin(0.1f),
+  fTwoGamStepMin(0.00005),
+  fTwoGamNumOfIterations(50),
+  fThr0(0.f),
+  fSqdCut(0.f)
 {
   // default ctor
-//    fOneGamChisqCut = 3.;
-  fOneGamChisqCut = 1.3; // bvp 31.08.2001 
-
-  fOneGamInitialStep = 0.00005;
-  fOneGamChisqMin = 1.;
-  fOneGamStepMin = 0.0005;
-  fOneGamNumOfIterations = 50;
-
-  fTwoGamInitialStep = 0.00005;
-  fTwoGamChisqMin = 1.;
-  fTwoGamEmin = 0.1;
-  fTwoGamStepMin = 0.00005;
-  fTwoGamNumOfIterations = 50;  
-
-//    fThr0 = 0.6;
-  fThr0 = 0.;
-//    fSqdCut = 3.;
-//    fSqdCut = 0.5; // bvp 31.08.2001
-  fSqdCut = 0.;
-
   SetTitle("Emc Reconstruction Manager");
-
 }
 
 AliPHOSRecEmcManager::~AliPHOSRecEmcManager(void) {}