Fix effc++ warnings and shadowed variables.
authormtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 20 May 2008 10:32:40 +0000 (10:32 +0000)
committermtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 20 May 2008 10:32:40 +0000 (10:32 +0000)
EVE/EveBase/AliEveCosmicRayFitter.cxx
EVE/EveBase/AliEveJetPlaneEditor.cxx
EVE/EveDet/AliEveITSDigitsInfo.cxx
EVE/EveDet/AliEveITSModuleStepper.cxx
EVE/EveDet/AliEveITSScaledModule.cxx
EVE/EveDet/AliEveTPCSectorData.cxx

index 51cea3c..96cd9e1 100644 (file)
@@ -48,6 +48,9 @@ ClassImp(AliEveCosmicRayFitter)
 AliEveCosmicRayFitter::AliEveCosmicRayFitter(const Text_t* name, Int_t n_points) :
     TEvePointSet    (name, n_points),
 
+    fLineFitter1    (0),
+    fLineFitter2    (0),
+
     fConnected      (kFALSE),
     fSPMap          (),
     fTrackList      (0),
index 6cd0df5..ebc807a 100644 (file)
@@ -256,8 +256,7 @@ AliEveJetPlaneEditor::StaticDataWindow::StaticDataWindow(const TGWindow *p, cons
   //    fRad2->Connect("Clicked()", "TestDialog", this, "HandleButtons()");
 
 
-  TGLayoutHints *fL5 = new TGLayoutHints(kLHintsBottom | kLHintsExpandX |
-                                        kLHintsExpandY, 2, 2, 5, 1);
+  fL5 = new TGLayoutHints(kLHintsBottom | kLHintsExpandX | kLHintsExpandY, 2, 2, 5, 1);
   AddFrame(fTab, fL5);
 
   MapSubwindows();
index 4586ea2..487eced 100644 (file)
@@ -78,13 +78,13 @@ ClassImp(AliEveITSDigitsInfo)
 AliEveITSDigitsInfo::AliEveITSDigitsInfo() :
   TObject(),
   TEveRefCnt(),
-  fSPDmap(), fSDDmap(), fSSDmap(),
-  fTree (0),
+  fTree       (0),
   fSegSPD     (0), fSegSDD     (0), fSegSSD     (0),
   fDDLMapSDD  (0),
   fSPDMinVal  (0), fSSDMinVal  (0), fSDDMinVal  (0),
   fSPDMaxVal  (0), fSSDMaxVal  (0), fSDDMaxVal  (0),
-  fSPDHighLim (0), fSDDHighLim (0), fSSDHighLim (0)
+  fSPDHighLim (0), fSDDHighLim (0), fSSDHighLim (0),
+  fSPDmap     (),  fSDDmap     (),  fSSDmap     ()
 {
   // Default constructor.
 
index c90c290..a69b4d8 100644 (file)
@@ -48,6 +48,7 @@ AliEveITSModuleStepper::AliEveITSModuleStepper(AliEveITSDigitsInfo* di) :
   fPosition(0),
   fSubDet(-1),
 
+  fModuleFont(), fTextFont(), fSymbolFont(),
   fAxis(0),
 
   fMenuHeight(0.13),
@@ -531,18 +532,21 @@ void AliEveITSModuleStepper::Render(TGLRnrCtx& rnrCtx)
   // Render the overlay elements.
 
   AliEveITSScaledModule* sm = dynamic_cast<AliEveITSScaledModule*>(*BeginChildren());
-  Int_t scale = fScaleInfo->GetScale() - 1;
+  Int_t scaleIdx = fScaleInfo->GetScale() - 1;
   Int_t cnx = 0, cnz = 0;
   switch(sm->GetSubDetID())
   {
     case 0:
-      cnx = fDigitsInfo->fSPDScaleX[scale], cnz = fDigitsInfo->fSPDScaleZ[scale];
+      cnx = fDigitsInfo->fSPDScaleX[scaleIdx];
+      cnz = fDigitsInfo->fSPDScaleZ[scaleIdx];
       break;
     case 1:
-      cnx = fDigitsInfo->fSDDScaleX[scale], cnz = fDigitsInfo->fSDDScaleZ[scale];
+      cnx = fDigitsInfo->fSDDScaleX[scaleIdx];
+      cnz = fDigitsInfo->fSDDScaleZ[scaleIdx];
       break;
     case 2:
-      cnx = fDigitsInfo->fSSDScale[scale], cnz = 1;
+      cnx = fDigitsInfo->fSSDScale[scaleIdx];
+      cnz = 1;
       break;
   }
 
@@ -576,17 +580,17 @@ void AliEveITSModuleStepper::Render(TGLRnrCtx& rnrCtx)
     glPushMatrix();
     glLoadIdentity();
     glTranslatef(-1, -1, 0); // translate to lower left corner
-    Float_t scale = fMenuHeight/fTextSize*0.5; // scale text
-    glScalef(scale, scale, 1.);
+    Float_t txtScale = fMenuHeight/fTextSize*0.5; // scale text
+    glScalef(txtScale, txtScale, 1.);
 
     //menu
     glPushName(0);
     RenderMenu(GetCurrentPage(), GetPages(), cnx, cnz);
     glPopName();
     //palette
-    Double_t ls = 1.6*scale*fTextSize;
-    fAxis->SetLabelsSize(ls);
-    fAxis->SetLabelsOffset(ls*1.2);
+    Double_t labelSize = 1.6*txtScale*fTextSize;
+    fAxis->SetLabelsSize(labelSize);
+    fAxis->SetLabelsOffset(1.2*labelSize);
     RenderPalette(sm->GetPalette());
 
     glPopMatrix();
index aaa9c59..34e4fcb 100644 (file)
@@ -107,12 +107,12 @@ ClassImp(AliEveITSScaledModule)
 
 AliEveITSScaledModule::AliEveITSScaledModule(Int_t gid, AliEveITSDigitsInfo* info, AliEveDigitScaleInfo* si):
   AliEveITSModule("AliEveITSScaledModule", "AliEveITSScaledModule"),
-  fDigitsMap(),
   fNx(-1),
   fNz(-1),
   fNCx(-1),
   fNCz(-1),
-  fScaleInfo(si)
+  fScaleInfo(si),
+  fDigitsMap()
 {
   SetOwnIds(kTRUE);
 
@@ -146,7 +146,6 @@ void AliEveITSScaledModule::LoadQuads()
   digits  = fInfo->GetDigits(fID, fDetID);
   ndigits = digits->GetEntriesFast();
 
-  ScaledDigit_t* sd;
   Int_t scale = fScaleInfo->GetScale() -1;
   switch(fDetID)
   {
@@ -170,10 +169,11 @@ void AliEveITSScaledModule::LoadQuads()
 
        fInfo->GetSPDLocalZ(od->GetCoord1(),zo);
         c1 = od->GetCoord1(); c2 = od->GetCoord2();
-       i = Int_t((zo+fDz)/dpz);
-       j = Int_t((od->GetCoord2()*fNx)/fInfo->fSegSPD->Npx());
+       i  = Int_t((zo+fDz)/dpz);
+       j  = Int_t((od->GetCoord2()*fNx)/fInfo->fSegSPD->Npx());
        id = j*fNx + i;
 
+        ScaledDigit_t* sd = 0;
         miter = dmap.find(id);
        if(miter == dmap.end())
        {
@@ -213,23 +213,25 @@ void AliEveITSScaledModule::LoadQuads()
       fNCx = fInfo->fSDDScaleX[scale];
       fNz  = Int_t(fInfo->fSegSDD->Npz()/fNCz);
       fNx  = Int_t(fInfo->fSegSDD->Npx()/fNCx);
-      dpz = 2*fDz/fNz;
-      dpx = 2*fDx/fNx;
+      dpz  = 2*fDz/fNz;
+      dpx  = 2*fDx/fNx;
 
-      AliITSdigitSDD *od=0;
-      for (Int_t k=0; k<ndigits; k++) {
-       od=(AliITSdigitSDD*)digits->UncheckedAt(k);
+      AliITSdigitSDD *od = 0;
+      for (Int_t k = 0; k < ndigits; ++k)
+      {
+       od = (AliITSdigitSDD*)digits->UncheckedAt(k);
        fInfo->fSegSDD->DetToLocal(od->GetCoord2(), od->GetCoord1(),x,z);
-       z+= fDz;
-       x+= fDx;
-       i = Int_t(z/dpz);
-       j = Int_t(x/dpx);
+       z += fDz;
+       x += fDx;
+       i  = Int_t(z/dpz);
+       j  = Int_t(x/dpx);
        //printf("Mod %d coord %d,%d out of %d,%d :: ORIG coord %d,%d out of %d,%d \n",fID,
        //       i,j,Nz,Nx,od->GetCoord1(),od->GetCoord2(),fInfo->fSegSDD->Npz(),fInfo->fSegSDD->Npx());
 
        id = j*fNx + i;
        c1 = od->GetCoord1(); c2 = od->GetCoord2();
 
+       ScaledDigit_t* sd = 0;
        miter = dmap.find(id);
        if(miter == dmap.end())
        {
@@ -276,7 +278,7 @@ void AliEveITSScaledModule::LoadQuads()
       dpz = 2*fDz/fNz;
       dpx = 2*fDx/fNx;
 
-      AliITSdigitSSD *od=0;
+      AliITSdigitSSD *od = 0;
       for (Int_t k=0; k<ndigits; k++) {
        od=(AliITSdigitSSD*)digits->UncheckedAt(k);
        if(od->GetCoord1() == 1)
@@ -287,8 +289,8 @@ void AliEveITSScaledModule::LoadQuads()
        c1 = od->GetCoord1(); c2 = od->GetCoord2();
        id = j*i;
 
+       ScaledDigit_t* sd = 0;
        miter = dmap.find(id);
-       ScaledDigit_t* sd;
        if(miter == dmap.end())
        {
          // printf("orig digit %d,%d scaled %d,%d \n",od->GetCoord1(),od->GetCoord2(),i,j);
index dd79270..37e211a 100644 (file)
@@ -59,19 +59,20 @@ void AliEveTPCSectorData::InitStatics()
   fgNAllPads = 0;
   fgRowBegs  = new Int_t[fgNAllRows + 1];
 
-  Int_t row = 0;
-  for (Int_t i=0; i<fgParam->GetNRowLow(); ++i, ++row)
   {
-    fgRowBegs[row] = fgNAllPads;
-    fgNAllPads += fgParam->GetNPadsLow(i);
-  }
-  for (Int_t i=0; i<fgParam->GetNRowUp(); ++i, ++row)
-  {
-    fgRowBegs[row] = fgNAllPads;
-    fgNAllPads += fgParam->GetNPadsUp(i);
+    Int_t row = 0;
+    for (Int_t i=0; i<fgParam->GetNRowLow(); ++i, ++row)
+    {
+      fgRowBegs[row] = fgNAllPads;
+      fgNAllPads += fgParam->GetNPadsLow(i);
+    }
+    for (Int_t i=0; i<fgParam->GetNRowUp(); ++i, ++row)
+    {
+      fgRowBegs[row] = fgNAllPads;
+      fgNAllPads += fgParam->GetNPadsUp(i);
+    }
+    fgRowBegs[fgNAllRows] = fgNAllPads;
   }
-  fgRowBegs[fgNAllRows] = fgNAllPads;
-
 
   // Fill SegmentInfos, used by rendering classes.
 
@@ -103,44 +104,49 @@ void AliEveTPCSectorData::InitStatics()
   fgOut2Seg.fNMaxPads  = fgParam->GetNPadsUp(fgParam->GetNRowUp() - 1);
   fgSegInfoPtrs[2]     = &fgOut2Seg;
 
-  // Set stepsize array
-  Int_t k, npads;
-  // Inn
-  k=0, npads = fgParam->GetNPadsLow(0);
-  for (int row = 0; row < fgInnSeg.fNRows; ++row)
-  {
-    if (fgParam->GetNPadsLow(row) > npads)
+  // Set stepsize arrays
+
+  { // Inner
+    Int_t k=0, npads = fgParam->GetNPadsLow(0);
+    for (Int_t row = 0; row < fgInnSeg.fNRows; ++row)
     {
-      npads = fgParam->GetNPadsLow(row);
-      fgInnSeg.fYStep[k] = row*fgInnSeg.fPadHeight + fgInnSeg.fRLow;
-      k++;
+      if (fgParam->GetNPadsLow(row) > npads)
+      {
+        npads = fgParam->GetNPadsLow(row);
+        fgInnSeg.fYStep[k] = row*fgInnSeg.fPadHeight + fgInnSeg.fRLow;
+        k++;
+      }
     }
+    fgInnSeg.fNYSteps = k;
   }
-  fgInnSeg.fNYSteps = k;
-  // Out1 seg
-  k=0; npads = fgParam->GetNPadsUp(0);
-  for (int row = 0; row < fgOut1Seg.fNRows; ++row)
-  {
-    if (fgParam->GetNPadsUp(row) > npads)
+
+  {  // Outer 1 seg
+    Int_t k=0, npads = fgParam->GetNPadsUp(0);
+    for (Int_t row = 0; row < fgOut1Seg.fNRows; ++row)
     {
-      npads = fgParam->GetNPadsUp(row);
-      fgOut1Seg.fYStep[k] = row*fgOut1Seg.fPadHeight + fgOut1Seg.fRLow ;
-      k++;
+      if (fgParam->GetNPadsUp(row) > npads)
+      {
+        npads = fgParam->GetNPadsUp(row);
+        fgOut1Seg.fYStep[k] = row*fgOut1Seg.fPadHeight + fgOut1Seg.fRLow ;
+        k++;
+      }
     }
+    fgOut1Seg.fNYSteps = k;
   }
-  fgOut1Seg.fNYSteps = k;
-  // Out2 seg
-  k=0; npads = fgParam->GetNPadsUp(fgOut1Seg.fNRows);
-  for (int row = fgOut1Seg.fNRows; row < fgParam->GetNRowUp() ;row++ )
-  {
-    if (fgParam->GetNPadsUp(row) > npads)
+
+  {  // Outer 2 seg
+    Int_t k=0, npads = fgParam->GetNPadsUp(fgOut1Seg.fNRows);
+    for (Int_t row = fgOut1Seg.fNRows; row < fgParam->GetNRowUp(); ++row)
     {
-      npads = fgParam->GetNPadsUp(row);
-      fgOut2Seg.fYStep[k] = (row - fgOut1Seg.fNRows)*fgOut2Seg.fPadHeight + fgOut2Seg.fRLow ;
-      k++;
+      if (fgParam->GetNPadsUp(row) > npads)
+      {
+        npads = fgParam->GetNPadsUp(row);
+        fgOut2Seg.fYStep[k] = (row - fgOut1Seg.fNRows)*fgOut2Seg.fPadHeight + fgOut2Seg.fRLow ;
+        k++;
+      }
     }
+    fgOut2Seg.fNYSteps = k;
   }
-  fgOut2Seg.fNYSteps = k;
 }
 
 Int_t AliEveTPCSectorData::GetNPadsInRow(Int_t row)