From 58c0119e45dbe25b025585e8f5f4dbff2a3c211f Mon Sep 17 00:00:00 2001
From: ivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Date: Wed, 2 May 2001 10:08:08 +0000
Subject: [PATCH] copy constructor changed with usage of = operator

---
 AliGeant4/AliColour.cxx                   |  6 ++----
 AliGeant4/AliDetSwitch.cxx                |  9 ++-------
 AliGeant4/AliGunParticle.cxx              |  7 ++-----
 AliGeant4/AliLVStructure.cxx              | 21 ++-------------------
 AliGeant4/AliLegoSensitiveDetector.cxx    |  4 ++--
 AliGeant4/AliModuleConstruction.cxx       | 23 +++++++++++------------
 AliGeant4/AliMoreModulesConstruction.cxx  | 18 +++---------------
 AliGeant4/AliParticleGun.cxx              | 18 +++++++++---------
 AliGeant4/AliSensitiveDetector.cxx        |  3 ++-
 AliGeant4/AliSingleModuleConstruction.cxx | 23 +++++++++++++----------
 TGeant4/TG4G3ControlVector.cxx            | 10 +++++-----
 TGeant4/TG4G3CutVector.cxx                | 12 ++++++------
 TGeant4/TG4Limits.cxx                     |  8 ++++++--
 13 files changed, 65 insertions(+), 97 deletions(-)

diff --git a/AliGeant4/AliColour.cxx b/AliGeant4/AliColour.cxx
index 3f33a8abf0b..eeeddc14154 100644
--- a/AliGeant4/AliColour.cxx
+++ b/AliGeant4/AliColour.cxx
@@ -26,10 +26,8 @@ AliColour::AliColour(G4String name, G4double red, G4double blue, G4double green)
 
 AliColour::AliColour(const AliColour& right) {
 //
-  fName = right.fName;
-  fRed = right.fRed;
-  fBlue = right.fBlue;
-  fGreen = right.fGreen;
+  // copy stuff
+  *this = right;
 }
 
 AliColour::~AliColour() {
diff --git a/AliGeant4/AliDetSwitch.cxx b/AliGeant4/AliDetSwitch.cxx
index 7525dac0539..4bff2ee1cdb 100644
--- a/AliGeant4/AliDetSwitch.cxx
+++ b/AliGeant4/AliDetSwitch.cxx
@@ -22,13 +22,8 @@ AliDetSwitch::AliDetSwitch(G4String detName, G4int nofVersions,
 
 AliDetSwitch::AliDetSwitch(const AliDetSwitch& right) {
 //
-  fDetName = right.fDetName;
-  fNofVersions = right.fNofVersions;
-  fDefaultVersion = right.fDefaultVersion;
-  fPPRVersion = right.fPPRVersion;
-  fIsStandalone = right.fIsStandalone;
-  fType = right.fType;
-  fSwitchedVersion = right.fSwitchedVersion;
+  // copy stuff
+  *this = right;
 }
 
 AliDetSwitch::~AliDetSwitch(){
diff --git a/AliGeant4/AliGunParticle.cxx b/AliGeant4/AliGunParticle.cxx
index f47bce889e6..113997b7cdb 100644
--- a/AliGeant4/AliGunParticle.cxx
+++ b/AliGeant4/AliGunParticle.cxx
@@ -47,11 +47,8 @@ AliGunParticle::AliGunParticle( G4ParticleDefinition* particleDef,
 
 AliGunParticle::AliGunParticle(const AliGunParticle& right) {
 //
-  fParticleDefinition = right.fParticleDefinition;
-  fParticleMomentum = right.fParticleMomentum;
-  fPosition = right.fPosition; 
-  fTime = right.fTime;
-  fPolarization = right.fPolarization; 
+  // copy stuff
+  *this = right;
 }
 
 AliGunParticle::~AliGunParticle() {
diff --git a/AliGeant4/AliLVStructure.cxx b/AliGeant4/AliLVStructure.cxx
index 4a4a138e27f..d9ac1ba81d6 100644
--- a/AliGeant4/AliLVStructure.cxx
+++ b/AliGeant4/AliLVStructure.cxx
@@ -31,25 +31,8 @@ AliLVStructure::AliLVStructure(G4String path)
 
 AliLVStructure::AliLVStructure(const AliLVStructure& right)
 {
-  // copy vector of structures
-  fStructures.clearAndDestroy();
-  G4int i;
-  for (i=0; i<right.fStructures.entries(); i++) {
-    // new full structure tree has to be created
-    AliLVStructure* rhsStructure = right.fStructures[i];
-    fStructures.insert(new AliLVStructure(*rhsStructure)); 
-  }  
-  
-  // copy vector of logical volumes
-  fLogicalVolumes.clear();
-  for (i=0; i<right.fLogicalVolumes.entries(); i++) {
-    G4LogicalVolume* rhsLV = right.fLogicalVolumes[i];
-    fLogicalVolumes.insert(rhsLV); 
-  }  
-  
-  fPathName = right.fPathName;
-  fDirName = right.fPathName;
-  fVerboseLevel = right.fVerboseLevel;
+  // copy stuff
+  *this = right;
 }
 
 AliLVStructure::AliLVStructure() {
diff --git a/AliGeant4/AliLegoSensitiveDetector.cxx b/AliGeant4/AliLegoSensitiveDetector.cxx
index e1887290721..5d950b1c4d6 100644
--- a/AliGeant4/AliLegoSensitiveDetector.cxx
+++ b/AliGeant4/AliLegoSensitiveDetector.cxx
@@ -21,8 +21,8 @@ AliLegoSensitiveDetector::AliLegoSensitiveDetector(
   : TG4VSensitiveDetector(right)			     
 {
 //
-  fLego = right.fLego;
-  fStandardSD = right.fStandardSD;
+  // copy stuff
+  *this = right;
 }
 
 AliLegoSensitiveDetector::AliLegoSensitiveDetector() {
diff --git a/AliGeant4/AliModuleConstruction.cxx b/AliGeant4/AliModuleConstruction.cxx
index eb7685b09fe..4a8c8b96782 100644
--- a/AliGeant4/AliModuleConstruction.cxx
+++ b/AliGeant4/AliModuleConstruction.cxx
@@ -35,18 +35,11 @@ AliModuleConstruction::AliModuleConstruction(G4String moduleName)
 AliModuleConstruction::AliModuleConstruction(const AliModuleConstruction& right)
 {
 //
-  fModuleName = right.fModuleName; 
-  fModuleFrameName = right.fModuleFrameName;
-  fModuleFrameLV = right.fModuleFrameLV;
-  fAliModule = right.fAliModule;
-  fReadGeometry = right.fReadGeometry;
-  fWriteGeometry = right.fWriteGeometry;
-  fDataFilePath = right.fDataFilePath;
-
-  // new messenger
-  G4String moduleName = fModuleName;
-  moduleName.toLower();
-  fMessenger = new AliModuleConstructionMessenger(this, moduleName);
+  // allocation in assignement operator
+  fMessenger = 0;
+  
+  // copy stuff
+  *this = right;
 }
 
 AliModuleConstruction::AliModuleConstruction()
@@ -85,6 +78,12 @@ AliModuleConstruction::operator=(const AliModuleConstruction& right)
   fWriteGeometry = right.fWriteGeometry;
   fDataFilePath = right.fDataFilePath;
 
+  // new messenger
+  if (fMessenger) delete fMessenger;
+  G4String moduleName = fModuleName;
+  moduleName.toLower();
+  fMessenger = new AliModuleConstructionMessenger(this, moduleName);
+
   return *this;
 }
 
diff --git a/AliGeant4/AliMoreModulesConstruction.cxx b/AliGeant4/AliMoreModulesConstruction.cxx
index 60a71956479..147e91d4052 100644
--- a/AliGeant4/AliMoreModulesConstruction.cxx
+++ b/AliGeant4/AliMoreModulesConstruction.cxx
@@ -24,20 +24,8 @@ AliMoreModulesConstruction::AliMoreModulesConstruction() {
 AliMoreModulesConstruction::AliMoreModulesConstruction(
                                const AliMoreModulesConstruction& right)
 {
-  // delete current module constructions
-  fModuleConstructionVector.erase(
-    fModuleConstructionVector.begin(), fModuleConstructionVector.end());
-    
-  // create new module constructions   
-  G4int nofModules = right.fModuleConstructionVector.size();
-  for (G4int i=0; i<nofModules; i++) {
-    G4String name = right.fModuleConstructionVector[i]->GetDetName();
-    G4int version = right.fModuleConstructionVector[i]->GetVersion();
-    AliModuleType type = right.fModuleConstructionVector[i]->GetType();
-    AddModule(name, version, type);
-  }  
-  
-  fSDManager = right.fSDManager;
+  // copy stuff
+  *this = right;
 }  
   			       
 
@@ -68,7 +56,7 @@ AliMoreModulesConstruction::operator=(const AliMoreModulesConstruction& right)
     AliModuleType type = right.fModuleConstructionVector[i]->GetType();
     AddModule(name, version, type);
   }  
-  
+
   fSDManager = right.fSDManager;
 
   return *this;  
diff --git a/AliGeant4/AliParticleGun.cxx b/AliGeant4/AliParticleGun.cxx
index b27be2618b1..b3fd5b57e6d 100644
--- a/AliGeant4/AliParticleGun.cxx
+++ b/AliGeant4/AliParticleGun.cxx
@@ -19,16 +19,13 @@ AliParticleGun::AliParticleGun() {
 }
 
 AliParticleGun::AliParticleGun(const AliParticleGun& right)
+  : G4VPrimaryGenerator(right)
 {
-  // particles vector
-  fGunParticlesVector.clearAndDestroy();
-  for (G4int i=0; i<right.fGunParticlesVector.entries(); i++) {
-    AliGunParticle* rhsParticle = right.fGunParticlesVector[i];
-    AliGunParticle* newParticle = new AliGunParticle(*rhsParticle);
-    fGunParticlesVector.insert(newParticle);
-  }  
-
+  // allocation
   fMessenger = new AliParticleGunMessenger(this);
+
+  // copy stuff
+  *this = right;
 }
 
 AliParticleGun::~AliParticleGun() {
@@ -44,6 +41,9 @@ AliParticleGun& AliParticleGun::operator=(const AliParticleGun& right)
   // check assignement to self
   if (this == &right) return *this;
 
+  // base class assignment
+  this->G4VPrimaryGenerator::operator=(right);
+
   // particles vector
   fGunParticlesVector.clearAndDestroy();
   for (G4int i=0; i<right.fGunParticlesVector.entries(); i++) {
@@ -51,7 +51,7 @@ AliParticleGun& AliParticleGun::operator=(const AliParticleGun& right)
     AliGunParticle* newParticle = new AliGunParticle(*rhsParticle);
     fGunParticlesVector.insert(newParticle);
   }
-  
+
   return *this;  
 }
   
diff --git a/AliGeant4/AliSensitiveDetector.cxx b/AliGeant4/AliSensitiveDetector.cxx
index 30a167cc46f..b250ef47bb5 100644
--- a/AliGeant4/AliSensitiveDetector.cxx
+++ b/AliGeant4/AliSensitiveDetector.cxx
@@ -33,7 +33,8 @@ AliSensitiveDetector::AliSensitiveDetector(const AliSensitiveDetector& right)
   : TG4VSensitiveDetector(right)
 {
 //
-  fModule = right.fModule;
+  // copy stuff
+  *this = right;
 }  
   
 AliSensitiveDetector::AliSensitiveDetector(){
diff --git a/AliGeant4/AliSingleModuleConstruction.cxx b/AliGeant4/AliSingleModuleConstruction.cxx
index 69cb80761f2..108a75409f9 100644
--- a/AliGeant4/AliSingleModuleConstruction.cxx
+++ b/AliGeant4/AliSingleModuleConstruction.cxx
@@ -45,15 +45,11 @@ AliSingleModuleConstruction::AliSingleModuleConstruction(
   : AliModuleConstruction(right)				
 {
 //
-  fVersion = right.fVersion;
-  fType = right.fType;
-  fProcessConfig = right.fProcessConfig;
-  fAllLVSensitive = right.fAllLVSensitive;
-  fSDManager = right.fSDManager;
-
-  G4String moduleName = right.fModuleName;
-  moduleName.toLower();
-  fMessenger = new AliSingleModuleConstructionMessenger(this, moduleName);
+  // allocation in assignement operator
+  fMessenger = 0;
+  
+  // copy stuff
+  *this = right;
 }
 
 AliSingleModuleConstruction::AliSingleModuleConstruction() {
@@ -81,7 +77,13 @@ AliSingleModuleConstruction::operator=(const AliSingleModuleConstruction& right)
   fProcessConfig = right.fProcessConfig;
   fAllLVSensitive = right.fAllLVSensitive;
   fSDManager = right.fSDManager;
-  
+ 
+  // messenger is protected from copying  
+  if (fMessenger) delete fMessenger;
+  G4String moduleName = right.fModuleName;
+  moduleName.toLower();
+  fMessenger = new AliSingleModuleConstructionMessenger(this, moduleName);
+ 
   return *this;
 }
 
@@ -161,6 +163,7 @@ void AliSingleModuleConstruction::Configure(const AliFiles& files)
   if (fProcessConfig) {
     gROOT->LoadMacro(rootFilePath);
     G4String macroName = files.GetDefaultMacroName();
+    //macroName = macroName + "_" + fModuleName;
     macroName = macroName + "(";
     AliGlobals::AppendNumberToString(macroName, fVersion);
     macroName = macroName + ")";
diff --git a/TGeant4/TG4G3ControlVector.cxx b/TGeant4/TG4G3ControlVector.cxx
index 59c8bd3d942..eccce75a572 100644
--- a/TGeant4/TG4G3ControlVector.cxx
+++ b/TGeant4/TG4G3ControlVector.cxx
@@ -20,11 +20,11 @@ TG4G3ControlVector::TG4G3ControlVector()
 
 TG4G3ControlVector::TG4G3ControlVector(const TG4G3ControlVector& right)
 {
-  // copy fControlVector 
+  // allocation
   fControlVector = new TG4ControlValueVector;
-  for (G4int i=0; i<kNoG3Controls; i++) {
-    fControlVector->insert((*right.fControlVector)[i]);
-  }   
+  
+  // copy stuff
+  *this = right;  
 }
 
 TG4G3ControlVector::~TG4G3ControlVector() {
@@ -41,7 +41,7 @@ TG4G3ControlVector& TG4G3ControlVector::operator=(
   if (this == &right) return *this;
 
   // initialize fControlVector 
-  fControlVector->clear();
+  fControlVector->clear();;
   for (G4int i=0; i<kNoG3Controls; i++) {
     fControlVector->insert((*right.fControlVector)[i]);
   }
diff --git a/TGeant4/TG4G3CutVector.cxx b/TGeant4/TG4G3CutVector.cxx
index f5d1a3187cf..7073a18344f 100644
--- a/TGeant4/TG4G3CutVector.cxx
+++ b/TGeant4/TG4G3CutVector.cxx
@@ -13,17 +13,17 @@
 TG4G3CutVector::TG4G3CutVector()
 {
   // initialize fCutVector 
-  fCutVector = new TG4doubleVector;
+  fCutVector = new TG4doubleVector();
   for (G4int i=0; i<kNoG3Cuts; i++) fCutVector->insert(0.); 
 }
 
 TG4G3CutVector::TG4G3CutVector(const TG4G3CutVector& right)
 {
-  // copy fCutVector 
-  fCutVector = new TG4doubleVector;
-  for (G4int i=0; i<kNoG3Cuts; i++) {
-    fCutVector->insert((*right.fCutVector)[i]);
-  }   
+ // allocation
+  fCutVector = new TG4doubleVector();
+
+  // copy stuff
+  *this = right;
 }
 
 TG4G3CutVector::~TG4G3CutVector() {
diff --git a/TGeant4/TG4Limits.cxx b/TGeant4/TG4Limits.cxx
index 2cd09c7fa73..b5c486d43e4 100644
--- a/TGeant4/TG4Limits.cxx
+++ b/TGeant4/TG4Limits.cxx
@@ -24,8 +24,12 @@ TG4Limits::TG4Limits(const TG4Limits& right)
   : G4UserLimits(right) 
 {
 //    
-  fCutVector = new TG4G3CutVector(*right.fCutVector);
-  fControlVector = new TG4G3ControlVector(*right.fControlVector);
+  // allocation
+  fCutVector = new TG4G3CutVector();
+  fControlVector = new TG4G3ControlVector();
+
+  // copy stuff
+  *this = right;
 }  
 
 TG4Limits::~TG4Limits() {
-- 
2.43.0