Correcting some trivial warnings on Alpha
authorhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 15 Jul 2003 14:44:02 +0000 (14:44 +0000)
committerhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 15 Jul 2003 14:44:02 +0000 (14:44 +0000)
EMCAL/AliEMCALJetFinder.cxx
EMCAL/AliEMCALRecParticle.cxx
HBTAN/AliHBTFunction.cxx
PHOS/AliPHOSPIDv1.cxx
STEER/AliConfig.cxx
STEER/AliLoader.h
THbtp/AliGenHBTprocessor.cxx
TOF/AliTOFReconstructionerV2.cxx
TPC/AliTPCPRF2D.cxx
TPC/AliTPCtrack.cxx

index b9c04fd..c391d8f 100644 (file)
@@ -1037,7 +1037,7 @@ void AliEMCALJetFinder::FillFromDigits(Int_t flag)
     
     if (!fLego) BookLego();
     if (flag == 0) fLego->Reset();
-    Int_t nbytes;
+    Int_t nbytes=0;
     
 
 //
index bb7aa8f..c101bad 100644 (file)
@@ -93,14 +93,11 @@ const TParticle * AliEMCALRecParticle::GetPrimary(Int_t index) const
              index, GetNPrimaries()) ;
     return 0 ; 
   } 
-  else { 
-    Int_t dummy ; 
-    AliEMCALGetter * gime = AliEMCALGetter::Instance() ; 
+  Int_t dummy ; 
+  AliEMCALGetter * gime = AliEMCALGetter::Instance() ; 
 
-    Int_t ecaRPindex = dynamic_cast<AliEMCALTrackSegment*>(gime->TrackSegments()->At(GetEMCALTSIndex()))->GetECAIndex();
-    Int_t primaryindex = dynamic_cast<AliEMCALTowerRecPoint*>(gime->ECARecPoints()->At(ecaRPindex))->GetPrimaries(dummy)[index] ; 
-    return gime->Primary(primaryindex) ;
-   } 
-  return 0 ; 
+  Int_t ecaRPindex = dynamic_cast<AliEMCALTrackSegment*>(gime->TrackSegments()->At(GetEMCALTSIndex()))->GetECAIndex();
+  Int_t primaryindex = dynamic_cast<AliEMCALTowerRecPoint*>(gime->ECARecPoints()->At(ecaRPindex))->GetPrimaries(dummy)[index] ; 
+  return gime->Primary(primaryindex) ;
 }
 
index 110a173..b93d514 100644 (file)
@@ -293,8 +293,8 @@ Double_t AliHBTFunction1D::Scale(TH1D* num,TH1D* den)
   
   if(fNBinsToScale < 1) 
    {
-    return 0.0;
     Error("Scale","Number of bins for scaling is smaller thnan 1");
+    return 0.0;
    }
   UInt_t nbins = num->GetNbinsX();
   if (fNBinsToScale > nbins) 
@@ -463,8 +463,8 @@ Double_t AliHBTFunction2D::Scale()
   
   if( (fNBinsToScaleX < 1) || (fNBinsToScaleY < 1) ) 
    {
-    return 0.0;
     Error("Scale","Number of bins for scaling is smaller thnan 1");
+    return 0.0;
    }
   UInt_t nbinsX = fNumerator->GetNbinsX();
   if (fNBinsToScaleX > nbinsX) 
@@ -654,8 +654,8 @@ Double_t AliHBTFunction3D::Scale()
   
   if( (fNBinsToScaleX < 1) || (fNBinsToScaleY < 1) || (fNBinsToScaleZ < 1)) 
    {
-    return 0.0;
     Error("Scale","Number of bins for scaling is smaller thnan 1");
+    return 0.0;
    }
   UInt_t nbinsX = fNumerator->GetNbinsX();
   if (fNBinsToScaleX > nbinsX) 
index ca775fc..2c554e3 100644 (file)
@@ -408,7 +408,7 @@ const Float_t  AliPHOSPIDv1::GetDistance(AliPHOSEmcRecPoint * emc,AliPHOSRecPoin
   return 100000000 ;
 }
 //____________________________________________________________________________
-const Int_t  AliPHOSPIDv1::GetCPVBit(AliPHOSEmcRecPoint * emc,AliPHOSRecPoint * cpv,const Int_t effPur, const Float_t e) const
+const Int_t  AliPHOSPIDv1::GetCPVBit(AliPHOSEmcRecPoint * emc,AliPHOSRecPoint * cpv,const Int_t effPur, Float_t e) const
 {
   if(effPur>2 || effPur<0)
     Error("GetCPVBit","Invalid Efficiency-Purity choice %d",effPur);
@@ -426,7 +426,7 @@ const Int_t  AliPHOSPIDv1::GetCPVBit(AliPHOSEmcRecPoint * emc,AliPHOSRecPoint *
 }
 
 //____________________________________________________________________________
-const Int_t  AliPHOSPIDv1::GetPrincipalBit(TString particle, const Double_t* p,const Int_t effPur, const Float_t e)const
+const Int_t  AliPHOSPIDv1::GetPrincipalBit(TString particle, const Double_t* p,const Int_t effPur, Float_t e)const
 {
   //Is the particle inside de PCA ellipse?
   
index 63331ca..3f2c2ff 100644 (file)
@@ -738,21 +738,15 @@ Int_t AliConfig::AddSubFolder(TFolder* topfolder, const char* infoler,
     }
    else return 0;//success
   }
- else
-  {//such an object already exists
-    TFolder* fol = dynamic_cast<TFolder*>(obj);
-    if (fol == 0x0)
-     {
-      Error("AddSubFodler(TFolder*, ....",
-             "Object named %s already exists in folder %s AND IT IS NOT A FOLDER",newfoldname,infoler);
-      return 3;
-     }
-    else
-     {
-      Warning("AddSubFodler(TFolder*, ....",
-             "Folder named %s already exists in folder %s",newfoldname,infoler);
-      return 0;
-     }
-  }
- return 0; //never reached
+ //such an object already exists
+ TFolder* fol = dynamic_cast<TFolder*>(obj);
+ if (fol == 0x0)
+   {
+     Error("AddSubFodler(TFolder*, ....",
+          "Object named %s already exists in folder %s AND IT IS NOT A FOLDER",newfoldname,infoler);
+     return 3;
+   }
+ Warning("AddSubFodler(TFolder*, ....",
+        "Folder named %s already exists in folder %s",newfoldname,infoler);
+ return 0;
 }
index 29a6dc5..43c927f 100644 (file)
@@ -114,9 +114,9 @@ class AliLoader: public TNamed
     TTree*         TreeT(){return GetTracksDataLoader()->Tree();}    //returns the tree from folder; shortcut method
     TTree*         TreeP(){return GetRecParticlesDataLoader()->Tree();} //returns the tree from folder; shortcut method
 
-    Int_t          LoadHits(Option_t* opt=""){return GetHitsDataLoader()->Load(opt);SetTAddrInDet();}
-    Int_t          LoadSDigits(Option_t* opt=""){return GetSDigitsDataLoader()->Load(opt);SetTAddrInDet();}
-    Int_t          LoadDigits(Option_t* opt=""){return GetDigitsDataLoader()->Load(opt);SetTAddrInDet();}
+    Int_t          LoadHits(Option_t* opt=""){Int_t status = GetHitsDataLoader()->Load(opt);SetTAddrInDet();return status;}
+    Int_t          LoadSDigits(Option_t* opt=""){Int_t status = GetSDigitsDataLoader()->Load(opt);SetTAddrInDet(); return status;}
+    Int_t          LoadDigits(Option_t* opt=""){Int_t status = GetDigitsDataLoader()->Load(opt);SetTAddrInDet();return status;}
     Int_t          LoadRecPoints(Option_t* opt=""){return GetRecPointsDataLoader()->Load(opt);}
     Int_t          LoadTracks(Option_t* opt=""){return GetTracksDataLoader()->Load(opt);}
     Int_t          LoadRecParticles(Option_t* opt=""){return GetRecParticlesDataLoader()->Load(opt);}
index 2ee260d..fe8257a 100644 (file)
@@ -352,9 +352,8 @@ Int_t AliGenHBTprocessor::GetHbtPStatusCode(Int_t part) const
     Error("GetHbtPStatusCode","GetTrackEventIndex returned error");
     return 0;
   }
- else return fHbtPStatCodes[ev][idx];
+ return fHbtPStatCodes[ev][idx];
   
- return 0;
 }
 
 /*******************************************************************/
@@ -923,12 +922,8 @@ TParticle* AliGenHBTprocessor::GetTrack(Int_t n)
     Error("GetTrack","GetTrackEventIndex returned error");
     return 0x0;
   }
- else
-  {
-    if (fgDebug > 5) Info("GetTrack","Number of Tracks in Event(%d) = %d",ev,cab->GetStack(ev)->GetNprimary());
-    return cab->GetStack(ev)->Particle(idx); //safe - in case stack does not exist 
-  }                                               //GetTrackEventIndex would have returned error
- return 0x0;  
+ if (fgDebug > 5) Info("GetTrack","Number of Tracks in Event(%d) = %d",ev,cab->GetStack(ev)->GetNprimary());
+ return cab->GetStack(ev)->Particle(idx);//safe - in case stack does not exist 
 }
 /*******************************************************************/
 
index 33e6093..8bfaca3 100644 (file)
@@ -924,9 +924,9 @@ Int_t AliTOFReconstructionerV2::SaveTracks(const Char_t *outname, const Int_t sp
   T.Write();
   savedir->cd();
   out->Close();
-  return 0;
   gBenchmark->Stop(name);
   gBenchmark->Show(name);
+  return 0;
 }
 
 //__________________________________________________________________
index 7ec8a35..0331fbb 100644 (file)
@@ -177,42 +177,39 @@ Float_t AliTPCPRF2D::GetPRF(Float_t xin, Float_t yin)
     fcharge   = &(fChargeArray[i*fNPRF]);
     return GetPRFActiv(xin);
   }
-  else{
-    //make interpolation from more fore lines
-    Int_t i= Int_t(y);
-    Float_t res;
-    if ((i<0) || (i>=fNYdiv) ) return 0;
-    Float_t z0=0;
-    Float_t z1=0;
-    Float_t z2=0;
-    Float_t z3=0;
-    if (i>0) {
-      fcharge =&(fChargeArray[(i-1)*fNPRF]);
-      z0 = GetPRFActiv(xin);
-    }
-    fcharge =&(fChargeArray[i*fNPRF]);
-    z1=GetPRFActiv(xin);
-    if ((i+1)<fNYdiv){
-      fcharge =&(fChargeArray[(i+1)*fNPRF]);
-      z2 = GetPRFActiv(xin);
-    }
-    if ((i+2)<fNYdiv){
-      fcharge =&(fChargeArray[(i+2)*fNPRF]);
-      z3 = GetPRFActiv(xin);
-    }
-    Float_t a,b,c,d,k,l;
-    a=z1;
-    b=(z2-z0)/2.;
-    k=z2-a-b;
-    l=(z3-z1)/2.-b;
-    d=l-2*k;
-    c=k-d;
-    Float_t dy=y-Float_t(i);
-
-    res = a+b*dy+c*dy*dy+d*dy*dy*dy;  
-    return res;            
-  }        
-  return 0.;
+  //make interpolation from more fore lines
+  Int_t i= Int_t(y);
+  Float_t res;
+  if ((i<0) || (i>=fNYdiv) ) return 0;
+  Float_t z0=0;
+  Float_t z1=0;
+  Float_t z2=0;
+  Float_t z3=0;
+  if (i>0) {
+    fcharge =&(fChargeArray[(i-1)*fNPRF]);
+    z0 = GetPRFActiv(xin);
+  }
+  fcharge =&(fChargeArray[i*fNPRF]);
+  z1=GetPRFActiv(xin);
+  if ((i+1)<fNYdiv){
+    fcharge =&(fChargeArray[(i+1)*fNPRF]);
+    z2 = GetPRFActiv(xin);
+  }
+  if ((i+2)<fNYdiv){
+    fcharge =&(fChargeArray[(i+2)*fNPRF]);
+    z3 = GetPRFActiv(xin);
+  }
+  Float_t a,b,c,d,k,l;
+  a=z1;
+  b=(z2-z0)/2.;
+  k=z2-a-b;
+  l=(z3-z1)/2.-b;
+  d=l-2*k;
+  c=k-d;
+  Float_t dy=y-Float_t(i);
+  
+  res = a+b*dy+c*dy*dy+d*dy*dy*dy;  
+  return res;            
 } 
 
 
index 1361aaa..94f170f 100644 (file)
@@ -139,7 +139,7 @@ AliTPCtrack::AliTPCtrack(const AliESDtrack& t) : AliKalmanTrack() {
   fC30=c[6 ];   fC31=c[7 ];   fC32=c32;   fC33=c[9 ];
   fC40=c[10];   fC41=c[11];   fC42=c42;   fC43=c[13]; fC44=c[14];
 
-  if (t.GetStatus()&AliESDtrack::kTIME == 0) return;
+  if ((t.GetStatus()&AliESDtrack::kTIME) == 0) return;
   StartTimeIntegral();
   Double_t times[10]; t.GetIntegratedTimes(times); SetIntegratedTimes(times);
   SetIntegratedLength(t.GetIntegratedLength());