Coverity fixes
authorpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 7 Apr 2011 07:48:19 +0000 (07:48 +0000)
committerpulvir <pulvir@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 7 Apr 2011 07:48:19 +0000 (07:48 +0000)
PWG2/RESONANCES/AliRsnAnalysisTask.cxx
PWG2/RESONANCES/AliRsnAnalysisTask.h
PWG2/RESONANCES/AliRsnCutPIDNSigma.cxx
PWG2/RESONANCES/AliRsnCutPIDTPC.cxx
PWG2/RESONANCES/AliRsnLoopEffPair.cxx
PWG2/RESONANCES/extra/AddAnalysisTaskResonanceQA.C
PWG2/RESONANCES/extra/AliAnalysisTaskResonanceQA.cxx
PWG2/RESONANCES/extra/AliAnalysisTaskResonanceQA.h

index 95315e0..a9b3c7a 100644 (file)
@@ -91,7 +91,7 @@ AliRsnAnalysisTask::~AliRsnAnalysisTask()
 }
 
 //__________________________________________________________________________________________________
-void AliRsnAnalysisTask::Add(AliRsnLoop *obj)
+void AliRsnAnalysisTask::AddLoop(AliRsnLoop *obj)
 {
 //
 // Add new computation object
@@ -195,18 +195,21 @@ void AliRsnAnalysisTask::UserExecMix(Option_t*)
 
    // gets first input handler form mixing buffer
    AliMultiInputEventHandler *ihMultiMix = dynamic_cast<AliMultiInputEventHandler*>(fInputEHMix->InputEventHandler(0));
-   rsnMixIH = dynamic_cast<AliRsnInputHandler*>(ihMultiMix->InputEventHandler(id));
-   evMix = rsnMixIH->GetRsnEvent();
-   
-   if (!evMix) return;
-
-   TObjArrayIter next(&fRsnObjects);
-   AliRsnLoop *obj = 0x0;
-   while ( (obj = (AliRsnLoop*)next()) ) {
-      if (!obj->IsMixed()) continue;
-      obj->DoLoop(evMain, rsnIH->GetSelector(), evMix, rsnMixIH->GetSelector());
+   if (ihMultiMix) {
+      rsnMixIH = dynamic_cast<AliRsnInputHandler*>(ihMultiMix->InputEventHandler(id));
+      if (rsnMixIH) {
+         evMix = rsnMixIH->GetRsnEvent();
+         if (!evMix) return;
+
+         TObjArrayIter next(&fRsnObjects);
+         AliRsnLoop *obj = 0x0;
+         while ( (obj = (AliRsnLoop*)next()) ) {
+            if (!obj->IsMixed()) continue;
+            obj->DoLoop(evMain, rsnIH->GetSelector(), evMix, rsnMixIH->GetSelector());
+         }
+      }
    }
-
+   
    PostData(1, fOutput);
 }
 
index ad06a9a..520affe 100644 (file)
@@ -27,8 +27,8 @@ public:
    virtual void     UserExecMix(Option_t*);
    virtual void     Terminate(Option_t *);
    
-   virtual void     Add(AliRsnLoop *object);
-   void InitInputHandlers();
+   void             AddLoop(AliRsnLoop *object);
+   void             InitInputHandlers();
    
 private:
 
index 7e5bdde..121bd12 100644 (file)
@@ -167,17 +167,17 @@ void AliRsnCutPIDNSigma::Print(const Option_t *) const
 // Print information on this cut
 //
 
-   Char_t mom[100], det[10], match[100];
+   Char_t mom[200], det[10], match[200];
    
    if (fRejectOutside)
-      sprintf(mom, "Tracks are accepted only in the momentum range %.2f --> %.2f", fMomMin, fMomMax);
+      snprintf(mom, 200, "Tracks are accepted only in the momentum range %.2f --> %.2f", fMomMin, fMomMax);
    else
-      sprintf(mom, "No check in momentum range");
+      snprintf(mom, 200, "No check in momentum range");
       
    if (fRejectUnmatched)
-      sprintf(match, "Unmatched tracks are rejected");
+      snprintf(match, 200, "Unmatched tracks are rejected");
    else
-      sprintf(match, "No check on track matching");
+      snprintf(match, 200, "No check on track matching");
       
    switch (fDetector) {
       case kITS: sprintf(det, "ITS"); break;
index abcf06e..11b7483 100644 (file)
@@ -39,6 +39,8 @@ AliRsnCutPIDTPC::AliRsnCutPIDTPC
 //
 // Main constructor.
 //
+
+   fBB[0] = fBB[1] = fBB[2] = fBB[3] = fBB[4] = 0.0;
 }
 
 //_________________________________________________________________________________________________
@@ -55,6 +57,9 @@ AliRsnCutPIDTPC::AliRsnCutPIDTPC
 //
 // Copy constructor.
 //
+
+   Int_t i;
+   for (i = 0; i < 5; i++) fBB[i] = copy.fBB[i];
 }
 
 //_________________________________________________________________________________________________
@@ -72,6 +77,9 @@ AliRsnCutPIDTPC& AliRsnCutPIDTPC::operator=(const AliRsnCutPIDTPC& copy)
    fRefType       = copy.fRefType;
    fESDpid        = copy.fESDpid;
    fAODpid        = copy.fAODpid;
+   
+   Int_t i;
+   for (i = 0; i < 5; i++) fBB[i] = copy.fBB[i];
 
    return (*this);
 }
index 13b360b..366e7dc 100644 (file)
@@ -9,6 +9,7 @@
 
 #include "AliStack.h"
 
+#include "AliRsnEvent.h"
 #include "AliRsnCutManager.h"
 #include "AliRsnDaughterDef.h"
 #include "AliRsnPairDef.h"
@@ -75,14 +76,13 @@ Int_t AliRsnLoopEffPair::DoLoop(AliRsnEvent *rsn, AliRsnDaughterSelector*, AliRs
       AliError("Need to process ESD or AOD input");
       return 0;
    }
-   Bool_t isESD = rsn->IsESD();
    
    // additional coherence checks
    AliVEvent    *mcEvent = 0x0;
    AliStack     *listESD = 0x0;
    TClonesArray *listAOD = 0x0;
    Int_t         npart   = 0;
-   if (isESD) {
+   if (rsn->IsESD()) {
       mcEvent = rsn->GetRefMCESD();
       listESD = ((AliMCEvent*)mcEvent)->Stack();
       if (!listESD) {
@@ -126,7 +126,7 @@ Int_t AliRsnLoopEffPair::DoLoop(AliRsnEvent *rsn, AliRsnDaughterSelector*, AliRs
       
       // get next particle and take some quantities
       // in different way from ESD and AOD MC
-      if (isESD) {
+      if (rsn->IsESD()) {
          mother = mcEvent->GetTrack(ipart);
          motherESD = static_cast<AliMCParticle*>(mother);
          pdg = TMath::Abs(motherESD->Particle()->GetPdgCode());
@@ -155,7 +155,7 @@ Int_t AliRsnLoopEffPair::DoLoop(AliRsnEvent *rsn, AliRsnDaughterSelector*, AliRs
       // then, if both daughters are not 'good', skip the pair
       fDaughter[0].Reset();
       fDaughter[1].Reset();
-      if (isESD) {
+      if (rsn->IsESD()) {
          dindex[0] = motherESD->Particle()->GetFirstDaughter();
          dindex[1] = motherESD->Particle()->GetLastDaughter();
       } else {
@@ -173,8 +173,7 @@ Int_t AliRsnLoopEffPair::DoLoop(AliRsnEvent *rsn, AliRsnDaughterSelector*, AliRs
             break;
          }
          // retrieve daughter and copy some info
-         label = TMath::Abs(daughter->GetLabel());
-         if (isESD) {
+         if (rsn->IsESD()) {
             daughter = mcEvent->GetTrack(dindex[id]);
             daughterESD = static_cast<AliMCParticle*>(daughter);
             pdg = TMath::Abs(daughterESD->Particle()->GetPdgCode());
@@ -185,6 +184,7 @@ Int_t AliRsnLoopEffPair::DoLoop(AliRsnEvent *rsn, AliRsnDaughterSelector*, AliRs
             pdg = TMath::Abs(daughterAOD->GetPdgCode());
             charge = (Short_t)daughterAOD->Charge();
          }
+         label = TMath::Abs(daughter->GetLabel());
          // check if can be assigned
          imatch = -1;
          if ( fDef->GetDef1().MatchesPDG(pdg) && fDef->GetDef1().MatchesChargeS(charge) )
index 7b6208e..f084773 100644 (file)
@@ -37,6 +37,7 @@ AliAnalysisTaskResonanceQA::AliAnalysisTaskResonanceQA(const char *name) :
    AliAnalysisTaskSE(name), 
    fT0(AliESDpid::kTOF_T0),
    fPrimaryThr(1E-6),
+   fVz(10.0),
    fOutputList(0),
    fSelectedEvts(0),
    fdEdxTPC(0),
@@ -283,7 +284,7 @@ void AliAnalysisTaskResonanceQA::UserExec(Option_t *)
          if (v0->GetNContributors() < 1) return;
       }
       if (!v0) return;
-      if (TMath::Abs(v0->GetZv()) > 10) return;
+      if (TMath::Abs(v0->GetZv()) > fVz) return;
    
       // settings for TOF time zero
       if (fESD->GetTOFHeader()) 
index 81d4ba4..2c638b0 100644 (file)
@@ -42,6 +42,7 @@ public:
    
    void           SetT0(AliESDpid::EStartTimeType_t ftype) {fT0 = ftype;}
    void           SetPrimaryThr(Double_t d) {fPrimaryThr = d;}
+   void           SetVz(Double_t vz)        {fVz = vz;}
    
    const char*    RsnName  (Int_t type) {return RsnName  ((ERsn)type);}
    const char*    RsnSymbol(Int_t type) {return RsnSymbol((ERsn)type);}
@@ -54,6 +55,7 @@ private:
 
    AliESDpid::EStartTimeType_t fT0;
    Double_t fPrimaryThr;
+   Double_t fVz;
    
    TList *fOutputList;