Coverity: Functions concerned was actually not use, and was deleted.
authorphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 28 Apr 2011 10:48:40 +0000 (10:48 +0000)
committerphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 28 Apr 2011 10:48:40 +0000 (10:48 +0000)
In addition: Refactoring + removal of unused code

HLT/CALO/AliHLTCaloProcessor.cxx
HLT/CALO/AliHLTCaloProcessor.h

index 31da68e..ab0660e 100644 (file)
  **************************************************************************/
 
 #include "AliHLTCaloProcessor.h"
-#include "unistd.h"
-
-
 
 const AliHLTComponentDataType AliHLTCaloProcessor::fgkInputDataTypes[]={kAliHLTVoidDataType,{0,"",""}}; //'zero' terminated array
 
-
 AliHLTCaloProcessor::AliHLTCaloProcessor():AliHLTProcessor(), 
-                                          fCaloEventCount(0), 
-                                          fModuleID(2), 
-                                          fPrintInfoModule(0), 
-                                          fPrintInfoFrequncyModule(1000), 
-                                          fRunNumber(0) 
+                                          fCaloEventCount(0)
 {
-  //  ScanRunNumberFromFile();
   lineNumber[0] = '\0';
-  fFilepath[0] = '\0';
-  fMessage[0] = '\0';
 }
 
 
-
 AliHLTCaloProcessor::~AliHLTCaloProcessor()
 {
 
 }
 
-bool 
-AliHLTCaloProcessor::CheckFileLog(const char */*origin*/,   const char */*filename*/,  const char */*opt*/)
-{
-  // sprintf(fFilepath, "%s/%s", getenv("PWD"), filename);
-  // FILE *fp = fopen(filename, opt);
-
-  // if(fp == 0)
-  //   {
-  //     //      if( (opt == "w")  || (opt == "a")) \\OD
-  //     if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
-  //   {
-  //     sprintf(fMessage, "for writing  please check that the directory exists and that you have write access to it"  );
-  //   }
-  //     else
-  //   {
-  //     sprintf(fMessage, "for reading  please check that the directory exists and that you have read access to it"  );
-  //   }
-  //    Logging(kHLTLogFatal, origin , "cannot open file" , "Was not able to open file %s  %s", fFilepath, fMessage);
-  //    return false;
-  //   }
-  // else
-  //   {
-  //     //      if( (opt == "w")  || (opt == "a")) \\OD
-  //     if( (!strcmp(opt,"w"))  || (!strcmp(opt,"a")))
-  //   {
-  //     sprintf(fMessage, "for writing" );
-  //   }
-  //     else
-  //   {
-  //     sprintf(fMessage, "for reading");
-  //   }
-  //     //    Logging(kHLTLogInfo, origin , "opening file" , "Sucessfully opening %s  %s", fFilepath, fMessage);
-  //     fclose(fp); 
-  //     return true;
-  //   }
-  return false;
-}
-
-void 
-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);
-
-  //  Logging(kHLTLogInfo, igin , "Don't use this function" , "Don't use this function", fFilepath);  
-  
-}
-
-
-void 
-AliHLTCaloProcessor::ScanRunNumberFromFile()
-{
-  printf("AliHLTCaloProcessor::ScanRunNumberFromFile: don't use this function");
-}
 
 const char*
 AliHLTCaloProcessor::IntToChar(int number)
 {
-  sprintf(lineNumber,"%d", number);
+  // comment
+  snprintf(lineNumber, 256, "%d", number);
   return lineNumber;
 }
 
-
-
-int
-AliHLTCaloProcessor::ScanArgumentsModule(int argc, const char** argv)
-{
-  fPrintInfoModule = kFALSE;
-  int iResult=0;
-  TString argument="";
-
-  for(int i=0; i<argc && iResult>=0; i++) 
-    {
-      argument=argv[i];
-      
-      if (argument.IsNull()) 
-       {
-         continue;
-       }
-                         
-    if (argument.CompareTo("-printinfo") == 0) 
-      {
-       if(i+1 <= argc)
-         {
-           argument=argv[i+1];
-           fPrintInfoFrequncyModule = atoi(argv[i+1]);
-           fPrintInfoModule = kTRUE;
-         }
-       else
-         {
-           Logging(kHLTLogWarning, __FILE__ , "inconsistency during init" , "asking for event info, but no update frequency is specified, option is ignored");
-         }
-      }
-    }
-  return 0;
-}
-
index 501f35e..7131869 100644 (file)
@@ -7,15 +7,8 @@
  * See cxx source for full Copyright notice                               */
 
 #include "AliHLTProcessor.h"
-//#include "AliHLTCaloBase.h"
-// #include "AliHLTCaloDefinitions.h"
-#include "AliHLTDataTypes.h"
 
-// #include "AliHLTCaloConstant.h"
 
-// using namespace CaloHLTConst;
-
-//class AliHLTCaloProcessor:public AliHLTProcessor, public AliHLTCaloBase
 class AliHLTCaloProcessor:public AliHLTProcessor
 {
 
@@ -23,37 +16,17 @@ class AliHLTCaloProcessor:public AliHLTProcessor
   AliHLTCaloProcessor();
   virtual ~AliHLTCaloProcessor();
   virtual int DoInit(int argc, const char** argv) = 0;
-  //  virtual int DoDeinit() = 0;
   virtual const char* GetComponentID() = 0;
   virtual void GetInputDataTypes( std::vector <AliHLTComponentDataType>& list) =0;
   virtual AliHLTComponentDataType GetOutputDataType() =0;
   virtual void GetOutputDataSize(unsigned long& constBase, double& inputMultiplier) =0;
   virtual AliHLTComponent* Spawn() = 0; 
-
-
   char lineNumber[256];
   const char *IntToChar(int number);
-  /*
-   *Check file and write messages to AliLog system
-   */
-  bool CheckFileLog(const char *origin, const char *filename, const char *opt);
-  void DoneWritingLog(const char *origin, const char *filename);
-
-  using  AliHLTProcessor::DoEvent;
 
  protected:
-  void ScanRunNumberFromFile();
-  virtual int ScanArgumentsModule(int argc, const char** argv);
   int fCaloEventCount;                  /**<Global event counter for this component*/
-  AliHLTUInt8_t  fModuleID;             /**<ID of the module this component read data from (0-4)*/
-
-  Bool_t fPrintInfoModule;                    /**<wether or not to print debugg info to std out*/
-  int fPrintInfoFrequncyModule;               /**<Defines the update frequency for information printet to std out*/
-
   static const AliHLTComponentDataType fgkInputDataTypes[]; /**<List of  datatypes that can be given to this component*/
-  int fRunNumber;
-  char fFilepath[1024];
-  char fMessage[1024];
 
  private:
   AliHLTCaloProcessor(const AliHLTCaloProcessor & );