fix coverity
authorabercuci <abercuci@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 22 Dec 2011 10:30:09 +0000 (10:30 +0000)
committerabercuci <abercuci@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 22 Dec 2011 10:30:09 +0000 (10:30 +0000)
PWG1/TRD/AliTRDinfoGen.cxx
PWG1/TRD/AliTRDrecoTask.cxx
PWG1/TRD/AliTRDresolution.cxx
PWG1/TRD/info/AliTRDtrackInfo.cxx

index 2dca0a9..83da7eb 100644 (file)
@@ -668,27 +668,28 @@ void AliTRDinfoGen::UserExec(Option_t *){
     fTrackInfo->Delete("");\r
   }\r
   // read clusters REC info\r
-  TTree * treeR = (dynamic_cast<AliESDInputHandlerRP*>(fInputHandler))->GetTreeR("TRD");\r
-  if(treeR) {\r
-    TObjArray *recPoints(NULL);\r
-    if((treeR->GetBranch("TRDcluster"))){\r
-      treeR->SetBranchAddress("TRDcluster", &recPoints);\r
-      for(Int_t idet(0); idet<treeR->GetEntries(); idet++){\r
-        treeR->GetEntry(idet);\r
-        if(!recPoints->GetEntries()){\r
-          AliDebug(1, Form("Missing entry %d from TreeR", idet));\r
-          continue;\r
-        }\r
-        AliTRDcluster *c = (AliTRDcluster*)(*recPoints)[0];\r
-        if(!c){\r
-          AliDebug(1, Form("Missing first cluster in entry %d from TreeR", idet));\r
-          continue;\r
+  TTree * treeR(NULL);\r
+  if(fInputHandler && dynamic_cast<AliESDInputHandlerRP*>(fInputHandler)){\r
+    if((treeR = (dynamic_cast<AliESDInputHandlerRP*>(fInputHandler))->GetTreeR("TRD"))) {\r
+      TObjArray *recPoints(NULL);\r
+      if((treeR->GetBranch("TRDcluster"))){\r
+        treeR->SetBranchAddress("TRDcluster", &recPoints);\r
+        for(Int_t idet(0); idet<treeR->GetEntries(); idet++){\r
+          treeR->GetEntry(idet);\r
+          if(!recPoints->GetEntries()){\r
+            AliDebug(1, Form("Missing entry %d from TreeR", idet));\r
+            continue;\r
+          }\r
+          AliTRDcluster *c = (AliTRDcluster*)(*recPoints)[0];\r
+          if(!c){\r
+            AliDebug(1, Form("Missing first cluster in entry %d from TreeR", idet));\r
+            continue;\r
+          }\r
+          fClusters->AddAt(recPoints->Clone(Form("%03d", c->GetDetector())), c->GetDetector());\r
         }\r
-        fClusters->AddAt(recPoints->Clone(Form("%03d", c->GetDetector())), c->GetDetector());\r
-      }\r
-    } else AliDebug(3, "No TRDcluster branch");\r
-  } else AliDebug(3, "No RecPoints");\r
-\r
+      } else AliDebug(3, "No TRDcluster branch");\r
+    } else AliDebug(3, "No RecPoints");\r
+  } else AliDebug(3, "No AliESDInputHandlerRP");\r
 \r
 \r
   // LOOP 2 - over MC tracks which are passing TRD where the track is not reconstructed\r
index 9e84722..304ecde 100644 (file)
@@ -175,36 +175,36 @@ void AliTRDrecoTask::UserExec(Option_t *)
   if(!fTracks->GetEntriesFast()) return;\r
   else AliDebug(2, Form("Tracks[%d] for %s", fTracks->GetEntriesFast(), GetName()));\r
 \r
-  AliTRDtrackInfo *trackInfo = NULL;\r
+  Int_t itrk(-1);\r
+  AliTRDtrackInfo *trackInfo(NULL);\r
   TIter plotIter(fPlotFuncList);\r
   TObjArrayIter trackIter(fTracks);\r
   while((trackInfo = dynamic_cast<AliTRDtrackInfo*>(trackIter()))){\r
-    fkTrack = trackInfo->GetTrack();\r
+    itrk++; fPt=-1; fEta=0.; fPhi=0.; fSpecies=-6;\r
     fkMC    = trackInfo->GetMCinfo();\r
     fkESD   = trackInfo->GetESDinfo();\r
-    // cache properties of the track at TRD entrance\r
-    // check input track status\r
-    AliExternalTrackParam *tin(NULL);\r
-    fPt=-1; fEta=0.; fPhi=0.; fSpecies=-6;\r
-    if(!fkTrack || !(tin = fkTrack->GetTrackIn())) AliDebug(2, "Track did not entered TRD fiducial volume.");\r
-    else {\r
-      fPt   = tin->Pt();\r
-      fEta  = tin->Eta();\r
-      Double_t xyz[3];\r
-      if(!tin->GetXYZ(xyz)) AliDebug(2, "Failed getting global track postion");\r
-      else fPhi  = TMath::ATan2(xyz[1], xyz[0]);\r
-      fSpecies= fkTrack->Charge()*(AliTRDpidUtil::Mass2Pid(fkTrack->GetMass())+1);\r
-    }\r
-\r
-    TMethodCall *plot = NULL;\r
+    if((fkTrack = trackInfo->GetTrack())){\r
+      // cache properties of the track at TRD entrance\r
+      // check input track status\r
+      AliExternalTrackParam *tin(NULL);\r
+      if(!(tin = fkTrack->GetTrackIn())) AliDebug(2, Form("Missing TRD track[%d] :: entry point.", itrk));\r
+      else {\r
+        fPt   = tin->Pt();\r
+        fEta  = tin->Eta();\r
+        Double_t xyz[3];\r
+        if(!tin->GetXYZ(xyz)) AliDebug(2, Form("Failed TRD track[%d] :: global track postion", itrk));\r
+        else fPhi  = TMath::ATan2(xyz[1], xyz[0]);\r
+        fSpecies= fkTrack->Charge()*(AliTRDpidUtil::Mass2Pid(fkTrack->GetMass())+1);\r
+      }\r
+    } else AliDebug(2, Form("Missing TRD track[%d].", itrk));\r
+\r
+    TMethodCall *plot(NULL);\r
     plotIter.Reset();\r
-    while((plot=dynamic_cast<TMethodCall*>(plotIter()))){\r
-      plot->Execute(this);\r
-    }\r
+    while((plot=dynamic_cast<TMethodCall*>(plotIter()))) plot->Execute(this);\r
   }\r
   if(!fClusters) return;\r
   if(!fDetFuncList){\r
-    AliDebug(1, "No detector functor list defined for the task");\r
+    AliDebug(1, "No detector functor list defined for task");\r
     return;\r
   }\r
   TIter detIter(fDetFuncList);\r
@@ -212,9 +212,7 @@ void AliTRDrecoTask::UserExec(Option_t *)
     if(!(fkClusters = (TObjArray*)fClusters->At(idet))) continue;\r
     TMethodCall *det(NULL);\r
     detIter.Reset();\r
-    while((det=dynamic_cast<TMethodCall*>(detIter()))){\r
-      det->Execute(this);\r
-    }\r
+    while((det=dynamic_cast<TMethodCall*>(detIter()))) det->Execute(this);\r
   }\r
 }\r
 \r
index 27a4aba..18cf19f 100644 (file)
@@ -470,7 +470,7 @@ TH1* AliTRDresolution::PlotTracklet(const AliTRDtrackV1 *track)
     return NULL;
   }
 
-  const Int_t ndim(kNdim+7);
+  const Int_t ndim(kNdim+8);
   Double_t val[ndim],
            alpha(0.), cs(-2.), sn(0.);
   Float_t sz[AliTRDseedV1::kNtb], pos[AliTRDseedV1::kNtb];
@@ -1406,12 +1406,12 @@ Bool_t AliTRDresolution::MakeProjectionCluster(Bool_t mc)
     AliError(Form("Missing/Wrong data @ %s.", projName[Int_t(mc)]));
     return kFALSE;
   }
-  Int_t ndim(H->GetNdimensions()); Bool_t debug(ndim>kNdimCl);
+  Int_t ndim(H->GetNdimensions()); Bool_t debug(ndim>Int_t(kNdimCl));
   Int_t coord[kNdim]; memset(coord, 0, sizeof(Int_t) * kNdim); Double_t v = 0.;
   TAxis *aa[kNdim], *as(NULL), *apt(NULL); memset(aa, 0, sizeof(TAxis*) * kNdim);
   for(Int_t id(0); id<ndim; id++) aa[id] = H->GetAxis(id);
-  if(ndim > kPt) apt = H->GetAxis(kPt);
-  if(ndim > kSpeciesChgRC) as  = H->GetAxis(kSpeciesChgRC);
+  if(ndim > Int_t(kPt)) apt = H->GetAxis(kPt);
+  if(ndim > Int_t(kSpeciesChgRC)) as  = H->GetAxis(kSpeciesChgRC);
   // build list of projections
   const Int_t nsel(12);
   // define rebinning strategy
@@ -1518,18 +1518,18 @@ Bool_t AliTRDresolution::MakeProjectionTracklet(Bool_t mc)
     return kFALSE;
   }
   const Int_t mdim(kNdim+8);
-  Int_t ndim(H->GetNdimensions());
+  Int_t ndim(H->GetNdimensions()); Bool_t debug(ndim>Int_t(kNdimTrklt));
   Int_t coord[mdim]; memset(coord, 0, sizeof(Int_t) * mdim); Double_t v = 0.;
   TAxis *aa[mdim], *as(NULL), *ap(NULL), *ac(NULL); memset(aa, 0, sizeof(TAxis*) * mdim);
   for(Int_t id(0); id<ndim; id++) aa[id] = H->GetAxis(id);
-  if(ndim > kSpeciesChgRC) as = H->GetAxis(kSpeciesChgRC); // init species/charge selection
-  if(ndim > kPt) ap = H->GetAxis(kPt);                     // init pt selection
-  if(ndim > kNdim+1) ac = H->GetAxis(kNdim+1);             // init centrality selection
+  if(ndim > Int_t(kSpeciesChgRC)) as = H->GetAxis(kSpeciesChgRC); // init species/charge selection
+  if(ndim > Int_t(kPt))           ap = H->GetAxis(kPt);           // init pt selection
+  if(ndim > Int_t(kNdim)+1)       ac = H->GetAxis(kNdim+1);       // init centrality selection
   // calculate size depending on debug level
-  const Int_t nCen(ndim>kNdimTrklt?AliTRDeventInfo::kCentralityClasses:1);
-  const Int_t nPt(ndim>kNdimTrklt?kNpt:1);
+  const Int_t nCen(debug?Int_t(AliTRDeventInfo::kCentralityClasses):1);
+  const Int_t nPt(debug?Int_t(kNpt):1);
   const Int_t nSpc(1);//ndim>kNdimTrklt?fgkNbins[kSpeciesChgRC]:1);
-  const Int_t nCh(ndim>kNdimTrklt?kNcharge:1);
+  const Int_t nCh(debug?Int_t(kNcharge):1);
 
   // build list of projections
   const Int_t nsel(AliTRDeventInfo::kCentralityClasses*AliTRDgeometry::kNlayer*kNpt*(AliPID::kSPECIES*kNcharge + 1));
@@ -1834,12 +1834,12 @@ Bool_t AliTRDresolution::MakeProjectionTrackIn(Bool_t mc)
   Int_t ndim(H->GetNdimensions());
   TAxis *aa[mdim], *as(NULL), *ap(NULL), *ax(NULL), *abf(NULL); memset(aa, 0, sizeof(TAxis*) * (mdim));
   for(Int_t id(0); id<ndim; id++) aa[id] = H->GetAxis(id);
-  if(ndim > kSpeciesChgRC) as = H->GetAxis(kSpeciesChgRC);
-  if(ndim > kPt)           ap = H->GetAxis(kPt);
-  if(ndim > (kNdim+1))     ax = H->GetAxis(kNdim+1);
-  if(ndim > (kNdim+2))    abf = H->GetAxis(kNdim+2);
+  if(ndim > Int_t(kSpeciesChgRC)) as = H->GetAxis(kSpeciesChgRC);
+  if(ndim > Int_t(kPt))           ap = H->GetAxis(kPt);
+  if(ndim > Int_t(kNdim)+1)       ax = H->GetAxis(kNdim+1);
+  if(ndim > Int_t(kNdim)+2)      abf = H->GetAxis(kNdim+2);
   //AliInfo(Form("Using : Species[%c] Pt[%c] BunchFill[%c]", as?'y':'n', ap?'y':'n', abf?'y':'n'));
-  const Int_t nPt(ndim>kNdimTrkIn?kNpt:1);
+  const Int_t nPt(ndim>Int_t(kNdimTrkIn)?Int_t(kNpt):1);
 
   // build list of projections
   const Int_t nsel(kNpt*(AliPID::kSPECIES*kNcharge + 1));
index 32eb107..89c4198 100644 (file)
@@ -308,7 +308,7 @@ AliTRDtrackInfo::AliESDinfo& AliTRDtrackInfo::AliESDinfo::operator=(const AliESD
   fTRDpidQuality= esd.fTRDpidQuality;
   fTRDnSlices  = esd.fTRDnSlices;
   fTRDslices   = NULL;
-
+  
   memcpy(fTRDr, esd.fTRDr, AliPID::kSPECIES*sizeof(Double32_t));
   memcpy(fTRDv0pid, esd.fTRDv0pid, AliPID::kSPECIES*sizeof(Int_t));