Fix effc++ warnings.
authormtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Nov 2006 19:05:29 +0000 (19:05 +0000)
committermtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 14 Nov 2006 19:05:29 +0000 (19:05 +0000)
19 files changed:
EVE/Alieve/TPCSectorData.h
EVE/Reve/BoxSet.cxx
EVE/Reve/BoxSetGL.h
EVE/Reve/QuadSet.cxx
EVE/Reve/QuadSet.h
EVE/Reve/QuadSetGL.h
EVE/Reve/RGBAPalette.cxx
EVE/Reve/RGBAPaletteEditor.cxx
EVE/Reve/RGEditor.cxx
EVE/Reve/RGEditor.h
EVE/Reve/RGValuators.h
EVE/Reve/RenderElement.cxx
EVE/Reve/RenderElement.h
EVE/Reve/TriangleSet.cxx
EVE/Reve/TriangleSet.h
EVE/Reve/ZTrans.cxx
EVE/Reve/ZTrans.h
EVE/Reve/ZTransEditor.cxx
EVE/Reve/ZTransEditor.h

index 47d5b8f..9d8465c 100644 (file)
@@ -29,6 +29,10 @@ public:
   public:
     PadData(Short_t* d=0, Short_t l=0) : fData(d), fLength(l) {}
 
+    PadData(const PadData& p) : fData(p.fData), fLength(p.fLength) {}
+    PadData& operator=(const PadData& p)
+    { fData = p.fData; fLength = p.fLength; return *this; }
+
     Short_t* Data()   const { return fData; }
     Short_t  Length() const { return fLength; }
 
index d23d8dc..859d84f 100644 (file)
@@ -69,7 +69,10 @@ ClassImp(BoxSet)
 BoxSet::BoxSet(const Text_t* n, const Text_t* t) :
   RenderElement(fDefaultColor),
   TNamed(n, t),
-  fRenderMode (RM_AsIs)
+  fDefaultColor(0),
+  fRenderMode(RM_AsIs),
+  fHMTrans(),
+  fBoxes()
 {}
 
 /**************************************************************************/
index b510412..1c263d0 100644 (file)
@@ -11,6 +11,9 @@ class BoxSet;
 
 class BoxSetGL : public TGLObject
 {
+  BoxSetGL(const BoxSetGL&);            // Not implemented
+  BoxSetGL& operator=(const BoxSetGL&); // Not implemented
+
 protected:
   BoxSet* fM;
 
index b450d0c..8cd1e98 100644 (file)
@@ -189,7 +189,9 @@ QuadSet::QuadSet(const Text_t* n, const Text_t* t) :
   fDefWidth(1), fDefHeight(1), fDefCoord(0),
 
   fFrame  (0),
-  fPalette(0)
+  fPalette(0),
+  fRenderMode(RM_AsIs),
+  fHMTrans()
 {}
 
 QuadSet::QuadSet(QuadType_e quadType, Bool_t valIsCol, Int_t chunkSize,
@@ -206,7 +208,9 @@ QuadSet::QuadSet(QuadType_e quadType, Bool_t valIsCol, Int_t chunkSize,
   fDefWidth(1), fDefHeight(1), fDefCoord(0),
 
   fFrame  (0),
-  fPalette(0)
+  fPalette(0),
+  fRenderMode(RM_AsIs),
+  fHMTrans()
 {}
 
 QuadSet::~QuadSet()
index 31c614c..f84cf92 100644 (file)
@@ -81,6 +81,9 @@ class QuadSet : public RenderElement,
 {
   friend class QuadSetGL;
 
+  QuadSet(const QuadSet&);            // Not implemented
+  QuadSet& operator=(const QuadSet&); // Not implemented
+
 public:
   enum QuadType_e
   { 
index 3a786b9..63df2ba 100644 (file)
@@ -28,6 +28,9 @@ public:
 
 class QuadSetGL : public TGLObject
 {
+  QuadSetGL(const QuadSetGL&);            // Not implemented
+  QuadSetGL& operator=(const QuadSetGL&); // Not implemented
+
 protected:
   QuadSet* fM;
 
index 72d2720..b576f7f 100644 (file)
@@ -16,10 +16,14 @@ ClassImp(RGBAPalette)
 RGBAPalette::RGBAPalette() :
   TObject(),
   Reve::ReferenceCount(),
+
+  fLowLimit(0), fHighLimit(0), fMinVal(0), fMaxVal(0), fNBins(0),
+
   fCutLow      (kTRUE),
   fCutHigh     (kFALSE),
   fInterpolate (kFALSE),
   fWrap        (kFALSE),
+  fDefaultColor(0),
   fColorArray  (0)
 {
   SetLimits(0, 1000);
@@ -29,10 +33,14 @@ RGBAPalette::RGBAPalette() :
 RGBAPalette::RGBAPalette(Int_t min, Int_t max) :
   TObject(),
   Reve::ReferenceCount(),
+
+  fLowLimit(0), fHighLimit(0), fMinVal(0), fMaxVal(0), fNBins(0),
+
   fCutLow      (kTRUE),
   fCutHigh     (kFALSE),
   fInterpolate (kFALSE),
   fWrap        (kFALSE),
+  fDefaultColor(0),
   fColorArray  (0)
 {
   SetLimits(0, 1000);
@@ -42,8 +50,14 @@ RGBAPalette::RGBAPalette(Int_t min, Int_t max) :
 RGBAPalette::RGBAPalette(Int_t min, Int_t max, Bool_t interp, Bool_t wrap) :
   TObject(),
   Reve::ReferenceCount(),
+
+  fLowLimit(0), fHighLimit(0), fMinVal(0), fMaxVal(0), fNBins(0),
+
+  fCutLow      (kTRUE),
+  fCutHigh     (kFALSE),
   fInterpolate (interp),
   fWrap        (wrap),
+  fDefaultColor(0),
   fColorArray  (0)
 {
   SetLimits(0, 1023);
index 9379441..d32722c 100644 (file)
@@ -18,7 +18,14 @@ using namespace Reve;
 
 
 RGBAPaletteSubEditor::RGBAPaletteSubEditor(const TGWindow* p) :
-  TGVerticalFrame(p)
+  TGVerticalFrame(p),
+
+  fM(0),
+  fMinVal(0),
+  fMaxVal(0),
+  fInterpolate(0),
+  fWrap(0), 
+  fDefaultColor(0)
 {
   // Create weeds.
 
index 42433e8..cc98a3e 100644 (file)
@@ -16,7 +16,9 @@ using namespace Reve;
 
 ClassImp(RGEditor)
 
-RGEditor::RGEditor(TCanvas* canvas) : TGedEditor(canvas)
+RGEditor::RGEditor(TCanvas* canvas) :
+  TGedEditor(canvas),
+  fRnrElement(0)
 {}
 
 void RGEditor::DisplayRenderElement(RenderElement* re)
index 3f18bc5..10d8712 100644 (file)
@@ -11,6 +11,9 @@ class RenderElement;
 
 class RGEditor : public TGedEditor
 {
+  RGEditor(const RGEditor&);            // Not implemented
+  RGEditor& operator=(const RGEditor&); // Not implemented
+
 protected:
   RenderElement* fRnrElement;
 
index a176517..150dc5d 100644 (file)
@@ -135,6 +135,9 @@ public:
 
 class RGTriVecValuator : public TGCompositeFrame
 {
+  RGTriVecValuator(const RGTriVecValuator&);            // Not implemented
+  RGTriVecValuator& operator=(const RGTriVecValuator&); // Not implemented
+
 protected:
   RGValuator* fVal[3];
 
index 88c45c5..a96db7f 100644 (file)
@@ -478,6 +478,28 @@ RenderElementList::RenderElementList(const Text_t* n, const Text_t* t, Bool_t do
 
 ClassImp(ReferenceBackPtr)
 
+ReferenceBackPtr::ReferenceBackPtr() :
+  ReferenceCount(),
+  fBackRefs()
+{}
+
+ReferenceBackPtr::~ReferenceBackPtr()
+{
+  // !!!! Complain if list not empty.
+}
+
+ReferenceBackPtr::ReferenceBackPtr(const ReferenceBackPtr&) :
+  ReferenceCount(),
+  fBackRefs()
+{}
+
+ReferenceBackPtr& ReferenceBackPtr::operator=(const ReferenceBackPtr&)
+{
+  return *this;
+}
+
+/**************************************************************************/
+
 void ReferenceBackPtr::IncRefCount(RenderElement* re)
 {
   ReferenceCount::IncRefCount();
index 93d5aa1..402e692 100644 (file)
@@ -216,11 +216,11 @@ protected:
   std::list<RenderElement*> fBackRefs;
 
 public:
-  ReferenceBackPtr() : ReferenceCount() {}
-  virtual ~ReferenceBackPtr() {}
+  ReferenceBackPtr();
+  virtual ~ReferenceBackPtr();
 
-  ReferenceBackPtr(const ReferenceBackPtr&) : ReferenceCount() {}
-  ReferenceBackPtr& operator=(const ReferenceBackPtr&) { return *this; }
+  ReferenceBackPtr(const ReferenceBackPtr&);
+  ReferenceBackPtr& operator=(const ReferenceBackPtr&);
 
   void IncRefCount(RenderElement* re);
   void DecRefCount(RenderElement* re);
index 6fbde3c..0fdb134 100644 (file)
@@ -18,9 +18,10 @@ ClassImp(TriangleSet)
 TriangleSet::TriangleSet(Int_t nv, Int_t nt, Bool_t norms, Bool_t cols) :
   RenderElement(fColor),
   TNamed("TriangleSet", 0),
-  fNVerts  (nv),
-  fNTrings (nt),
-  fColor   (2)
+  fNVerts  (nv), fVerts(0),
+  fNTrings (nt), fTrings(0), fTringNorms(0), fTringCols(0),
+  fColor   (2),
+  fHMTrans ()
 {
   fVerts  = new Float_t[3*fNVerts];
   fTrings = new Int_t  [3*fNTrings];
index 32d9ca4..2231455 100644 (file)
@@ -22,6 +22,9 @@ class TriangleSet : public RenderElement,
   friend class TriangleSetEditor;
   friend class TriangleSetGL;
 
+  TriangleSet(const TriangleSet&);            // Not implemented
+  TriangleSet& operator=(const TriangleSet&); // Not implemented
+
 protected:
 
   // Vertex data
index e40deab..409b0f6 100644 (file)
@@ -62,9 +62,23 @@ ClassImp(ZTrans)
 
 /**************************************************************************/
 
-ZTrans::ZTrans() { UnitTrans(); fUseTrans = kTRUE; fEditTrans = kFALSE; }
+ZTrans::ZTrans() :
+  TObject(),
+  mA1(0), mA2(0), mA3(0), bAsOK(kFALSE),
+  fUseTrans (kTRUE),
+  fEditTrans(kFALSE)
+{
+  UnitTrans();
+}
 
-ZTrans::ZTrans(const ZTrans& z) : TObject() { *this = z; }
+ZTrans::ZTrans(const ZTrans& t) :
+  TObject(),
+  mA1(t.mA1), mA2(t.mA2), mA3(t.mA3), bAsOK(t.bAsOK),
+  fUseTrans (t.fUseTrans),
+  fEditTrans(t.fEditTrans)
+{
+  SetTrans(t, kFALSE);
+}
 
 /**************************************************************************/
 
@@ -75,7 +89,7 @@ void ZTrans::UnitTrans()
   memset(M, 0, 16*sizeof(Double_t));
   M[F00] = M[F11] = M[F22] = M[F33] = 1;
   mA1 = mA2 = mA3 = 0;
-  bAsOK = true;
+  bAsOK = kTRUE;
 }
 
 void ZTrans::UnitRot()
@@ -85,16 +99,20 @@ void ZTrans::UnitRot()
   memset(M, 0, 12*sizeof(Double_t));
   M[F00] = M[F11] = M[F22] = 1;
   mA1 = mA2 = mA3 = 0;
-  bAsOK = true;
+  bAsOK = kTRUE;
 }
 
-void ZTrans::SetTrans(const ZTrans& t)
+void ZTrans::SetTrans(const ZTrans& t, Bool_t copyAngles)
 {
   memcpy(M, t.M, sizeof(M));
-  bAsOK = false;
+  if (copyAngles && t.bAsOK) {
+    bAsOK = kTRUE;
+    mA1 = t.mA1; mA2 = t.mA2; mA3 = t.mA3; 
+  } else {
+    bAsOK = kFALSE;
+  }
 }
 
-
 void ZTrans::SetupRotation(Int_t i, Int_t j, Double_t f)
 {
   // Setup the matrix as an elementary rotation.
@@ -107,7 +125,7 @@ void ZTrans::SetupRotation(Int_t i, Int_t j, Double_t f)
   M(i,i) = M(j,j) = TMath::Cos(f);
   Double_t s = TMath::Sin(f);
   M(i,j) = -s; M(j,i) = s;
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 /**************************************************************************/
@@ -126,7 +144,7 @@ void ZTrans::MultLeft(const ZTrans& t)
       B[r] = T[0]*C[0] + T[4]*C[1] + T[8]*C[2] + T[12]*C[3];
     C[0] = B[0]; C[1] = B[1]; C[2] = B[2]; C[3] = B[3];
   }
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 void ZTrans::MultRight(const ZTrans& t)
@@ -139,7 +157,7 @@ void ZTrans::MultRight(const ZTrans& t)
       B[c] = C[0]*T[0] + C[4]*T[1] + C[8]*T[2] + C[12]*T[3];
     C[0] = B[0]; C[4] = B[1]; C[8] = B[2]; C[12] = B[3];
   }
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 ZTrans ZTrans::operator*(const ZTrans& t)
@@ -182,7 +200,7 @@ void ZTrans::RotateLF(Int_t i1, Int_t i2, Double_t amount)
     b2 = cos*C[i2] - sin*C[i1];
     C[i1] = b1; C[i2] = b2;
   }
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 /**************************************************************************/
@@ -216,7 +234,7 @@ void ZTrans::RotatePF(Int_t i1, Int_t i2, Double_t amount)
     b2 = cos*C[i2] + sin*C[i1];
     C[i1] = b1; C[i2] = b2;
   }
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 /**************************************************************************/
@@ -245,7 +263,7 @@ void ZTrans::Rotate(const ZTrans& a, Int_t i1, Int_t i2, Double_t amount)
   MultLeft(X);
   RotatePF(i1, i2, amount);
   MultLeft(a);
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 /**************************************************************************/
@@ -256,14 +274,14 @@ void ZTrans::SetBaseVec(Int_t b, Double_t x, Double_t y, Double_t z)
 {
   Double_t* C = M + 4*--b;
   C[0] = x; C[1] = y; C[2] = z;
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 void ZTrans::SetBaseVec(Int_t b, const TVector3& v)
 {
   Double_t* C = M + 4*--b;
   v.GetXYZ(C);
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 TVector3 ZTrans::GetBaseVec(Int_t b) const
@@ -357,7 +375,7 @@ void ZTrans::SetRotByAnyAngles(Float_t a1, Float_t a2, Float_t a3,
     case 'z': case 'Z': RotateLF(1, 2, a[i]); break;
     }
   }
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 void ZTrans::GetRotAngles(Float_t* x) const
@@ -559,7 +577,7 @@ Double_t ZTrans::Invert()
   M[F32] = det3_013_012 * mn1OverDet;
   M[F33] = det3_012_012 * oneOverDet;
 
-  bAsOK = false;
+  bAsOK = kFALSE;
   return det;
 }
 
@@ -571,7 +589,7 @@ void ZTrans::Streamer(TBuffer &R__b)
 
    if (R__b.IsReading()) {
       ZTrans::Class()->ReadBuffer(R__b, this);
-      bAsOK = false;
+      bAsOK = kFALSE;
    } else {
       ZTrans::Class()->WriteBuffer(R__b, this);
    }
@@ -609,7 +627,7 @@ void ZTrans::SetFrom(Double_t* carr)
 {
   fUseTrans = kTRUE;
   memcpy(M, carr, 16*sizeof(Double_t));
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 void ZTrans::SetFrom(const TGeoMatrix& mat)
@@ -623,7 +641,7 @@ void ZTrans::SetFrom(const TGeoMatrix& mat)
   m[0] = r[1]*s[1]; m[1] = r[4]*s[1]; m[2] = r[7]*s[1]; m[3] = 0; m += 4;
   m[0] = r[2]*s[2]; m[1] = r[5]*s[2]; m[2] = r[8]*s[2]; m[3] = 0; m += 4;
   m[0] = t[0];      m[1] = t[1];      m[2] = t[2];      m[3] = 1;
-  bAsOK = false;
+  bAsOK = kFALSE;
 }
 
 void ZTrans::SetBuffer3D(TBuffer3D& buff)
index 5c0c5eb..3251c32 100644 (file)
@@ -47,14 +47,14 @@ protected:
 
 public:
   ZTrans();
-  ZTrans(const ZTrans& z);
+  ZTrans(const ZTrans& t);
   virtual ~ZTrans() {}
 
   // General operations
 
   void     UnitTrans();
   void     UnitRot();
-  void     SetTrans(const ZTrans& t);
+  void     SetTrans(const ZTrans& t, Bool_t copyAngles=kTRUE);
   ZTrans&  operator=(const ZTrans& t) { SetTrans(t); return *this; }
   void     SetupRotation(Int_t i, Int_t j, Double_t f);
 
index 680f536..1af312a 100644 (file)
@@ -19,7 +19,21 @@ ClassImp(ZTransSubEditor)
 
 ZTransSubEditor::ZTransSubEditor(TGWindow* p) :
   TGVerticalFrame(p),
-  fTrans  (0)
+  fTrans  (0),
+
+  fTopHorFrame(0),
+
+  fUseTrans(0),
+  fEditTrans(0),
+
+  fEditTransFrame(0),
+
+  fPos(0),
+  fRot(0),
+  fScale(0),
+
+  fAutoUpdate(0),
+  fUpdate(0)
 {
   // --- Top controls
 
index 4ffcba9..ff1437d 100644 (file)
@@ -15,6 +15,9 @@ class RGTriVecValuator;
 
 class ZTransSubEditor : public TGVerticalFrame
 {
+  ZTransSubEditor(const ZTransSubEditor&);            // Not implemented
+  ZTransSubEditor& operator=(const ZTransSubEditor&); // Not implemented
+
 protected:
   ZTrans            *fTrans;
 
@@ -30,7 +33,7 @@ protected:
   RGTriVecValuator  *fScale;
 
   TGCheckButton     *fAutoUpdate;
-  TGTextButton      *fUpdate;
+  TGTextButton      *fUpdate;          
 
 public:
   ZTransSubEditor(TGWindow* p);