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 e9b6e5a..f68f914 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 7cc7722..f57be1e 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 32e37fa..89efbfa 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 559bff6..6a0417e 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 a936081..90ffb00 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 2f331bb..60ad966 100644 (file)
@@ -108,36 +108,6 @@ class AliHLTSystem : public AliHLTLogging {
   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
    * objects which are added to the HLT system handler. This is an iterative
@@ -151,14 +121,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.
    */
   int CleanTaskList();
@@ -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.
@@ -521,6 +485,16 @@ class AliHLTSystem : public AliHLTLogging {
   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.
    */
   int SetStatusFlags(int 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