warnings with gcc 4.3.2
authormasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Nov 2008 17:49:19 +0000 (17:49 +0000)
committermasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Nov 2008 17:49:19 +0000 (17:49 +0000)
16 files changed:
ITS/AliITSBeamTestDigSDD.cxx
ITS/AliITSBeamTestDigSPD.cxx
ITS/AliITSBeamTestDigSSD.cxx
ITS/AliITSClusterFinderV2SPD.cxx
ITS/AliITSHandleDaSSD.cxx
ITS/AliITSModuleDaSSD.cxx
ITS/AliITSPid.cxx
ITS/AliITSPlaneEffSDD.cxx
ITS/AliITSPlaneEffSPD.cxx
ITS/AliITSPlaneEffSSD.cxx
ITS/AliITSVertexer.h
ITS/AliITSclustererV2.cxx
ITS/AliITStrackSA.h
ITS/AliITStrackV2.h
ITS/AliITStrackerMI.cxx
ITS/AliITStrackerSA.cxx

index bd844cf..a302c9a 100644 (file)
@@ -77,9 +77,10 @@ void AliITSBeamTestDigSDD::Exec(Option_t* /*opt*/)
     for(Int_t nlad=1;nlad<=fITSgeom->GetNladders(nlay);nlad++){
       for(Int_t ndet=1;ndet<=fITSgeom->GetNdetectors(nlay);ndet++){
        Int_t index=fITSgeom->GetModuleIndex(nlay,nlad,ndet);
-       if(fITSgeom->GetModuleTypeName(index)=="kSPD") nspd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSDD") nsdd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSSD") nssd++;
+       TString dtype(fITSgeom->GetModuleTypeName(index));
+       if(dtype.Contains("SPD")) nspd++;
+       if(dtype.Contains("SDD")) nsdd++;
+       if(dtype.Contains("SSD")) nssd++;
       }
     }
   }
index 0ae2e74..62212fb 100644 (file)
@@ -67,9 +67,10 @@ void AliITSBeamTestDigSPD::Exec(Option_t* /*opt*/)
     for(Int_t nlad=1;nlad<=fITSgeom->GetNladders(nlay);nlad++){
       for(Int_t ndet=1;ndet<=fITSgeom->GetNdetectors(nlay);ndet++){
        Int_t index=fITSgeom->GetModuleIndex(nlay,nlad,ndet);
-       if(fITSgeom->GetModuleTypeName(index)=="kSPD") nspd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSDD") nsdd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSSD") nssd++;
+       TString dtype(fITSgeom->GetModuleTypeName(index));
+       if(dtype.Contains("SPD")) nspd++;
+       if(dtype.Contains("SDD")) nsdd++;
+       if(dtype.Contains("SSD")) nssd++;
       }
     }
   }
index 9c34753..35b6e48 100644 (file)
@@ -57,9 +57,10 @@ void AliITSBeamTestDigSSD::Exec(Option_t* /*opt*/) {
     for(Int_t nlad=1;nlad<=fITSgeom->GetNladders(nlay);nlad++){
       for(Int_t ndet=1;ndet<=fITSgeom->GetNdetectors(nlay);ndet++){
        Int_t index=fITSgeom->GetModuleIndex(nlay,nlad,ndet);
-       if(fITSgeom->GetModuleTypeName(index)=="kSPD") nspd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSDD") nsdd++;
-       if(fITSgeom->GetModuleTypeName(index)=="kSSD") nssd++;
+       TString dtype(fITSgeom->GetModuleTypeName(index));
+       if(dtype.Contains("SPD")) nspd++;
+       if(dtype.Contains("SDD")) nsdd++;
+       if(dtype.Contains("SSD")) nssd++;
       }
     }
   }
index 13dbdd2..a9de7da 100644 (file)
@@ -202,7 +202,7 @@ Int_t AliITSClusterFinderV2SPD::ClustersSPD(AliBin* bins, TClonesArray* digits,T
            if(TMath::Abs(iz-iiz)>0.75*idz) continue;
          }
          ndigits++;
-         Float_t qBin;
+         Float_t qBin=0.;
          if(rawdata) qBin = bins[idxBins[idx]].GetQ();
          if(!rawdata){
            AliITSdigitSPD* dig = (AliITSdigitSPD*)digits->UncheckedAt(idxBins[idx]);
index d408114..e99d203 100644 (file)
@@ -842,7 +842,7 @@ Bool_t AliITSHandleDaSSD::CalculateCM(AliITSModuleDaSSD *const module)
       }
       if ((n = AliITSModuleDaSSD::GetStripsPerChip() - ovstr)) cmsum /= (Float_t)(n);
       else cmsum = 0.0L;
-      if (!module->SetCM(cmsum, chipind, ev));
+      if (!(module->SetCM(cmsum, chipind, ev)));
     } 
   }
   return kTRUE; 
@@ -1221,8 +1221,8 @@ Bool_t AliITSHandleDaSSD::SaveEqSlotCalibrationData(const Int_t ddl, const Int_t
        AliError(Form("Can not open the file %s for output!", fname)); 
     return kFALSE;
   }
-  for (zsml = 0; fgkZsBitMask >> zsml; zsml++);
-  for (offsetml = 0; fgkOffSetBitMask >> offsetml; offsetml++);
+  for (zsml = 0; fgkZsBitMask >> zsml; zsml++) ;
+  for (offsetml = 0; fgkOffSetBitMask >> offsetml; offsetml++) ;
   for (Int_t strind = 0; strind < AliITSModuleDaSSD::GetStripsPerModuleConst(); strind++) {
     for (Int_t adcb = 0; adcb < fgkAdcPerDBlock; adcb++) {
       zsoffset = 0x0;
index 380cf84..f2bcc9a 100644 (file)
@@ -408,14 +408,14 @@ Bool_t AliITSModuleDaSSD::SetEventsNumber(const Long_t eventsnumber)
   Int_t i;
   if (!fStrips) return kFALSE;
   for (i = 0; i < fNumberOfStrips; i++) {
-    if (fStrips[i])  
+    if (fStrips[i]) { 
       if (!fStrips[i]->SetEvenetsNumber(eventsnumber)) {
         for (Int_t j = 0; j < i; j++) fStrips[j]->DeleteSignal();
-        AliError(Form("AliITSModuleDaSSD: Error allocating memory for i% events for module %i, strip %i", 
-                                   eventsnumber, (Int_t)fModuleId, i));
+        AliError(Form("AliITSModuleDaSSD: Error allocating memory for i% events for module %i, strip %i",eventsnumber, (Int_t)fModuleId, i));
         return kFALSE;
       }
-    else 
+    }
+    else { 
       if (!(fStrips[i] = new AliITSChannelDaSSD(i, eventsnumber))) {
         for (Int_t j = 0; j < i; j++) delete fStrips[j];
         delete [] fStrips;
@@ -424,6 +424,7 @@ Bool_t AliITSModuleDaSSD::SetEventsNumber(const Long_t eventsnumber)
         AliError(Form("AliITSModuleDaSSD: Error allocating memory for strip %i of module %i!", (Int_t)fModuleId, i));
         return kFALSE;
       }
+    }
   } 
   if (fCmFerom) {
     for (Int_t ie = 0; ie < fgkChipsPerModule; ie++) {
index 95cc318..8343dbe 100644 (file)
@@ -243,9 +243,14 @@ Int_t      AliITSPid::GetPcode(Float_t q,Float_t pm)
            else{return Proton();}
            } 
        }
-    if ( pm<=0.750 )
-       if ( q>fCutPr->Eval(pm)  )
-           {return Proton();} else {return Wpik(pm,q);};
+    if ( pm<=0.750 ){
+       if ( q>fCutPr->Eval(pm)  ){
+         return Proton();
+       }
+       else {
+         return Wpik(pm,q);
+       }
+    }
     if( pm<=1.10 ){ return Wpikp(pm,q); }
     return fPcode;    
 }
index 92a5a46..95c347d 100644 (file)
@@ -1018,7 +1018,7 @@ Bool_t AliITSPlaneEffSDD::WriteHistosToFile(TString filename, Option_t* option)
   // Saves the histograms into a tree and saves the trees into a file
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("WriteHistosToFile: null output filename!");
      return kFALSE;
   }
@@ -1105,7 +1105,7 @@ Bool_t AliITSPlaneEffSDD::ReadHistosFromFile(TString filename) {
   // Read histograms from an already existing file 
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("ReadHistosFromFile: incorrect output filename!");
      return kFALSE;
   }
index 1b32cbd..1ec8d5e 100644 (file)
@@ -849,7 +849,7 @@ Bool_t AliITSPlaneEffSPD::WriteHistosToFile(TString filename, Option_t* option)
   // Saves the histograms into a tree and saves the trees into a file
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("WriteHistosToFile: null output filename!");
      return kFALSE;
   }
@@ -934,7 +934,7 @@ Bool_t AliITSPlaneEffSPD::ReadHistosFromFile(TString filename) {
   // Read histograms from an already existing file 
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("ReadHistosFromFile: incorrect output filename!");
      return kFALSE;
   }
index 51cf673..08d58e4 100644 (file)
@@ -578,7 +578,7 @@ Bool_t AliITSPlaneEffSSD::WriteHistosToFile(TString filename, Option_t* option)
   // Saves the histograms into a tree and saves the trees into a file
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("WriteHistosToFile: null output filename!");
      return kFALSE;
   }
@@ -632,7 +632,7 @@ Bool_t AliITSPlaneEffSSD::ReadHistosFromFile(TString filename) {
   // Read histograms from an already existing file
   //
   if (!fHis) return kFALSE;
-  if (filename.Data()=="") {
+  if (filename.IsNull() || filename.IsWhitespace()) {
      AliWarning("ReadHistosFromFile: incorrect output filename!");
      return kFALSE;
   }
index a7b7947..c42e90d 100644 (file)
@@ -26,7 +26,7 @@ class AliITSVertexer : public AliVertexer {
     void FindMultiplicity(TTree *itsClusterTree);
     void SetFirstEvent(Int_t ev){fFirstEvent = ev;}
     void SetLastEvent(Int_t ev){fLastEvent = ev;}
-    const Float_t GetPipeRadius()const {return fgkPipeRadius;}
+    static Float_t GetPipeRadius() {return fgkPipeRadius;}
     void SetLaddersOnLayer2(Int_t ladwid=4);
     virtual void SetUseModule(Int_t imod, Bool_t optUse){
       if(imod>=0 && imod<kNSPDMod) fUseModule[imod]=optUse;
index 542cf13..1f7b7e3 100644 (file)
@@ -279,9 +279,15 @@ static void CheckLabels(Int_t lab[3]) {
        label=m;
        part=(TParticle*)gAlice->GetMCApp()->Particle(label);
       }
-      if      (lab[1]<0) lab[1]=label;
-      else if (lab[2]<0) lab[2]=label;
-      else ;//cerr<<"CheckLabels : No empty labels !\n";
+      if(lab[1]<0){
+       lab[1]=label;
+      }
+      else if (lab[2]<0) {
+       lab[2]=label;
+      }
+      else {
+       //      cerr<<"CheckLabels : No empty labels !\n";
+      }
     }
   }
 }
index 4c1fb79..96f4b13 100755 (executable)
@@ -28,7 +28,7 @@ class AliITStrackSA : public AliITStrackMI {
   Int_t GetClusterMark(Int_t layer,Int_t i) const {return fCluMark[layer][i];}
   Int_t GetNumberOfClustersSA() const {return fNSA;}
   Int_t GetNumberOfMarked(Int_t lay) const {return fNM[lay];}
-  static const Int_t GetMaxNumberOfClusters() {return kMaxNumberOfClusters;}
+  static Int_t GetMaxNumberOfClusters() {return kMaxNumberOfClusters;}
   Int_t GetMaxNMarkedPerLayer() const {return kMaxNumberOfClustersL;}
   void  AddClusterSA(Int_t layer, Int_t clnumb);
   void  AddClusterV2(Int_t layer,Int_t clnumb);
index 94afebd..c5b8807 100644 (file)
@@ -145,11 +145,22 @@ inline void  AliITStrackV2::SetModuleIndexInfo(Int_t ilayer,Int_t idet,Int_t sta
   Int_t xInt = Int_t(xloc*10.);
   Int_t zInt = Int_t(zloc*10.);
 
-  if(TMath::Abs(xloc*10.-(Float_t)xInt)>0.5)
-    if(zloc>0) { xInt++; } else { xInt--; }
-  if(TMath::Abs(zloc*10.-(Float_t)zInt)>0.5)
-    if(zloc>0) { zInt++; } else { zInt--; }
-
+  if(TMath::Abs(xloc*10.-(Float_t)xInt)>0.5){
+    if(zloc>0) { 
+      xInt++; 
+    } 
+    else { 
+      xInt--; 
+    }
+  }
+  if(TMath::Abs(zloc*10.-(Float_t)zInt)>0.5){
+    if(zloc>0) { 
+      zInt++; 
+    } 
+    else { 
+      zInt--; 
+    }
+  }
   Int_t signs=0;
   if(xInt>=0 && zInt>=0) signs=10000;
   if(xInt>=0 && zInt<0)  signs=20000;
index 1ec2c1b..85b1ccd 100644 (file)
@@ -4399,7 +4399,7 @@ void AliITStrackerMI::FindV02(AliESDEvent *event)
        if (fnorm0<0) fnorm0*=-3;
        Float_t fnorm1 = normdist[itrack1];
        if (fnorm1<0) fnorm1*=-3;
-       if (pvertex->GetAnglep()[2]>0.1 ||  (pvertex->GetRr()<10.5)&& pvertex->GetAnglep()[2]>0.05 || pvertex->GetRr()<3){
+       if ((pvertex->GetAnglep()[2]>0.1) || ( (pvertex->GetRr()<10.5)&& pvertex->GetAnglep()[2]>0.05 ) || (pvertex->GetRr()<3)){
          pb0    =  TMath::Exp(-TMath::Min(fnorm0,Float_t(16.))/12.);
          pb1    =  TMath::Exp(-TMath::Min(fnorm1,Float_t(16.))/12.);
        }
index ace560a..2737902 100644 (file)
@@ -332,7 +332,7 @@ Int_t AliITStrackerSA::FindTracks(AliESDEvent* event){
   if(AliITSReconstructor::GetRecoParam()->GetSAUseAllClusters()==kFALSE) {
     while (nentr--) {
       AliESDtrack *track=event->GetTrack(nentr);
-      if (track->GetStatus()&AliESDtrack::kITSin==AliESDtrack::kITSin){
+      if ((track->GetStatus()&AliESDtrack::kITSin) == AliESDtrack::kITSin){
        Int_t idx[12];
        Int_t ncl = track->GetITSclusters(idx);
        for(Int_t k=0;k<ncl;k++){