code cleanup
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 26 Apr 2010 21:49:31 +0000 (21:49 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 26 Apr 2010 21:49:31 +0000 (21:49 +0000)
- deleting obsolete function definitions and dummy implementations
- making all members of AliHLTSystem protected
- making function private which is intended for internal use only

HLT/BASE/AliHLTComponentHandler.cxx
HLT/BASE/AliHLTComponentHandler.h
HLT/BASE/AliHLTDataTypes.h
HLT/BASE/AliHLTOUTHandlerChain.cxx
HLT/BASE/AliHLTSystem.cxx
HLT/BASE/AliHLTSystem.h

index e9b6e5aa8141227ad071ed8a2ea2a7294cd47b54..f68f914e3ce8093d4fb2943e2c586174d8d75c5f 100644 (file)
@@ -550,7 +550,7 @@ int AliHLTComponentHandler::UnloadLibraries()
   return iResult;
 }
 
-void (*AliHLTComponentHandler::FindSymbol(const char* library, const char* symbol))()
+AliHLTfctVoid AliHLTComponentHandler::FindSymbol(const char* library, const char* symbol)
 {
   // see header file for class documentation
   AliHLTLibHandle* hLib=FindLibrary(library);
index 7cc7722ef9a2823a9b7b4aaa1941b9ac72a28da3..f57be1ec0b0c9694ed317d67c1c781d1b665d525 100644 (file)
@@ -108,7 +108,7 @@ class AliHLTComponentHandler : public AliHLTLogging {
    * @param symbol       the symbol to find
    * @return void pointer to function
    */
-  void (*FindSymbol(const char* library, const char* symbol))();
+  AliHLTfctVoid FindSymbol(const char* library, const char* symbol);
 
   /**
    * Unload a component shared library.
index 32e37fa9482b7e91ac3ebff852880e7c18da77b8..89efbfae80c52ee350ed35b7d31c921afccc62eb 100644 (file)
@@ -1107,6 +1107,9 @@ extern "C" {
   //
   //////////////////////////////////////////////////////////////////////////
 
+  /** definition of a void fct pointer */
+  typedef void (*AliHLTfctVoid)();
+
   /** logging function */
   typedef int (*AliHLTfctLogging)( void* param, 
                                   AliHLTComponentLogSeverity severity,
index 559bff62db37152d59ba42db950d99281983ef10..6a0417e3398fa13159d16c204ea39e1eee7d34f5 100644 (file)
@@ -162,7 +162,7 @@ int AliHLTOUTHandlerChain::InitSystem()
       if ((iResult=fpSystem->ScanOptions(fOptions.Data()))>=0) {
        // load configurations if not specified by external macro
        if (!fOptions.Contains("config="))
-         iResult=CreateConfigurations(fpSystem->fpConfigurationHandler);
+         iResult=CreateConfigurations(fpSystem->GetConfigurationHandler());
 
        if (iResult>=0) {
          iResult=fpSystem->BuildTaskList(fChains.Data());
index a93608131903b82a11cea7e52dbd965e7ec892b9..90ffb0047c25ac4bacafcf317838621986727507 100644 (file)
@@ -146,51 +146,6 @@ AliHLTSystem::~AliHLTSystem()
 
 int AliHLTSystem::fgNofInstances=0;
 
-int AliHLTSystem::AddConfiguration(AliHLTConfiguration* pConf)
-{
-  // see header file for class documentation
-  HLTLogKeyword("configuration handling");
-  int iResult=0;
-  if (pConf) {
-    HLTError("function not yet implemented");
-    iResult=-ENOSYS;
-  } else {
-    iResult=-EINVAL;
-  }
-  return iResult;
-}
-
-int AliHLTSystem::InsertConfiguration(AliHLTConfiguration* pConf, AliHLTConfiguration* pPrec)
-{
-  // see header file for class documentation
-  HLTLogKeyword("configuration handling");
-  int iResult=0;
-  if (pConf) {
-    if (pPrec) {
-      // find the position
-      HLTError("function not yet implemented");
-      iResult=-ENOSYS;
-    }
-  } else {
-    iResult=-EINVAL;
-  }
-  return iResult;
-}
-
-int AliHLTSystem::DeleteConfiguration(AliHLTConfiguration* pConf)
-{
-  // see header file for class documentation
-  HLTLogKeyword("configuration handling");
-  int iResult=0;
-  if (pConf) {
-    HLTError("function not yet implemented");
-    iResult=-ENOSYS;
-  } else {
-    iResult=-EINVAL;
-  }
-  return iResult;
-}
-
 int AliHLTSystem::BuildTaskList(const char* id)
 {
   // see header file for class documentation
@@ -1563,7 +1518,7 @@ int AliHLTSystem::ClearStatusFlags(int flags)
   return fState;
 }
 
-void (*AliHLTSystem::FindDynamicSymbol(const char* library, const char* symbol))()
+AliHLTfctVoid AliHLTSystem::FindDynamicSymbol(const char* library, const char* symbol)
 {
   // see header file for class documentation
   if (fpComponentHandler==NULL) return NULL;
index 2f331bb0c32ee0e1c949f98a3709e158de778e19..60ad966593984ac168a42495839c05e5b1c007d0 100644 (file)
@@ -107,36 +107,6 @@ class AliHLTSystem : public AliHLTLogging {
   /** destructor */
   virtual ~AliHLTSystem();
 
-  /**
-   * Pointer to an instance of @ref AliHLTComponentHandler.
-   */
-  AliHLTComponentHandler* fpComponentHandler;                      //! transient
-
-  /**
-   * Pointer to an instance of @ref AliHLTConfigurationHandler.
-   */
-  AliHLTConfigurationHandler* fpConfigurationHandler;              //! transient
-
-  /**
-   * Add a configuration to the end of the list.
-   * @param pConf    pointer to configuration to add
-   */
-  int AddConfiguration(AliHLTConfiguration* pConf);
-
-  /**
-   * Insert a configuration to the end of the list after the specified
-   * configuration.
-   * @param pConf    pointer to configuration to insert
-   * @param pPrec    pointer to configuration to insert the new one after
-   */
-  int InsertConfiguration(AliHLTConfiguration* pConf, AliHLTConfiguration* pPrec);
-
-  /**
-   * Remove a configuration from the list.
-   * @param pConf    pointer to configuration to delete
-   */
-  int DeleteConfiguration(AliHLTConfiguration* pConf);
-
   /**
    * Build a task list 
    * This method is used to build the tasks from the 'master' configuration
@@ -150,14 +120,6 @@ class AliHLTSystem : public AliHLTLogging {
    */
   int BuildTaskList(const char* pConf);
 
-  /**
-   * Build task list from a configuration object.
-   * This method is kept for backward compatibility. Use the version
-   * with the configuration name.
-   * @param pConf    pointer to configuration to build the task list from
-   */
-  int BuildTaskList(AliHLTConfiguration* pConf);
-
   /**
    * Clean the list of tasks and delete all the task objects.
    */
@@ -183,6 +145,8 @@ class AliHLTSystem : public AliHLTLogging {
   int AddHLTOUTTask(const char* chains);
 
   AliHLTOUTTask* GetHLTOUTTask() const {return fpHLTOUTTask;}
+  AliHLTComponentHandler* GetComponentHandler() const {return fpComponentHandler;}
+  AliHLTConfigurationHandler* GetConfigurationHandler() const {return fpConfigurationHandler;}
 
   /**
    * Find a task with an id.
@@ -398,7 +362,7 @@ class AliHLTSystem : public AliHLTLogging {
    * @param symbol       the symbol to find
    * @return void pointer to function
    */
-  void (*FindDynamicSymbol(const char* library, const char* symbol))();
+  AliHLTfctVoid FindDynamicSymbol(const char* library, const char* symbol);
 
   /**
    * Prepare the HLT system for running.
@@ -520,6 +484,16 @@ class AliHLTSystem : public AliHLTLogging {
   /** assignment operator prohibited */
   AliHLTSystem& operator=(const AliHLTSystem&);
 
+  /**
+   * Build task list from a configuration object. 
+   * This method implements the configuration parsing and transformation into
+   * a list of AliHLTTask objects. It has been made private in April 2007. 
+   * Use BuildTaskList(const char*) instead.
+   *
+   * @param pConf    pointer to configuration to build the task list from
+   */
+  int BuildTaskList(AliHLTConfiguration* pConf);
+
   /**
    * Set status flags.
    */
@@ -530,8 +504,11 @@ class AliHLTSystem : public AliHLTLogging {
    */
   int ClearStatusFlags(int flags);
 
-/*   TList fConfList; */
-/*   int fbListChanged; */
+  /// Pointer to an instance of @ref AliHLTComponentHandler.
+  AliHLTComponentHandler* fpComponentHandler;                      //! transient
+
+  /// Pointer to an instance of @ref AliHLTConfigurationHandler.
+  AliHLTConfigurationHandler* fpConfigurationHandler;              //! transient
 
   /** list of tasks */
   TList fTaskList;                                                 // see above