Effective C++ corrections (T.Pocheptsov)
authorkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Sep 2006 07:31:01 +0000 (07:31 +0000)
committerkharlov <kharlov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Sep 2006 07:31:01 +0000 (07:31 +0000)
27 files changed:
PHOS/AliPHOSAnalyze.cxx
PHOS/AliPHOSBeamTestEvent.cxx
PHOS/AliPHOSCPVDigit.cxx
PHOS/AliPHOSCPVDigit.h
PHOS/AliPHOSCalibrationData.cxx
PHOS/AliPHOSConTableDB.cxx
PHOS/AliPHOSEvalRecPoint.cxx
PHOS/AliPHOSFastGlobalReconstruction.cxx
PHOS/AliPHOSFastGlobalReconstruction.h
PHOS/AliPHOSGridFile.cxx
PHOS/AliPHOSGridFile.h
PHOS/AliPHOSIhepAnalyze.cxx
PHOS/AliPHOSIhepAnalyze.h
PHOS/AliPHOSImpact.cxx
PHOS/AliPHOSImpact.h
PHOS/AliPHOSLink.cxx
PHOS/AliPHOSOnlineMonitor.cxx
PHOS/AliPHOSOnlineMonitor.h
PHOS/AliPHOSRecParticle.cxx
PHOS/AliPHOSTick.cxx
PHOS/AliPHOSTracker.h
PHOS/AliPHOSv1.cxx
PHOS/AliPHOSv1.h
PHOS/AliPHOSvFast.cxx
PHOS/AliPHOSvFast.h
PHOS/AliPHOSvImpacts.cxx
PHOS/AliPHOSvImpacts.h

index 2f8da5d..d1c0068 100644 (file)
 ClassImp(AliPHOSAnalyze)
 
 //____________________________________________________________________________
-  AliPHOSAnalyze::AliPHOSAnalyze()
+AliPHOSAnalyze::AliPHOSAnalyze():
+  fCorrection(1.2),  //Value calculated for default parameters of reconstruction
+  fEvt(0),
+  ffileName(),
+  fRunLoader(0)
 {
   // default ctor (useless)
-  fCorrection = 1.2 ;  //Value calculated for default parameters of reconstruction  
-  fRunLoader = 0x0;
 }
 
 //____________________________________________________________________________
-AliPHOSAnalyze::AliPHOSAnalyze(Text_t * fileName)
+AliPHOSAnalyze::AliPHOSAnalyze(Text_t * fileName):
+  fCorrection(1.05),  //Value calculated for default parameters of reconstruction   
+  fEvt(0),
+  ffileName(fileName),
+  fRunLoader(0)
 {
   // ctor: analyze events from root file "name"
-  ffileName = fileName;
-  fCorrection = 1.05 ;  //Value calculated for default parameters of reconstruction   
   fRunLoader = AliRunLoader::Open(fileName,"AliPHOSAnalyze");
   if (fRunLoader == 0x0)
    {
@@ -113,8 +117,12 @@ AliPHOSAnalyze::AliPHOSAnalyze(Text_t * fileName)
 }
 
 //____________________________________________________________________________
-AliPHOSAnalyze::AliPHOSAnalyze(const AliPHOSAnalyze & ana)
-  : TObject(ana)
+AliPHOSAnalyze::AliPHOSAnalyze(const AliPHOSAnalyze & ana): 
+  TObject(ana),
+  fCorrection(0.),
+  fEvt(0),
+  ffileName(),
+  fRunLoader(0)
 {
   // copy ctor
   ( (AliPHOSAnalyze &)ana ).Copy(*this) ;
index 2727695..ad88cc1 100644 (file)
@@ -37,7 +37,9 @@ ClassImp(AliPHOSBeamTestEvent)
 
 
 //____________________________________________________________________________ 
-  AliPHOSBeamTestEvent::AliPHOSBeamTestEvent():TObject() 
+AliPHOSBeamTestEvent::AliPHOSBeamTestEvent():
+  fBeamEnergy(0.),
+  fPattern(0)
 {
 //Nothig should be set by default.
 }
index 6ba881a..fe8c83a 100644 (file)
 ClassImp(AliPHOSCPVDigit)
 
 //______________________________________________________________________________
-
-AliPHOSCPVDigit::AliPHOSCPVDigit(Int_t x, Int_t y, Float_t q)
+AliPHOSCPVDigit::AliPHOSCPVDigit():
+  fXpad(0),
+  fYpad(0),
+  fQpad(0.)
 {
   //
   // Create a CPV digit object
   //
+}
 
-  fXpad = x;
-  fYpad = y;
-  fQpad = q;
+
+//______________________________________________________________________________
+AliPHOSCPVDigit::AliPHOSCPVDigit(Int_t x, Int_t y, Float_t q):
+  fXpad(x),
+  fYpad(y),
+  fQpad(q)
+{
+  //
+  // Create a CPV digit object
+  //
 }
 
 //______________________________________________________________________________
index 2986d54..b989212 100644 (file)
@@ -20,7 +20,7 @@ class AliPHOSCPVDigit : public TObject {
   
 public:
   virtual ~AliPHOSCPVDigit() {}
-           AliPHOSCPVDigit() {}
+           AliPHOSCPVDigit();
            AliPHOSCPVDigit(Int_t x, Int_t y, Float_t q);
   
   void     SetQpad(Float_t q) { fQpad = q;     }
index 35271b1..32c09ac 100644 (file)
@@ -37,36 +37,42 @@ ClassImp(AliPHOSCalibrationData)
 
 
 //____________________________________________________________________________ 
-  AliPHOSCalibrationData::AliPHOSCalibrationData():TObject() 
+AliPHOSCalibrationData::AliPHOSCalibrationData():
+  fCategory(), 
+  fVersion(),
+  fData(0),
+  fDataCheck(0),
+  fBegin(0),
+  fEnd(0)
 {
-  // default ctor : does nothing
-  fBegin=0;
-  fEnd=0;
-  fData = 0 ;
-  fDataCheck = 0 ;
-  fCategory=""; 
-  fVersion="" ; 
+  // default ctor
 }
 
 //____________________________________________________________________________ 
-  AliPHOSCalibrationData::AliPHOSCalibrationData(const char * category, const char * version, Int_t nchannels)
+AliPHOSCalibrationData::AliPHOSCalibrationData(const char * category, const char * version, Int_t nchannels):
+  fCategory(category), 
+  fVersion(version),
+  fData(new TArrayF(nchannels)),
+  fDataCheck(new TArrayF(nchannels)),
+  fBegin(0),
+  fEnd(0)
 {
   // ctor: sets up the calibration IO
-  fData      = new TArrayF(nchannels) ;
-  fDataCheck = new TArrayF(nchannels) ;
-  fCategory=category; 
-  fVersion=version ; 
 }
+
 //____________________________________________________________________________ 
-AliPHOSCalibrationData::AliPHOSCalibrationData(const AliPHOSCalibrationData & cd)
-  : TObject(cd)
+AliPHOSCalibrationData::AliPHOSCalibrationData(const AliPHOSCalibrationData & cd):
+  TObject(cd),
+  fCategory(cd.fCategory),
+  fVersion(cd.fVersion),
+  fData(new TArrayF(*cd.fData)),
+  fDataCheck(new TArrayF(*cd.fDataCheck)),
+  fBegin(cd.fBegin),
+  fEnd(cd.fEnd)
 {
   //copy ctor
-  fData = new TArrayF(*cd.fData) ;
-  fDataCheck = new TArrayF(*cd.fDataCheck) ;
-  fCategory=cd.fCategory; 
-  fVersion=cd.fVersion ; 
 }
+
 //____________________________________________________________________________ 
   AliPHOSCalibrationData::~AliPHOSCalibrationData()
 {
index 176b05b..83c1676 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.11  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -46,49 +49,55 @@ ClassImp(AliPHOSConTableDB)
 
 
 //____________________________________________________________________________ 
-  AliPHOSConTableDB::AliPHOSConTableDB():TNamed("AliPHOSConTableDB","Beamtest2002") 
+AliPHOSConTableDB::AliPHOSConTableDB():
+  TNamed("AliPHOSConTableDB","Beamtest2002"),
+  fGeom(0),
+  fProtoRaws(0),
+  fProtoColumns(0),
+  fRawOffset(0),
+  fColOffset(0),
+  fNcrInProto(0),
+  fMinAbsId(0),
+  fMaxAbsId(0),
+  fAbsIdMap(0),
+  fRawIdMap(0)
 {
-//default constructor, nothing created.
-  fNcrInProto = 0 ;
-  fProtoRaws = 0 ;
-  fProtoColumns = 0 ;
-  fRawOffset = 0 ;
-  fColOffset = 0 ;
-  fGeom = 0;
-  fAbsIdMap = 0 ;
-  fRawIdMap = 0 ;
+  //default constructor, nothing created.
 }
 
 //____________________________________________________________________________ 
-  AliPHOSConTableDB::AliPHOSConTableDB(const char * title):TNamed("AliPHOSConTableDB",title) 
+AliPHOSConTableDB::AliPHOSConTableDB(const char * title):
+  TNamed("AliPHOSConTableDB",title),
+  fGeom(0),
+  fProtoRaws(0),
+  fProtoColumns(0),
+  fRawOffset(0),
+  fColOffset(0),
+  fNcrInProto(0),
+  fMinAbsId(0),
+  fMaxAbsId(0),
+  fAbsIdMap(0),
+  fRawIdMap(0)
 {
- //Normally used constructor 
-  fNcrInProto = 0 ;
-  fProtoRaws = 0 ;
-  fProtoColumns = 0 ;
-  fRawOffset = 0 ;
-  fColOffset = 0 ;
-  fAbsIdMap = 0 ;
-  fRawIdMap = 0 ;
-
+  //Normally used constructor 
   fGeom = AliPHOSGeometry::GetInstance("IHEP","") ;
-
 }
 
 //____________________________________________________________________________ 
-AliPHOSConTableDB::AliPHOSConTableDB(const AliPHOSConTableDB& cdb):TNamed(cdb.GetName(), cdb.GetTitle()) 
+AliPHOSConTableDB::AliPHOSConTableDB(const AliPHOSConTableDB& cdb):
+  TNamed(cdb.GetName(), cdb.GetTitle()),
+  fGeom(0),
+  fProtoRaws(cdb.fProtoRaws),
+  fProtoColumns(cdb.fProtoColumns),
+  fRawOffset(cdb.fRawOffset),
+  fColOffset(cdb.fColOffset),
+  fNcrInProto(cdb.fNcrInProto),
+  fMinAbsId(cdb.fMinAbsId),
+  fMaxAbsId(cdb.fMaxAbsId),
+  fAbsIdMap(new TArrayS(*(cdb.fAbsIdMap))),
+  fRawIdMap(new TArrayS(*(cdb.fRawIdMap)))
 {
   //Copy constructor
-  
-  fProtoRaws=cdb.fProtoRaws ;        //  Parameters
-  fProtoColumns=cdb.fProtoColumns ;     //  used to calculate
-  fRawOffset=cdb.fRawOffset ;        //  correspondance
-  fColOffset=cdb.fColOffset ;        //  map
-  fNcrInProto=cdb.fNcrInProto ;       //Number of channels in prototype
-  fMinAbsId=cdb.fMinAbsId ;         //Minimal AbsId, corresponding to some prototype cristall.
-  fMaxAbsId=cdb.fMaxAbsId ;         //Maximal AbsId, corresponding to some prototype cristall
-  fAbsIdMap=new TArrayS(*(cdb.fAbsIdMap)) ;         //Map of correspondance between Raw and PHOS ID
-  fRawIdMap=new TArrayS(*(cdb.fRawIdMap)) ;         //Map of correspondance between AbsId and Raw
 }
 
 //____________________________________________________________________________ 
index 4477d8c..92c8a22 100644 (file)
@@ -91,12 +91,14 @@ AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(Bool_t cpv, AliPHOSEvalRecPoint* parent
   
 }
 
-AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(Int_t i, Bool_t cpv) : fEventFolderName(AliConfig::GetDefaultEventFolderName())
+AliPHOSEvalRecPoint::AliPHOSEvalRecPoint(Int_t i, Bool_t cpv) : 
+  fIsEmc(kFALSE),
+  fIsCpv(kFALSE),
+  fParent(-333),
+  fChi2Dof(-111),
+  fEventFolderName(AliConfig::GetDefaultEventFolderName())
 {
   // ctor
-  fChi2Dof=-111;
-  fParent=-333;
-
   AliPHOSEmcRecPoint* rp=0;
 
   AliPHOSLoader* fLoader = AliPHOSLoader::GetPHOSLoader(fEventFolderName);
index ca40d7f..775da6f 100644 (file)
 ClassImp(AliPHOSFastGlobalReconstruction)
 
 //____________________________________________________________________________
-AliPHOSFastGlobalReconstruction::AliPHOSFastGlobalReconstruction(const char* headerFile )
+AliPHOSFastGlobalReconstruction::AliPHOSFastGlobalReconstruction(): 
+  fgime(0),
+  fGenerator(0),
+  fParticles(0),
+  fNParticles(0)
+{
+  //Def ctor.
+}
+
+//____________________________________________________________________________
+AliPHOSFastGlobalReconstruction::AliPHOSFastGlobalReconstruction(const char* headerFile):
+  fgime(AliPHOSGetter::Instance(headerFile)),
+  fGenerator(gAlice->Generator()),
+  fParticles(new TClonesArray("TParticle",100)),
+  fNParticles(0)
 {
   // Constructor of fast global reconstruction:
   // create an instance of the PHOS getter,
   // create an array or reconstructed particles.
+}
 
-  fgime = AliPHOSGetter::Instance(headerFile);
-  fGenerator = gAlice->Generator();
-  fParticles = new TClonesArray("TParticle",100);
-  fNParticles = 0;
+//____________________________________________________________________________
+AliPHOSFastGlobalReconstruction::AliPHOSFastGlobalReconstruction(const AliPHOSFastGlobalReconstruction & rhs):
+  TObject(rhs),
+  fgime(0),
+  fGenerator(0),
+  fParticles(0),
+  fNParticles(0)
+{
+  //Required by effc++, but not clear for me how to do correct copy. To be fixed.
+  Fatal("AliPHOSFastGlobalReconstruction", "copy ctor not implemented");
+}
+
+//____________________________________________________________________________
+AliPHOSFastGlobalReconstruction & AliPHOSFastGlobalReconstruction::operator = (const AliPHOSFastGlobalReconstruction &)
+{
+  //Required by effc++, but not clear for me how to do correct copy. To be fixed.
+  Fatal("operator = ", "copy ctor not implemented");
+  return *this;
 }
 
 //____________________________________________________________________________
index 0553736..a668bbd 100644 (file)
@@ -21,8 +21,12 @@ class AliPHOSGetter;
 class AliPHOSFastGlobalReconstruction : public TObject {
 
 public:
-  AliPHOSFastGlobalReconstruction() {};
+  AliPHOSFastGlobalReconstruction();
   AliPHOSFastGlobalReconstruction(const char* headerFile);
+  
+  AliPHOSFastGlobalReconstruction(const AliPHOSFastGlobalReconstruction &rhs);
+  AliPHOSFastGlobalReconstruction & operator = (const AliPHOSFastGlobalReconstruction &);
+  
   virtual ~AliPHOSFastGlobalReconstruction();
   void FastReconstruction(Int_t event);
   TClonesArray *GetRecParticles() const {return fParticles;}
index 3408191..0c59223 100644 (file)
@@ -17,6 +17,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.5  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 ClassImp(AliPHOSGridFile)
 
 //____________________________________________________________________________
-AliPHOSGridFile::AliPHOSGridFile(TString grid)
+AliPHOSGridFile::AliPHOSGridFile(TString grid):
+  fGrid(0),
+  fRoot("/alice/production/aliprod"),
+  fYear(""),
+  fProd(""),
+  fVers(""),
+  fType(""),
+  fRun(""),
+  fEvt(""),
+  fPath("")
 {
   // default ctor; Doing initialisation ;
-  fGrid = 0 ;
   if (grid == "alien")
     fGrid = TGrid::Connect("alien://aliendb1.cern.ch:15000/?direct") ;
   else
@@ -49,22 +60,31 @@ AliPHOSGridFile::AliPHOSGridFile(TString grid)
   if ( !fGrid )
     Error("ctor", "Cannot connect to alien://aliendb1.cern.ch:15000/?direct") ;
 
-  fRoot = "/alice/production/aliprod" ;
 #if ROOT_VERSION_CODE < ROOT_VERSION(5,0,0)
   if ( !fGrid->OpenDir(fRoot) )
     Error("ctor", "Cannot find directory %s ", fRoot.Data() ) ;
 #else
   Error("AliPHOSGridFile", "needs to be ported to new TGrid");
 #endif
-  fYear = "" ;
-  fProd = "" ;
-  fVers = "" ;
-  fType = "" ;
-  fRun  = "" ;
-  fEvt  = "" ;
 
   fPath += fRoot ;
+}
 
+//____________________________________________________________________________
+AliPHOSGridFile::AliPHOSGridFile(const AliPHOSGridFile &rhs) :
+  TObject(rhs),
+  fGrid(0),
+  fRoot(""),
+  fYear(""),
+  fProd(""),
+  fVers(""),
+  fType(""),
+  fRun(""),
+  fEvt(""),
+  fPath("")
+{
+  //copy ctor
+  rhs.Copy(*this);
 }
 
 //____________________________________________________________________________
index 260e518..db33857 100644 (file)
@@ -6,6 +6,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.2  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -26,9 +29,8 @@ class AliPHOSGridFile : public TObject {
  public:
   
   AliPHOSGridFile(TString grid="alien") ; 
-  AliPHOSGridFile(AliPHOSGridFile & lfn) : TObject(lfn) {
-    lfn.Copy(*this) ; 
-  } 
+  AliPHOSGridFile(const AliPHOSGridFile & lfn);
+  
   virtual ~AliPHOSGridFile(void) ; 
 
   virtual void Copy(TObject & obj) const;
@@ -72,10 +74,8 @@ private:
   TString fEvt  ; //! event number
   TString fPath ; //! the lfn is fRoot/fYear/fProd/fVers/fType/fRun/fEvt
  
-
   ClassDef(AliPHOSGridFile,1)  
-    
-    };
+};
 
 #endif // AliPHOSGRIDFILE_H
  
index 77fa2c4..b01381e 100644 (file)
 ClassImp(AliPHOSIhepAnalyze)
 
 //____________________________________________________________________________
-
-AliPHOSIhepAnalyze::AliPHOSIhepAnalyze() 
- {
-   fRunLoader = 0x0;
- }
+AliPHOSIhepAnalyze::AliPHOSIhepAnalyze():
+  fRunLoader(0),
+  fFileName()
+{
+}
 
 //____________________________________________________________________________
-
-AliPHOSIhepAnalyze::AliPHOSIhepAnalyze(Text_t * name) : fFileName(name) {
+AliPHOSIhepAnalyze::AliPHOSIhepAnalyze(Text_t * name) : 
+  fRunLoader(0),
+  fFileName(name)
+{
   // Constructor: open a header file
   fRunLoader = AliRunLoader::Open(fFileName);
   if (fRunLoader == 0x0)
    {
      AliFatal(Form("Can not load event from file %s",name));
    }
- }
+}
 
 //____________________________________________________________________________
 void AliPHOSIhepAnalyze::AnalyzeCPV1(Int_t Nevents)
index d7e6dc4..25579c2 100644 (file)
@@ -25,7 +25,8 @@ class AliPHOSIhepAnalyze : public TObject {
 
   AliPHOSIhepAnalyze() ;              // ctor
   AliPHOSIhepAnalyze(Text_t * name) ; // ctor
-  AliPHOSIhepAnalyze(const AliPHOSIhepAnalyze & obj) : TObject(obj) {
+  AliPHOSIhepAnalyze(const AliPHOSIhepAnalyze & obj) : TObject(obj), fRunLoader(0), fFileName()
+  {
     // cpy ctor: no implementation yet
     // requested by the Coding Convention
     Fatal("cpy ctor", "not implemented") ;
index e8a7d60..562da4e 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.6  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 ClassImp(AliPHOSImpact)
 
 //____________________________________________________________________________
-AliPHOSImpact::AliPHOSImpact(const AliPHOSImpact & hit) 
-  : AliHit(hit)
+AliPHOSImpact::AliPHOSImpact() : 
+  fPid(0),
+  fPrimary(0),
+  fMomentum()
+{
+  //Def ctor.
+}
+
+//____________________________________________________________________________
+AliPHOSImpact::AliPHOSImpact(const AliPHOSImpact & hit) : 
+  AliHit(hit),
+  fPid(hit.fPid),
+  fPrimary(hit.fPrimary),
+  fMomentum(hit.fMomentum)
 {
-   // copy ctor
-   
+  // copy ctor
   fTrack   = hit.fTrack ; 
-  fPrimary = hit.fPrimary ; 
-  fPid     = hit.fPid ;
-  fMomentum= hit.fMomentum ;
   fX       = hit.fX ; 
   fY       = hit.fY ; 
   fZ       = hit.fZ ; 
 } 
 
 //____________________________________________________________________________
-AliPHOSImpact::AliPHOSImpact(Int_t shunt, Int_t primary, Int_t track, Int_t pid, TLorentzVector p, Float_t *xyz)
-  : AliHit(shunt, track)
+AliPHOSImpact::AliPHOSImpact(Int_t shunt, Int_t primary, Int_t track, Int_t pid, TLorentzVector p, Float_t *xyz): 
+  AliHit(shunt, track),
+  fPid(pid),
+  fPrimary(primary),
+  fMomentum(p)
 {
   //
   // Create a PHOS hit impact object
   //
-
-  fPrimary   = primary ;
-  fPid       = pid ; 
-  fMomentum  = p;
   fX         = xyz[0];  //position of particle first entering cristall/pad
   fY         = xyz[1];
   fZ         = xyz[2];  
index d3b025b..9d39cab 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.2  2005/05/28 14:19:04  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -35,7 +38,7 @@ class AliPHOSImpact : public AliHit {
 
 public:
   
-  AliPHOSImpact() { }              // default ctor 
+  AliPHOSImpact();              // default ctor 
   AliPHOSImpact(const AliPHOSImpact & hit) ; 
   AliPHOSImpact(Int_t shunt, Int_t primary, Int_t track, Int_t pid, TLorentzVector p, Float_t *xyz);
   virtual ~AliPHOSImpact(void) { } // dtor 
index d0cb92f..63a431a 100644 (file)
@@ -37,14 +37,13 @@ ClassImp(AliPHOSLink)
 }
 
 //____________________________________________________________________________
-  AliPHOSLink::AliPHOSLink(Float_t r, Int_t emc, Int_t cpv, Int_t track)
+AliPHOSLink::AliPHOSLink(Float_t r, Int_t emc, Int_t cpv, Int_t track):
+  fEmcN(emc),
+  fCpvN(cpv),
+  fR(r),
+  fTrack(track)
 {
   // ctor
-
-  fR     = r ;  
-  fEmcN  = emc ;
-  fCpvN  = cpv ;   
-  fTrack = track ; 
 }
 
 //____________________________________________________________________________
index 1d87d19..23e0305 100644 (file)
@@ -70,9 +70,19 @@ ClassImp(AliPHOSOnlineMonitor)
 
 
 //____________________________________________________________________________ 
-AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(): TDialogCanvas("PHOS","PHOS",150,300)
+AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(): 
+  TDialogCanvas("PHOS","PHOS",150,300),
+  fScanPed(kFALSE),
+  fScanSig(kFALSE),
+  fReconstruct(kFALSE),
+  fNevents(0),
+  fNUpdate(1000),
+  fCanvasList(new TList),
+  fHistosList(new TList),
+  fInputFile(),
+  fGeom(0),
+  fcdb(0)
 {
-  
   MakeButtons() ;
   Modified(kTRUE);
   Update();
@@ -85,23 +95,22 @@ AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(): TDialogCanvas("PHOS","PHOS",150,30
   fRefObject = this;
   fRefPad    = (TPad*)gROOT->GetSelectedPad();
 
-  fScanPed = 0 ;
-  fScanSig = 0; 
-  fReconstruct = 0 ;
-
-  //Prepare list of canvas with histograms
-  fCanvasList = new TList() ;
-  fHistosList = new TList() ;
-
-  fcdb = 0 ;
   fGeom = AliPHOSGeometry::GetInstance("IHEP","") ;
-  fNUpdate = 1000 ;
-
 }
 //____________________________________________________________________________ 
-AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(const char * inputfile): TDialogCanvas("PHOS","PHOS",150,300)
+AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(const char * inputfile): 
+  TDialogCanvas("PHOS","PHOS",150,300),
+  fScanPed(kFALSE),
+  fScanSig(kFALSE),
+  fReconstruct(kFALSE),
+  fNevents(0),
+  fNUpdate(1000),
+  fCanvasList(new TList),
+  fHistosList(new TList),
+  fInputFile(inputfile),
+  fGeom(0),
+  fcdb(0)
 {
-
   MakeButtons() ;
   Modified(kTRUE);
   Update();
@@ -114,22 +123,35 @@ AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(const char * inputfile): TDialogCanva
   fRefObject = this;
   fRefPad    = (TPad*)gROOT->GetSelectedPad();
 
-  fScanPed = 0 ;
-  fScanSig = 0; 
-  fReconstruct = 0 ;
-
-  //Prepare list of canvas with histograms
-  fCanvasList = new TList() ;
-  fHistosList = new TList() ;
-
-  fInputFile = inputfile ;
   fGeom = AliPHOSGeometry::GetInstance("IHEP","") ;
-  fcdb = 0 ;
-  fNUpdate = 1000 ;
+}
 
+//____________________________________________________________________________ 
+AliPHOSOnlineMonitor::AliPHOSOnlineMonitor(const AliPHOSOnlineMonitor & /*rhs*/):
+  TDialogCanvas(),
+  fScanPed(kFALSE),
+  fScanSig(kFALSE),
+  fReconstruct(kFALSE),
+  fNevents(0),
+  fNUpdate(0),
+  fCanvasList(0),
+  fHistosList(0),
+  fInputFile(),
+  fGeom(0),
+  fcdb(0)
+{
+  Fatal("AliPHOSOnlineMonitor", "not implemented");
 }
+
+//____________________________________________________________________________ 
+AliPHOSOnlineMonitor & AliPHOSOnlineMonitor::operator = (const AliPHOSOnlineMonitor &)
+{
+  Fatal("operator = ", "not implemented");
+  return *this;
+}
+
 //____________________________________________________________________________ 
-  AliPHOSOnlineMonitor::~AliPHOSOnlineMonitor()
+AliPHOSOnlineMonitor::~AliPHOSOnlineMonitor()
 {
   //Obvious, but unevoidable comment for destructor: cleans up everething.
   TIter nextCanvas(fCanvasList);
index f76fc95..e757d33 100644 (file)
@@ -55,6 +55,11 @@ public:
   void ScanRecon(TClonesArray * recParticles) ;
 
 private:
+  //They call Fatal, but they are private, user will have compile time error instead
+  //of run-time error. Fatal - because it's not clear, should I copy canvases, 
+  //hists etc.
+  AliPHOSOnlineMonitor(const AliPHOSOnlineMonitor &);
+  AliPHOSOnlineMonitor & operator = (const AliPHOSOnlineMonitor &);
 
   Bool_t fScanPed ;     //should we analyse pedestal events
   Bool_t fScanSig;      //should we analyse signal events
index 2187421..08d5553 100644 (file)
 #include "AliLog.h"
 
 //____________________________________________________________________________
-  AliPHOSRecParticle::AliPHOSRecParticle(): fPHOSTrackSegment(0)  ,  fDebug( kFALSE )
+AliPHOSRecParticle::AliPHOSRecParticle(): 
+  fPHOSTrackSegment(0),
+  fDebug(kFALSE),
+  fPos()
 {
   // ctor
   const Int_t nSPECIES = AliPID::kSPECIESN;
 
 
 //____________________________________________________________________________
- AliPHOSRecParticle::AliPHOSRecParticle(const AliPHOSRecParticle & rp)
-   : AliPHOSFastRecParticle(rp)
+AliPHOSRecParticle::AliPHOSRecParticle(const AliPHOSRecParticle & rp):
+  AliPHOSFastRecParticle(rp),
+  fPHOSTrackSegment(rp.fPHOSTrackSegment),
+  fDebug(kFALSE),
+  fPos()
 {
   // copy ctor
-
-  fPHOSTrackSegment = rp.fPHOSTrackSegment ; 
-  fDebug            = kFALSE ; 
   fType             = rp.fType ; 
   fIndexInList      = rp.fIndexInList ;
 
index fe36a2c..ad87bb2 100644 (file)
@@ -33,20 +33,21 @@ ClassImp(AliPHOSTick)
 
 
 //____________________________________________________________________________ 
-  AliPHOSTick::AliPHOSTick():TObject() 
+AliPHOSTick::AliPHOSTick():
+  fTime(0),
+  fA(0),
+  fB(0)
 {
-  fTime = 0;
-  fA = 0;
-  fB = 0 ;
 }
 
 //____________________________________________________________________________ 
-AliPHOSTick::AliPHOSTick(Float_t time, Float_t a, Float_t slope):TObject()
+AliPHOSTick::AliPHOSTick(Float_t time, Float_t a, Float_t slope):
+  fTime(time),
+  fA(a),
+  fB(slope)
 {
-  fTime = time;
-  fA = a;
-  fB = slope ;  
 }
+
 //____________________________________________________________________________ 
 Int_t AliPHOSTick::Compare(const TObject * obj) const {
   if(obj->InheritsFrom("AliPHOSTick")){
index a1ab184..c081b27 100644 (file)
@@ -18,9 +18,17 @@ class AliRunLoader;
 class AliPHOSTracker : public AliTracker
 {
 public:
-  AliPHOSTracker():AliTracker()   {fRunLoader=0;}
-  AliPHOSTracker(AliRunLoader *loader):AliTracker() {fRunLoader=loader;}
+  AliPHOSTracker():AliTracker(), fRunLoader(0) {}
+  AliPHOSTracker(AliRunLoader *loader):AliTracker(), fRunLoader(loader) {}
+  AliPHOSTracker(const AliPHOSTracker & rhs):AliTracker(rhs), fRunLoader(rhs.fRunLoader){}
+  
   virtual ~AliPHOSTracker()       {AliDebug(1,"Start.");}
+  
+  AliPHOSTracker & operator = (const AliPHOSTracker & rhs)
+  {
+    fRunLoader = rhs.fRunLoader;
+    return *this;
+  }
 
   Int_t Clusters2Tracks(AliESD *) {AliDebug(1,"Start.");return 0;}
   Int_t RefitInward(AliESD *)     {AliDebug(1,"Start.");return 0;}
index d4f8de8..6d221a3 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.104  2005/05/28 14:19:05  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -50,23 +53,29 @@ ClassImp(AliPHOSv1)
 
 //____________________________________________________________________________
 AliPHOSv1::AliPHOSv1():
-AliPHOSv0()
+  fLightYieldMean(0.),
+  fIntrinsicPINEfficiency(0.),
+  fLightYieldAttenuation(0.),
+  fRecalibrationFactor(0.),
+  fElectronsPerGeV(0.),
+  fAPDGain(0.),
+  fLightFactor(0.),
+  fAPDFactor(0.)
 {
-
-  fLightYieldMean         = 0. ;         
-  fIntrinsicPINEfficiency = 0. ; 
-  fLightYieldAttenuation  = 0. ;  
-  fRecalibrationFactor    = 0. ;    
-  fElectronsPerGeV        = 0. ;
-  fAPDGain                = 0. ;  
-  fLightFactor            = 0. ; 
-  fAPDFactor              = 0. ; 
-
+  //Def ctor.
 }
 
 //____________________________________________________________________________
 AliPHOSv1::AliPHOSv1(const char *name, const char *title):
- AliPHOSv0(name,title) 
+  AliPHOSv0(name,title),
+  fLightYieldMean(0.),
+  fIntrinsicPINEfficiency(0.),
+  fLightYieldAttenuation(0.),
+  fRecalibrationFactor(0.),
+  fElectronsPerGeV(0.),
+  fAPDGain(0.),
+  fLightFactor(0.),
+  fAPDFactor(0.)
 {
   //
   // We store hits :
@@ -110,6 +119,22 @@ AliPHOSv1::AliPHOSv1(const char *name, const char *title):
   fAPDFactor              = (fRecalibrationFactor/100.) * fAPDGain ;   
 }
 
+AliPHOSv1::AliPHOSv1(AliPHOSv1 & phos) :
+  AliPHOSv0(phos),
+  fLightYieldMean(0.),
+  fIntrinsicPINEfficiency(0.),
+  fLightYieldAttenuation(0.),
+  fRecalibrationFactor(0.),
+  fElectronsPerGeV(0.),
+  fAPDGain(0.),
+  fLightFactor(0.),
+  fAPDFactor(0.)
+{
+  //Copy ctor. Can be wrong.
+  phos.Copy(*this) ; 
+}
+
+
 //____________________________________________________________________________
 AliPHOSv1::~AliPHOSv1()
 {
index 3f07a56..2187d28 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.39  2005/05/28 14:19:05  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -30,9 +33,7 @@ public:
 
   AliPHOSv1(void) ;
   AliPHOSv1(const char *name, const char *title="") ;
-  AliPHOSv1(AliPHOSv1 & phos) : AliPHOSv0(phos) {
-    phos.Copy(*this) ; 
-  }
+  AliPHOSv1(AliPHOSv1 & phos);
   virtual ~AliPHOSv1(void) ;
 
   virtual void   Copy(TObject &phos) const; 
index f511821..08c078b 100644 (file)
@@ -18,6 +18,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.29  2005/05/28 14:19:05  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
 
 ClassImp(AliPHOSvFast)
 
-//____________________________________________________________________________
-AliPHOSvFast::AliPHOSvFast() : AliPHOS()
+AliPHOSvFast::AliPHOSvFast() :
+  fBigBoxX(0.),
+  fBigBoxY(0.),
+  fBigBoxZ(0.),
+  fFastRecParticles(0),
+  fNRecParticles(0),
+  fRan(0),
+  fResPara1(0.),
+  fResPara2(0.),
+  fResPara3(0.),
+  fPosParaA0(0.),
+  fPosParaA1(0.),
+  fPosParaB0(0.),
+  fPosParaB1(0.),
+  fPosParaB2(0.)    
 {
   // default ctor : initialize data member
-   fBigBoxX = 0. ;                      
-   fBigBoxY = 0. ;                      
-   fBigBoxZ = 0. ;                       
-   fFastRecParticles = 0 ;        
-   fNRecParticles = 0 ;                
-   fRan = 0 ;                            
-   fResPara1 = 0. ;                       
-   fResPara2 = 0. ;                        
-   fResPara3 = 0. ;                      
-   fPosParaA0 = 0. ;                      
-   fPosParaA1 = 0. ;
-   fPosParaB0 = 0. ;     
-   fPosParaB1 = 0. ;    
-   fPosParaB2 = 0. ;    
-
 }
 
 //____________________________________________________________________________
 AliPHOSvFast::AliPHOSvFast(const char *name, const char *title):
-  AliPHOS(name,title)
+  AliPHOS(name,title),
+  fBigBoxX(0.),
+  fBigBoxY(0.),
+  fBigBoxZ(0.),
+  fFastRecParticles(new AliPHOSFastRecParticle::FastRecParticlesList("AliPHOSFastRecParticle", 100)),
+  fNRecParticles(0),
+  fRan(0),
+  fResPara1(0.030), // GeV
+  fResPara2(0.00003),
+  fResPara3(0.00001),
+  fPosParaA0(2.87),    // mm
+  fPosParaA1(-0.0975),
+  fPosParaB0(0.257),
+  fPosParaB1(0.137),
+  fPosParaB2(0.00619)
 {
   // ctor
-
-  
   // create the Loader 
-  
   SetBigBox(0, GetGeometry()->GetOuterBoxSize(0) ) ;
   SetBigBox(1, GetGeometry()->GetOuterBoxSize(3) + GetGeometry()->GetCPVBoxSize(1) ) ; 
   SetBigBox(2, GetGeometry()->GetOuterBoxSize(2) ); 
-  
-  fNRecParticles = 0 ; 
-  fFastRecParticles = new AliPHOSFastRecParticle::FastRecParticlesList("AliPHOSFastRecParticle", 100) ;
-  
-  fResPara1 = 0.030 ;    // GeV
-  fResPara2 = 0.00003 ; 
-  fResPara3 = 0.00001 ; 
-  
-  fPosParaA0 = 2.87 ;    // mm
-  fPosParaA1 = -0.0975 ;  
-  fPosParaB0 = 0.257 ;   
-  fPosParaB1 = 0.137 ; 
-  fPosParaB2 = 0.00619 ; 
 }
 
 //____________________________________________________________________________
+AliPHOSvFast::AliPHOSvFast(AliPHOSvFast & fast) :
+  AliPHOS(fast),
+  fBigBoxX(0.),
+  fBigBoxY(0.),
+  fBigBoxZ(0.),
+  fFastRecParticles(0),
+  fNRecParticles(0),
+  fRan(0),
+  fResPara1(0.),
+  fResPara2(0.),
+  fResPara3(0.),
+  fPosParaA0(0.),
+  fPosParaA1(0.),
+  fPosParaB0(0.),
+  fPosParaB1(0.),
+  fPosParaB2(0.)    
+{
+  //Copy ctor.
+  fast.Copy(*this) ; 
+}
+
+
+//____________________________________________________________________________
 AliPHOSvFast::~AliPHOSvFast()
 {
   // dtor
index 9b29db1..bae38b1 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.25  2005/05/28 14:19:05  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -37,9 +40,8 @@ public:
 
   AliPHOSvFast() ;
   AliPHOSvFast(const char *name, const char *title="") ;
-  AliPHOSvFast(AliPHOSvFast & fast) : AliPHOS(fast) {
-    fast.Copy(*this) ; 
-  }
+  AliPHOSvFast(AliPHOSvFast & fast);
+  
   virtual ~AliPHOSvFast(void) ;
 
   virtual void Copy(TObject &fast) const; 
index ffa1d92..5532662 100644 (file)
@@ -19,6 +19,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.22  2005/06/17 07:39:07  hristov
+ * Removing GetDebug and SetDebug from AliRun and AliModule. Using AliLog for the messages
+ *
  * Revision 1.21  2005/05/28 14:19:05  schutz
  * Compilation warnings fixed by T.P.
  *
 ClassImp(AliPHOSvImpacts)
 
 //____________________________________________________________________________
-AliPHOSvImpacts::AliPHOSvImpacts():AliPHOSv1()
+AliPHOSvImpacts::AliPHOSvImpacts():
+  fEMCImpacts(0),
+  fCPVImpacts(0),
+  fPPSDImpacts(0)
 {
   // ctor
-  fEMCImpacts  = 0 ;
-  fPPSDImpacts = 0 ;
-  fCPVImpacts  = 0 ; 
 }
 
 //____________________________________________________________________________
 AliPHOSvImpacts::AliPHOSvImpacts(const char *name, const char *title):
-AliPHOSv1(name,title) 
+  AliPHOSv1(name,title),
+  fEMCImpacts(new TList),
+  fCPVImpacts(new TList),
+  fPPSDImpacts(0)
 {
   // ctor : title is used to identify the layout
   //
@@ -78,9 +84,6 @@ AliPHOSv1(name,title)
   //  - fEMCImpacts, fCPVImpacts which are
   //    TList of EMC and CPV modules respectively, each
   //    modules contains TClonesArray of AliPHOSImpacts
-  
-  fEMCImpacts  = new TList();
-  fCPVImpacts  = new TList();
 
   Int_t nPHOSModules = GetGeometry()->GetNModules();
   Int_t nCPVModules  = GetGeometry()->GetNModules();
@@ -101,6 +104,16 @@ AliPHOSv1(name,title)
 }
 
 //____________________________________________________________________________
+AliPHOSvImpacts::AliPHOSvImpacts(AliPHOSvImpacts & phos):
+  AliPHOSv1(phos),
+  fEMCImpacts(0),
+  fCPVImpacts(0),
+  fPPSDImpacts(0)
+{
+  phos.Copy(*this);
+}
+
+//____________________________________________________________________________
 AliPHOSvImpacts::~AliPHOSvImpacts()
 {
   // dtor
index cf8c57b..e223fba 100644 (file)
@@ -8,6 +8,9 @@
 /* History of cvs commits:
  *
  * $Log$
+ * Revision 1.8  2005/05/28 14:19:05  schutz
+ * Compilation warnings fixed by T.P.
+ *
  */
 
 //_________________________________________________________________________
@@ -34,9 +37,7 @@ public:
 
   AliPHOSvImpacts(void) ;
   AliPHOSvImpacts(const char *name, const char *title="") ;
-  AliPHOSvImpacts(AliPHOSvImpacts & phos) : AliPHOSv1(phos) {
-    phos.Copy(*this) ; 
-  }
+  AliPHOSvImpacts(AliPHOSvImpacts & phos);
   virtual ~AliPHOSvImpacts(void) ;
 
   virtual void   Copy(TObject & phos) const;