changes to be compliant with Eff C++ rules

diff --git a/ITS/AliITSPidParItem.cxx b/ITS/AliITSPidParItem.cxx
index 015415527f9a1bf6c9bf62a7289c8de29fda1a07..3b41c9706e9a01b7600b1b5e457c390497e45e28 100644 (file)
--- a/ITS/AliITSPidParItem.cxx
+++ b/ITS/AliITSPidParItem.cxx
@@ -10,18 +10,20 @@
 
 ClassImp(AliITSPidParItem)
 //____________________________________________________________________
-AliITSPidParItem::AliITSPidParItem(){
+AliITSPidParItem::AliITSPidParItem():
+fPCenter(0),
+fPWidth(0)
+{
   // default constructor
-  fPCenter=0;
-  fPWidth=0;
   for(Int_t i=0;i<39;i++){
   fBuff[i]=0;
   }
 }//____________________________________________________________________
-AliITSPidParItem::AliITSPidParItem(Float_t center,Float_t width,Double_t *buff){
+AliITSPidParItem::AliITSPidParItem(Float_t center,Float_t width,Double_t *buff):
+fPCenter(center),
+fPWidth(width)
+{
   // standard constructor
-  fPCenter=center;
-  fPWidth=width;
   for (Int_t i=0;i<39;i++) fBuff[i]=buff[i];
 
 }

diff --git a/ITS/AliITSPident.cxx b/ITS/AliITSPident.cxx
index 3b51d0985c16efd21df9995750958f74c43548f1..fcdc421b7539a378661c218c6f98fb995957b5e7 100644 (file)
--- a/ITS/AliITSPident.cxx
+++ b/ITS/AliITSPident.cxx
@@ -9,18 +9,19 @@
 
 ClassImp(AliITSPident)
   //_______________________________________________________________________
-AliITSPident::AliITSPident(){
+AliITSPident::AliITSPident():
+fMom(0),
+fdEdx(0),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(0),
+fPPriorik(0),
+fPPrioripi(0),
+fPPriorie(0),
+fInvPt(0)
+{
   // default constructor
-  fMom=0;
-  fdEdx=0;
-  fInvPt=0;
-  fPBayesp=0;
-  fPBayesk=0;
-  fPBayespi=0;
-  fPPriorip=0;
-  fPPriorik=0;
-  fPPrioripi=0;
-  fPPriorie=0;
   for (Int_t i=0;i<4;i++){
     fCondFunProLay[i]=0;
     fCondFunKLay[i]=0;
@@ -32,28 +33,45 @@ AliITSPident::~AliITSPident(){
   // destructor
 }
 //______________________________________________________________________
-AliITSPident::AliITSPident(const AliITSPident &ob) :TObject(ob) {
+AliITSPident::AliITSPident(const AliITSPident &ob) :TObject(ob),
+fMom(ob.fMom),
+fdEdx(ob.fdEdx),
+fPBayesp(ob.fPBayesp),
+fPBayesk(ob.fPBayesk),
+fPBayespi(ob.fPBayespi),
+fPPriorip(ob.fPPriorip),
+fPPriorik(ob.fPPriorik),
+fPPrioripi(ob.fPPrioripi),
+fPPriorie(ob.fPPriorie),
+fInvPt(ob.fInvPt)  
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSPident","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSPident& AliITSPident::operator=(const AliITSPident& /* ob */){
+AliITSPident& AliITSPident::operator=(const AliITSPident&  ob){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSPident();
+  new(this) AliITSPident(ob);
   return *this;
 }
 
 
 //_______________________________________________________________________
-AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie){
+AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie):
+fMom(mom),
+fdEdx(dEdx),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(priorip),
+fPPriorik(priorik),
+fPPrioripi(prioripi),
+fPPriorie(priorie),
+fInvPt(invPt){
 
   //test
 
-  fMom=mom;
-  fInvPt=invPt;
   for(Int_t la=0;la<4;la++){//loop on layers
     Double_t parp[3];Double_t park[3];Double_t parpi[3];
     sp->GetParFitLayer(la,fMom,parp,park,parpi);
@@ -72,11 +90,6 @@ AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteer
       CookFunItsLay(la,2,parpi,Qlay[la],fMom,range[4],range[5],"fPi");
     
   }
-  fPPriorip=priorip;
-  fPPriorik=priorik;
-  fPPrioripi=prioripi;
-  fPPriorie=priorie;
-
 
   Float_t prior[4];Double_t condFun[4][3];
 
@@ -94,12 +107,22 @@ AliITSPident::AliITSPident(Double_t mom,Double_t invPt,Double_t dEdx,AliITSSteer
   fPBayesp=CookCombinedBayes(condFun,prior,0);
   fPBayesk=CookCombinedBayes(condFun,prior,1); 
   fPBayespi=CookCombinedBayes(condFun,prior,2); 
-  fdEdx=dEdx;
-}
+ }
 
 
 //_______________________________________________________________________
-AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie){
+AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Qlay,Float_t priorip,Float_t priorik,Float_t prioripi,Float_t priorie):
+fMom(0),
+fdEdx(0),
+fPBayesp(0),
+fPBayesk(0),
+fPBayespi(0),
+fPPriorip(priorip),
+fPPriorik(priorik),
+fPPrioripi(prioripi),
+fPPriorie(priorie),
+fInvPt(0)
+{
   //
   Double_t xr;
   Double_t par[5];
@@ -127,11 +150,6 @@ AliITSPident::AliITSPident(AliITStrackV2 *trackITS,AliITSSteerPid *sp,Float_t *Q
       CookFunItsLay(la,1,park,Qlay[la],fMom,range[2],range[3],"fKao");
       CookFunItsLay(la,2,parpi,Qlay[la],fMom,range[4],range[5],"fPi");
   }
-  fPPriorip=priorip;
-  fPPriorik=priorik;
-  fPPrioripi=prioripi;
-  fPPriorie=priorie;
-
 
   Float_t prior[4];Double_t condFun[4][3];
 

diff --git a/ITS/AliITSPident.h b/ITS/AliITSPident.h
index a122728354535bc028026f9407936ca0ab9ac5d0..123b74491473806d5b93aa78ed75d8b6a72f15b9 100644 (file)
--- a/ITS/AliITSPident.h
+++ b/ITS/AliITSPident.h
@@ -70,7 +70,7 @@ class AliITSPident : public TObject{
   Float_t CookProd(Double_t condfun[][3],Int_t part) const;
   Float_t CookSum(Double_t condfun[][3],Float_t *prior) const;
   AliITSPident(const AliITSPident &ob); // copy constructor
-  AliITSPident& operator=(const AliITSPident & /* source */); // ass. op.
+  AliITSPident& operator=(const AliITSPident & ob); // ass. op.
 
   Float_t fMom;                   // Particle momentum
   Double_t fdEdx;                 // Particle dE/dx

diff --git a/ITS/AliITSReconstructor.cxx b/ITS/AliITSReconstructor.cxx
index dabc5c1056920f8f2f78b2c6cadf9c1711c68ada..fbb7288670bf9e86ca4348423d70990bae2046ef 100644 (file)
--- a/ITS/AliITSReconstructor.cxx
+++ b/ITS/AliITSReconstructor.cxx
@@ -45,9 +45,10 @@
 ClassImp(AliITSReconstructor)
 
 //___________________________________________________________________________
-AliITSReconstructor::AliITSReconstructor() : AliReconstructor(){
+AliITSReconstructor::AliITSReconstructor() : AliReconstructor(),
+fItsPID(0)
+{
   // Default constructor
-  fItsPID=0;
 }
  //___________________________________________________________________________
 AliITSReconstructor::~AliITSReconstructor(){
@@ -55,21 +56,21 @@ AliITSReconstructor::~AliITSReconstructor(){
   delete fItsPID;
 } 
 //______________________________________________________________________
-AliITSReconstructor::AliITSReconstructor(const AliITSReconstructor &ob) :AliReconstructor(ob) {
+AliITSReconstructor::AliITSReconstructor(const AliITSReconstructor &ob) :AliReconstructor(ob),
+fItsPID(ob.fItsPID) 
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSpidESD2","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSReconstructor& AliITSReconstructor::operator=(const AliITSReconstructor& /* ob */){
+AliITSReconstructor& AliITSReconstructor::operator=(const AliITSReconstructor&  ob ){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSReconstructor();
+  new(this) AliITSReconstructor(ob);
   return *this;
 }
 //______________________________________________________________________
-void AliITSReconstructor::Init(AliRunLoader *runLoader){
+void AliITSReconstructor::Init(AliRunLoader *runLoader) const{
     // Initalize this constructor bet getting/creating the objects
     // nesseary for a proper ITS reconstruction.
     // Inputs:

diff --git a/ITS/AliITSReconstructor.h b/ITS/AliITSReconstructor.h
index 0f818367df552c941868f4bfa02c61754fd13e54..814f74aa96b144f9a36e190d2c452737f6fd08e8 100644 (file)
--- a/ITS/AliITSReconstructor.h
+++ b/ITS/AliITSReconstructor.h
@@ -23,8 +23,8 @@ public:
   AliITSReconstructor();
   virtual ~AliITSReconstructor();
   AliITSReconstructor(const AliITSReconstructor &ob); // copy constructor
-  AliITSReconstructor& operator=(const AliITSReconstructor & /* source */); // ass. op.
-  virtual void Init(AliRunLoader* runLoader);
+  AliITSReconstructor& operator=(const AliITSReconstructor & ob); // ass. op.
+  virtual void Init(AliRunLoader* runLoader) const;
   
   virtual void         Reconstruct(AliRunLoader* runLoader) const;
   virtual void         Reconstruct(AliRunLoader* runLoader,

diff --git a/ITS/AliITSSteerPid.cxx b/ITS/AliITSSteerPid.cxx
index 64e7a428aa58f10f7f07d286bd6a6c82f58da1b1..0aee419ed9fd264710b991e7e71a8dddf74af420 100644 (file)
--- a/ITS/AliITSSteerPid.cxx
+++ b/ITS/AliITSSteerPid.cxx
@@ -11,13 +11,20 @@
 ClassImp(AliITSSteerPid)
 
   //______________________________________________________________
-  AliITSSteerPid::AliITSSteerPid(){
+AliITSSteerPid::AliITSSteerPid():
+fClonarr2(0),
+fVect2(0),
+fVect2lay1(0),
+fVect2lay2(0),
+fVect2lay3(0),
+fVect2lay4(0),
+fFitTree(0),
+fItem(0),
+fPCenter(0),
+fPWidth(0)
+{
   // default constructor
-  fClonarr2=0;
-  fVect2=0;
-  fItem=0;
-  fFitTree=0;
-}
+ }
 //______________________________________________________________
 AliITSSteerPid::~AliITSSteerPid(){
   // destructor
@@ -26,17 +33,25 @@ AliITSSteerPid::~AliITSSteerPid(){
 }
 
 //______________________________________________________________________
-AliITSSteerPid::AliITSSteerPid(const AliITSSteerPid &ob) :TObject(ob) {
+AliITSSteerPid::AliITSSteerPid(const AliITSSteerPid &ob) :TObject(ob),
+fClonarr2(ob.fClonarr2),
+fVect2(ob.fVect2),
+fVect2lay1(ob.fVect2lay1),
+fVect2lay2(ob.fVect2lay2),
+fVect2lay3(ob.fVect2lay3),
+fVect2lay4(ob.fVect2lay4),
+fFitTree(ob.fFitTree),
+fItem(ob.fItem),
+fPCenter(ob.fPCenter),
+fPWidth(ob.fPWidth) {
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSSteerPid","Copy constructor not allowed\n");
 }
 
 //______________________________________________________________________
-AliITSSteerPid& AliITSSteerPid::operator=(const AliITSSteerPid& /* ob */){
+AliITSSteerPid& AliITSSteerPid::operator=(const AliITSSteerPid& ob){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSSteerPid();
+  new(this) AliITSSteerPid(ob);
   return *this;
 }
 

diff --git a/ITS/AliITSSteerPid.h b/ITS/AliITSSteerPid.h
index c4b4fa42d79f7c3245babd0fa857e6b95ef56e16..075d61ecb51d96302ee91e90f5a219e775eb20e9 100644 (file)
--- a/ITS/AliITSSteerPid.h
+++ b/ITS/AliITSSteerPid.h
@@ -34,7 +34,7 @@ class AliITSSteerPid : public TObject{
 
   AliITSPidParItem* Item(TClonesArray *Vect,Float_t mom);
   AliITSSteerPid(const AliITSSteerPid &ob); // copy constructor
-  AliITSSteerPid& operator=(const AliITSSteerPid & /* source */); // ass. op.
+  AliITSSteerPid& operator=(const AliITSSteerPid & ob); // ass. op.
 
 
   TFile *fClonarr2;              // file of TClones Array

diff --git a/ITS/AliITSpidESD1.cxx b/ITS/AliITSpidESD1.cxx
index 7a3cb0d2ecbf5553a956d7f516ad7940dd16d5e6..a9228bf97367759fd225b1c76f9946146fcdc390 100755 (executable)
--- a/ITS/AliITSpidESD1.cxx
+++ b/ITS/AliITSpidESD1.cxx
@@ -26,21 +26,22 @@
 
 ClassImp(AliITSpidESD1)
 
-AliITSpidESD1::AliITSpidESD1(): AliITSpidESD() {
+AliITSpidESD1::AliITSpidESD1(): AliITSpidESD(),
+fMIP(0),
+fRes(0),
+fRange(0) 
+{
   //Default constructor
-  fMIP=0;
-  fRes=0;
-  fRange=0;
 }
 //_________________________________________________________________________
-AliITSpidESD1::AliITSpidESD1(Double_t *param): AliITSpidESD()
+AliITSpidESD1::AliITSpidESD1(Double_t *param): AliITSpidESD(),
+fMIP(param[0]),
+fRes(param[1]),
+fRange(param[2])
 {
   //
   //  The main constructor
   //
-  fMIP=param[0];
-  fRes=param[1];
-  fRange=param[2];
 }
 
 

diff --git a/ITS/AliITSpidESD2.cxx b/ITS/AliITSpidESD2.cxx
index 1236f40191f60c016f740483ae0433bfce06e547..b21a084ef99e448684237b664f4be0a5ae817cd1 100644 (file)
--- a/ITS/AliITSpidESD2.cxx
+++ b/ITS/AliITSpidESD2.cxx
@@ -36,15 +36,18 @@
 
 ClassImp(AliITSpidESD2)
 //_________________________________________________________________________
-AliITSpidESD2::AliITSpidESD2():AliITSpidESD()
+  AliITSpidESD2::AliITSpidESD2():AliITSpidESD(),
+fTracker(0),
+fLoader(0),
+fSp(0)
 { //
   //  The main constructor
-  fTracker=0;
-  fLoader=0;
-  fSp=0;
 }
 //_________________________________________________________________________
-AliITSpidESD2::AliITSpidESD2(AliITStrackerMI* tracker,AliITSLoader* loader):AliITSpidESD()
+AliITSpidESD2::AliITSpidESD2(AliITStrackerMI* tracker,AliITSLoader* loader):AliITSpidESD(),
+fTracker(0),
+fLoader(0),
+fSp(0)
 { //
   //  The main constructor
   fTracker=tracker;
@@ -60,17 +63,20 @@ AliITSpidESD2::~AliITSpidESD2(){
 
 }
 //______________________________________________________________________
-AliITSpidESD2::AliITSpidESD2(const AliITSpidESD2 &ob) :AliITSpidESD(ob) {
+AliITSpidESD2::AliITSpidESD2(const AliITSpidESD2 &ob) :AliITSpidESD(ob),
+fTracker(ob.fTracker),
+fLoader(ob.fLoader),
+fSp(ob.fSp) 
+{
   // Copy constructor
-  // Copies are not allowed. The method is protected to avoid misuse.
-  Error("AliITSpidESD2","Copy constructor not allowed\n");
+ 
 }
 
 //______________________________________________________________________
-AliITSpidESD2& AliITSpidESD2::operator=(const AliITSpidESD2& /* ob */){
+AliITSpidESD2& AliITSpidESD2::operator=(const AliITSpidESD2& ob ){
   // Assignment operator
-  // Assignment is not allowed. The method is protected to avoid misuse.
-  Error("= operator","Assignment operator not allowed\n");
+  this->~AliITSpidESD2();
+  new(this) AliITSpidESD2(ob);
   return *this;
 }
 

diff --git a/ITS/AliITSpidESD2.h b/ITS/AliITSpidESD2.h
index 14e5eb2f51257d1c6e4dd201f62c6f261be4879b..f8c91680c4ee21431652597138611ec71d436cf4 100755 (executable)
--- a/ITS/AliITSpidESD2.h
+++ b/ITS/AliITSpidESD2.h
@@ -25,7 +25,7 @@ public:
   virtual ~AliITSpidESD2();
   virtual Int_t MakePID(AliESD *event);
   AliITSpidESD2(const AliITSpidESD2 &ob); // copy constructor
-  AliITSpidESD2& operator=(const AliITSpidESD2 & /* source */); // ass. op.
+  AliITSpidESD2& operator=(const AliITSpidESD2 & source); // ass. op.
 
 private:
   AliITStrackerMI *fTracker; //!tracker MI