Fixes for coverity

[u/mrichter/AliRoot.git] / PMD / AliPMDdigit.cxx

diff --git a/PMD/AliPMDdigit.cxx b/PMD/AliPMDdigit.cxx
index db53b437ac18f3e7f92f1f6bab7c5df302955033..38f2682f8c382f341314017085d4986bd850bb0c 100644 (file)
--- a/PMD/AliPMDdigit.cxx
+++ b/PMD/AliPMDdigit.cxx
@@ -26,48 +26,67 @@
 
 ClassImp(AliPMDdigit)
 
-AliPMDdigit::AliPMDdigit()
+AliPMDdigit::AliPMDdigit():
+  fTrNumber(0),
+  fTrPid(0),
+  fDet(0),
+  fSMNumber(0),
+  fRow(0),
+  fColumn(0),
+  fADC(0.)
 {
   // Default Constructor
-  fTrNumber   = 0;
-  fDet        = 0;
-  fSMNumber   = 0;
-  fRow        = 0;
-  fColumn     = 0;
-  fADC        = 0.;
 }
 
-AliPMDdigit::AliPMDdigit(Int_t trnumber, Int_t det, Int_t smnumber, 
-                        Int_t irow, Int_t icol, Float_t adc)
+AliPMDdigit::AliPMDdigit(Int_t trnumber, Int_t trpid, Int_t det,
+                        Int_t smnumber, 
+                        Int_t irow, Int_t icol, Float_t adc):
+  fTrNumber(trnumber),
+  fTrPid(trpid),
+  fDet(det),
+  fSMNumber(smnumber),
+  fRow(irow),
+  fColumn(icol),
+  fADC(adc)
 {
   // Constructor
-  fTrNumber   = trnumber;
-  fDet        = det;
-  fSMNumber   = smnumber;
-  fRow        = irow;
-  fColumn     = icol;
-  fADC        = adc;
 }
-AliPMDdigit::AliPMDdigit(const AliPMDdigit& pmddigit):TObject(pmddigit) {
+AliPMDdigit::AliPMDdigit(AliPMDdigit *pmddigit):
+  fTrNumber(0),
+  fTrPid(0),
+  fDet(0),
+  fSMNumber(0),
+  fRow(0),
+  fColumn(0),
+  fADC(0.)
+{
+  *this = *pmddigit;
+}
+
+AliPMDdigit::AliPMDdigit(const AliPMDdigit& pmddigit):
+  TObject(pmddigit),
+  fTrNumber(pmddigit.fTrNumber),
+  fTrPid(pmddigit.fTrPid),
+  fDet(pmddigit.fDet),
+  fSMNumber(pmddigit.fSMNumber),
+  fRow(pmddigit.fRow),
+  fColumn(pmddigit.fColumn),
+  fADC(pmddigit.fADC)
+{
   //Copy Constructor 
-  if(&pmddigit == this) return;
-  this->fTrNumber   = pmddigit.fTrNumber;
-  this->fDet        = pmddigit.fDet;
-  this->fSMNumber   = pmddigit.fSMNumber;
-  this->fRow        = pmddigit.fRow;
-  this->fColumn     = pmddigit.fColumn;
-  this->fADC        = pmddigit.fADC;
-  return;
 }
 AliPMDdigit & AliPMDdigit::operator=(const AliPMDdigit& pmddigit) {
   //Assignment operator 
-  if(&pmddigit == this) return *this;
-  this->fTrNumber   = pmddigit.fTrNumber;
-  this->fDet        = pmddigit.fDet;
-  this->fSMNumber   = pmddigit.fSMNumber;
-  this->fRow        = pmddigit.fRow;
-  this->fColumn     = pmddigit.fColumn;
-  this->fADC        = pmddigit.fADC;
+  if(this != &pmddigit)
+    {
+      fTrNumber   = pmddigit.fTrNumber;
+      fTrPid      = pmddigit.fTrPid;
+      fDet        = pmddigit.fDet;
+      fSMNumber   = pmddigit.fSMNumber;
+      fRow        = pmddigit.fRow;
+      fColumn     = pmddigit.fColumn;
+      fADC        = pmddigit.fADC;
+    }
   return *this;
 }
 AliPMDdigit::~AliPMDdigit()
@@ -78,6 +97,10 @@ Int_t AliPMDdigit::GetTrackNumber() const
 {
   return fTrNumber;
 }
+Int_t AliPMDdigit::GetTrackPid() const
+{
+  return fTrPid;
+}
 Int_t AliPMDdigit::GetDetector() const
 {
   return fDet;