Coverity 18354 solved by non-implemented private copy constructor and assignment...
authorhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 24 Feb 2012 09:21:45 +0000 (09:21 +0000)
committerhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 24 Feb 2012 09:21:45 +0000 (09:21 +0000)
TRD/AliTRDpadPlane.cxx
TRD/AliTRDpadPlane.h

index 636bac1..c9350e8 100644 (file)
@@ -95,48 +95,6 @@ AliTRDpadPlane::AliTRDpadPlane(Int_t layer, Int_t stack)
 }
 
 //_____________________________________________________________________________
-AliTRDpadPlane::AliTRDpadPlane(const AliTRDpadPlane &p)
-  :TObject(p)
-  ,fLayer(p.fLayer)
-  ,fStack(p.fStack)
-  ,fLength(p.fLength)
-  ,fWidth(p.fWidth)
-  ,fLengthRim(p.fLengthRim)
-  ,fWidthRim(p.fLengthRim)
-  ,fLengthOPad(p.fLengthOPad)
-  ,fWidthOPad(p.fWidthOPad)
-  ,fLengthIPad(p.fLengthIPad)
-  ,fWidthIPad(p.fWidthIPad)
-  ,fRowSpacing(p.fRowSpacing)
-  ,fColSpacing(p.fColSpacing)
-  ,fNrows(p.fNrows)
-  ,fNcols(p.fNcols)
-  ,fTiltingAngle(p.fTiltingAngle)
-  ,fTiltingTan(p.fTiltingTan)
-  ,fPadRow(0)
-  ,fPadCol(0)
-  ,fPadRowSMOffset(p.fPadRowSMOffset)
-  ,fAnodeWireOffset(p.fAnodeWireOffset)
-{
-  //
-  // AliTRDpadPlane copy constructor
-  //
-
-  Int_t iBin = 0;
-
-  fPadRow = new Double_t[fNrows];
-  for (iBin = 0; iBin < fNrows; iBin++) {
-    fPadRow[iBin] = ((AliTRDpadPlane &) p).fPadRow[iBin];
-  }                                                                             
-
-  fPadCol = new Double_t[fNrows];
-  for (iBin = 0; iBin < fNrows; iBin++) {
-    fPadCol[iBin] = ((AliTRDpadPlane &) p).fPadCol[iBin];
-  }                                                                             
-
-}
-
-//_____________________________________________________________________________
 AliTRDpadPlane::~AliTRDpadPlane()
 {
   //
@@ -156,60 +114,6 @@ AliTRDpadPlane::~AliTRDpadPlane()
 }
 
 //_____________________________________________________________________________
-AliTRDpadPlane &AliTRDpadPlane::operator=(const AliTRDpadPlane &p)
-{
-  //
-  // Assignment operator
-  //
-
-  if (this == &p) {
-    return *this;
-  }
-
-  fLayer           = p.fLayer;
-  fStack           = p.fStack;
-  fLength          = p.fLength;
-  fWidth           = p.fWidth;
-  fLengthRim       = p.fLengthRim;
-  fWidthRim        = p.fLengthRim;
-  fLengthOPad      = p.fLengthOPad;
-  fWidthOPad       = p.fWidthOPad;
-  fLengthIPad      = p.fLengthIPad;
-  fWidthIPad       = p.fWidthIPad;
-  fRowSpacing      = p.fRowSpacing;
-  fColSpacing      = p.fColSpacing;
-  fNrows           = p.fNrows;
-  fNcols           = p.fNcols;
-  fTiltingAngle    = p.fTiltingAngle;
-  fTiltingTan      = p.fTiltingTan;
-  fPadRow          = 0;
-  fPadCol          = 0;
-  fPadRowSMOffset  = p.fPadRowSMOffset;
-  fAnodeWireOffset = p.fAnodeWireOffset;
-
-  Int_t iBin = 0;
-
-  if (fPadRow) {
-    delete [] fPadRow;
-  }
-  fPadRow = new Double_t[fNrows];
-  for (iBin = 0; iBin < fNrows; iBin++) {
-    fPadRow[iBin] = ((AliTRDpadPlane &) p).fPadRow[iBin];
-  }                                                                             
-
-  if (fPadCol) {
-    delete [] fPadCol;
-  }
-  fPadCol = new Double_t[fNrows];
-  for (iBin = 0; iBin < fNrows; iBin++) {
-    fPadCol[iBin] = ((AliTRDpadPlane &) p).fPadCol[iBin];
-  }                                                                             
-
-  return *this;
-
-}
-
-//_____________________________________________________________________________
 void AliTRDpadPlane::Copy(TObject &p) const
 {
   //
index 2d23fba..f13aaf0 100644 (file)
@@ -25,9 +25,7 @@ class AliTRDpadPlane : public TObject {
 
   AliTRDpadPlane();
   AliTRDpadPlane(Int_t layer, Int_t stack);
-  AliTRDpadPlane(const AliTRDpadPlane &p);
   virtual           ~AliTRDpadPlane();
-  AliTRDpadPlane    &operator=(const AliTRDpadPlane &p);
   virtual void       Copy(TObject &p) const;
 
   void     SetLayer(Int_t l)                   { fLayer          = l; };
@@ -151,6 +149,10 @@ class AliTRDpadPlane : public TObject {
 
   Double_t  fAnodeWireOffset; //  Distance of first anode wire from pad edge
 
+ private:
+  AliTRDpadPlane(const AliTRDpadPlane &p);
+  AliTRDpadPlane    &operator=(const AliTRDpadPlane &p);
+
   ClassDef(AliTRDpadPlane,6)  //  TRD ROC pad plane
 
 };