attempt to address new coverity reports for AliCalo stuff and QA checker
authordsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Nov 2011 15:16:54 +0000 (15:16 +0000)
committerdsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Nov 2011 15:16:54 +0000 (15:16 +0000)
EMCAL/AliCaloFitSubarray.cxx
EMCAL/AliCaloRawAnalyzerPeakFinder.cxx
EMCAL/AliCaloRawAnalyzerPeakFinder.h
EMCAL/AliEMCALQAChecker.cxx

index bf75654..ddbc845 100644 (file)
@@ -37,14 +37,6 @@ AliCaloFitSubarray::AliCaloFitSubarray(const Int_t bunchIndex,
 {
 }
 
-AliCaloFitSubarray::AliCaloFitSubarray(const AliCaloFitSubarray & fitS) :
-  fBunchIndex( fitS.fBunchIndex ),
-  fMaxRev( fitS.fMaxRev ), 
-  fFirst( fitS.fFirst ), 
-  fLast( fitS.fLast )   
-{
-}
-
 AliCaloFitSubarray::AliCaloFitSubarray(const Int_t init) : 
   fBunchIndex(init),
   fMaxRev(init), 
@@ -53,6 +45,13 @@ AliCaloFitSubarray::AliCaloFitSubarray(const Int_t init) :
 {
 }
 
+AliCaloFitSubarray::AliCaloFitSubarray(const AliCaloFitSubarray & fitS) :
+  fBunchIndex( fitS.fBunchIndex ),
+  fMaxRev( fitS.fMaxRev ), 
+  fFirst( fitS.fFirst ), 
+  fLast( fitS.fLast )   
+{
+}
 
 AliCaloFitSubarray::~AliCaloFitSubarray()
 {
index 24f7df1..6b7771d 100644 (file)
@@ -267,72 +267,6 @@ AliCaloRawAnalyzerPeakFinder::LoadVectorsOCDB()
 }
 
 
-void 
-AliCaloRawAnalyzerPeakFinder::LoadVectorsASCII()
-{
-  //Read in the Peak finder vecors from ASCI files
-  fIsInitialized= true;  
-  const Int_t buffersize = 256;
-  for(int i = 0;  i < PF::MAXSTART ; i++)
-  {
-    for( int j=0; j < PF::SAMPLERANGE; j++)
-    {
-      char filenameCoarse[buffersize];
-      char filename[buffersize];
-      int n = j+fNsampleCut;
-      double start = (double)i+0;
-      
-      snprintf(filename, buffersize,       "%s/EMCAL/vectors-emcal/start%.1fN%dtau0.235fs10dt1.0.txt", getenv("ALICE_ROOT"), start, n);
-      snprintf(filenameCoarse, buffersize, "%s/EMCAL/vectors-emcal/start%.1fN%dtau0.235fs10dt3.0.txt", getenv("ALICE_ROOT"), start, n);
-      
-      FILE *fp  =  fopen(filename, "r");
-      FILE *fpc =  fopen(filenameCoarse, "r");
-
-      int status = 0;
-      int statusc = 0;
-      
-      if( fp == 0 )
-           {
-             AliFatal( Form( "could not open file: %s", filename ) );
-           }
-      else if(fpc == 0)
-           {
-             AliFatal( Form( "could not open file: %s", filenameCoarse ) );
-           }
-      else
-           {
-             for(int m = 0; m < n ; m++ )
-               {
-                 status = fscanf(fp,  "%lf\t", &fPFAmpVectors[i][j][m] );
-                 statusc = fscanf(fpc, "%lf\t", &fPFAmpVectorsCoarse[i][j][m] );
-                 if (!status) { AliFatal( Form( "could not read file: %s", filename ) ); }
-                 if (!statusc) { AliFatal( Form( "could not read file: %s", filenameCoarse ) ); }
-               }
-             status = fscanf(fp,   "\n" );
-             statusc = fscanf(fpc,  "\n" );
-             if (status < 0) { AliFatal( Form( "could not read file: %s", filename ) ); }
-             if (statusc < 0) { AliFatal( Form( "could not read file: %s", filenameCoarse ) ); }
-             for(int m = 0; m < n ; m++ )
-               {
-                 status = fscanf(fp, "%lf\t",   &fPFTofVectors[i][j][m]  );
-                 statusc = fscanf(fpc, "%lf\t",  &fPFTofVectorsCoarse[i][j][m]  );  
-                 if (!status) { AliFatal( Form( "could not read file: %s", filename ) ); }
-                 if (!statusc) { AliFatal( Form( "could not read file: %s", filenameCoarse ) ); }
-               }
-             
-             fPeakFinderVectors->SetVector( i, j, fPFAmpVectors[i][j], fPFTofVectors[i][j],    
-                                      fPFAmpVectorsCoarse[i][j], fPFTofVectorsCoarse[i][j] );   
-        
-           }
-      
-      if(fp) fclose (fp );
-      if(fpc)fclose (fpc);
-      
-    }
-  }
-}
-
-
 void   
 AliCaloRawAnalyzerPeakFinder::WriteRootFile() const
 {
index fff6985..8276e2e 100644 (file)
@@ -42,7 +42,6 @@ class  AliCaloRawAnalyzerPeakFinder : public AliCaloRawAnalyzer
   AliCaloRawAnalyzerPeakFinder();
   AliCaloRawAnalyzerPeakFinder( const AliCaloRawAnalyzerPeakFinder   & );
   AliCaloRawAnalyzerPeakFinder   & operator = ( const  AliCaloRawAnalyzerPeakFinder  & );
-  void     LoadVectorsASCII();
   void     LoadVectorsOCDB();
   void     CopyVectors(const AliCaloPeakFinderVectors *const pfvectors );
   void     ResetVectors();
index e455d41..c30c407 100644 (file)
@@ -112,6 +112,7 @@ fText(new TPaveText(0.2,0.7,0.8,0.9,"NDC"))
 //__________________________________________________________________
 AliEMCALQAChecker& AliEMCALQAChecker::operator = (const AliEMCALQAChecker &qac) 
 {
+  AliQACheckerBase(qac.GetName(), qac.GetTitle()); 
   fTextSM  = new TText*[fknSM] ;
   fLineCol = static_cast<TLine*>(qac.fLineCol->Clone()) ; 
   fText    = new TPaveText(0.2,0.7,0.8,0.9,"NDC") ;
@@ -209,7 +210,11 @@ void AliEMCALQAChecker::CheckRaws(Double_t * test, TObjArray ** list)
          TObject *obj;
          while ((obj = lstF->First())) { while(lstF->Remove(obj)) { } delete obj; }
          if (stats) lstF->Add(stats);
-       } 
+       }
+       else {
+         AliWarning(Form("Checker : empty list of data functions; returning"));
+         return;
+       }
       }
       lstF->Add(fLineCol->Clone()); 
       for(Int_t iLine = 0; iLine < 4; iLine++) {
@@ -227,6 +232,10 @@ void AliEMCALQAChecker::CheckRaws(Double_t * test, TObjArray ** list)
          TObject *obj;
          while ((obj = lstF->First())) { while(lstF->Remove(obj)) { } delete obj; }
          if (stats) lstF->Add(stats);
+       }
+       else {
+         AliWarning(Form("Checker : empty list of ratio functions; returning"));
+         return;
        } 
       }
       lstF->Add(fText->Clone()) ;