- fixing warnings/coverity
authorodjuvsla <odjuvsla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 24 Feb 2011 14:11:48 +0000 (14:11 +0000)
committerodjuvsla <odjuvsla@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 24 Feb 2011 14:11:48 +0000 (14:11 +0000)
HLT/CALO/AliHLTCaloClusterizer.cxx
HLT/CALO/AliHLTCaloDigitMaker.cxx
HLT/CALO/AliHLTCaloProcessor.cxx
HLT/CALO/AliHLTCaloRecoParamHandler.cxx

index e354633..da12889 100644 (file)
@@ -84,6 +84,8 @@ AliHLTCaloClusterizer::AliHLTCaloClusterizer(TString det):
 AliHLTCaloClusterizer::~AliHLTCaloClusterizer()\r
 {\r
     //See header file for documentation\r
+  delete [] fRecPointDataPtr;\r
+  delete [] fRecPointArray;\r
 }\r
 \r
 void\r
@@ -322,5 +324,5 @@ Int_t
 AliHLTCaloClusterizer::CompareDigitsByEnergy(const void *dig0, const void *dig1)\r
 {\r
     // See header file for documentation\r
-    return (*((AliHLTCaloDigitDataStruct**)(dig1)))->fEnergy - (*((AliHLTCaloDigitDataStruct**)(dig0)))->fEnergy;\r
+  return (float)((*((AliHLTCaloDigitDataStruct**)(dig1)))->fEnergy - (*((AliHLTCaloDigitDataStruct**)(dig0)))->fEnergy);\r
 }\r
index ce6c934..7e28a94 100644 (file)
@@ -99,6 +99,12 @@ AliHLTCaloDigitMaker::AliHLTCaloDigitMaker(TString det) :
 AliHLTCaloDigitMaker::~AliHLTCaloDigitMaker() \r
 {\r
   //See header file for documentation\r
+  delete [] fHighGainFactors;\r
+  delete [] fLowGainFactors;\r
+  delete [] fBadChannelMask;  \r
+  delete [] fChannelBook;\r
+  delete fShmPtr;\r
+\r
 }\r
 \r
 Int_t\r
index caa0725..c47c58a 100644 (file)
@@ -81,37 +81,22 @@ AliHLTCaloProcessor::CheckFileLog(const char *origin,   const char *filename,  c
 
  
 void 
-AliHLTCaloProcessor::DoneWritingLog(const char *origin, const char *filename)
+AliHLTCaloProcessor::DoneWritingLog(const char */*origin*/, const char * /*filename*/)
 {
   //  char filepath[1024];
-  sprintf(fFilepath, "%s/%s", getenv("PWD"), filename);
-  Logging(kHLTLogInfo, origin , "finnished writing file" , "wrote file %s", fFilepath);
+  //  sprintf(fFilepath, "%s/%s", getenv("PWD"), filename);
+  //  Logging(kHLTLogInfo, origin , "finnished writing file" , "wrote file %s", fFilepath);
+
+  //  Logging(kHLTLogInfo, igin , "Don't use this function" , "Don't use this function", fFilepath);  
+  
 }
 
 
 void 
 AliHLTCaloProcessor::ScanRunNumberFromFile()
 {
-  char tmpDirectory[512];
-  char tmpFileName[512];
-  sprintf(tmpDirectory, "%s", getenv("HOME"));  
-
-  //TODO, remove hardcoded file path
-  
-  sprintf(tmpFileName, "%s%s", tmpDirectory, "/hlt/rundir/runNumber.txt");
-  int tmp = 0;
-  if(CheckFileLog( __FILE__ , tmpFileName , "r")== true) 
-    { 
-      FILE *fp = fopen(tmpFileName, "r");
-      tmp = fscanf(fp, "%d", &fRunNumber);
-      fclose(fp);
-    }
-
-  
-
-
- }
+  printf("AliHLTCaloProcessor::ScanRunNumberFromFile: don't use this function");
+}
 
 const char*
 AliHLTCaloProcessor::IntToChar(int number)
index 75232a7..4c6176d 100644 (file)
@@ -52,14 +52,17 @@ Int_t AliHLTCaloRecoParamHandler::GetParametersFromCDB()
       AliCDBEntry *pEntry = AliCDBManager::Instance()->Get(fRecoParamPath/*,GetRunNo()*/);
       if (pEntry) 
        {
+           
            TObjArray *paramArray = dynamic_cast<TObjArray*>(pEntry->GetObject());
-           fRecoParamPtr = dynamic_cast<AliDetectorRecoParam*>((paramArray)->At(0));
-         
+           if(paramArray)
+             {
+               fRecoParamPtr = dynamic_cast<AliDetectorRecoParam*>((paramArray)->At(0));
+             }
            if(!fRecoParamPtr)
-           {
-              HLTError("can not fetch object reconstruction parameters from \"%s\"", fRecoParamPath.GetPath().Data());
-              return -1;
-           }
+             {
+               HLTError("can not fetch object reconstruction parameters from \"%s\"", fRecoParamPath.GetPath().Data());
+               return -1;
+             }
        }
       else
        {