compilation warnings fixed
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 14 Sep 2007 08:19:41 +0000 (08:19 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 14 Sep 2007 08:19:41 +0000 (08:19 +0000)
HLT/BASE/AliHLTComponent.cxx
HLT/BASE/AliHLTComponent.h
HLT/BASE/AliHLTConfiguration.cxx
HLT/BASE/util/AliHLTFilePublisher.cxx
HLT/BASE/util/AliHLTFilePublisher.h
HLT/TPCLib/AliHLTTPCCalibPedestalComponent.cxx
HLT/TPCLib/AliHLTTPCCalibPulserComponent.cxx

index 077e45d..9da7b05 100644 (file)
@@ -861,11 +861,11 @@ AliHLTMemoryFile* AliHLTComponent::CreateMemoryFile(int capacity,
   // see header file for function documentation
   ALIHLTCOMPONENT_BASE_STOPWATCH();
   AliHLTMemoryFile* pFile=NULL;
-  if (capacity>=0 && capacity<=fOutputBufferSize-fOutputBufferFilled){
+  if (capacity>=0 && static_cast<unsigned int>(capacity)<=fOutputBufferSize-fOutputBufferFilled){
     AliHLTUInt8_t* pTgt=fpOutputBuffer+fOutputBufferFilled;
     pFile=new AliHLTMemoryFile((char*)pTgt, capacity);
     if (pFile) {
-      int nofBlocks=fOutputBlocks.size();
+      unsigned int nofBlocks=fOutputBlocks.size();
       if (nofBlocks+1>fMemFiles.size()) {
        fMemFiles.resize(nofBlocks+1, NULL);
       }
@@ -1008,7 +1008,7 @@ int AliHLTComponent::ProcessEvent( const AliHLTComponentEventData& evtData,
 
   // find special events
   if (fpInputBlocks) {
-    for (int i=0; i<evtData.fBlockCnt && iResult>=0; i++) {
+    for (unsigned int i=0; i<evtData.fBlockCnt && iResult>=0; i++) {
       if (fpInputBlocks[i].fDataType==kAliHLTDataTypeSOR) {
        // start of run
        if (fpRunDesc==NULL) {
@@ -1218,7 +1218,7 @@ AliHLTUInt32_t AliHLTComponent::GetRunType() const
   return fpRunDesc->fRunType;
 }
 
-int AliHLTComponent::CopyStruct(void* pStruct, int iStructSize, int iBlockNo,
+int AliHLTComponent::CopyStruct(void* pStruct, unsigned int iStructSize, unsigned int iBlockNo,
                                const char* structname, const char* eventname)
 {
   int iResult=0;
index 2733f55..498212a 100644 (file)
@@ -902,7 +902,7 @@ class AliHLTComponent : public AliHLTLogging {
    * @param eventname   name of the event (log messages)
    * @return size copied, neg. error if failed
    */
-  int CopyStruct(void* pStruct, int iStructSize, int iBlockNo,
+  int CopyStruct(void* pStruct, unsigned int iStructSize, unsigned int iBlockNo,
                 const char* structname="", const char* eventname="");
 
  private:
index eac0eab..c7dda48 100644 (file)
@@ -815,10 +815,17 @@ int AliHLTTask::ProcessTask(Int_t eventNo)
       pSrcTask=(AliHLTTask*)lnk->GetObject();
       if (pSrcTask) {
        int iMatchingDB=pSrcTask->GetNofMatchingDataBlocks(this);
-       if (iMatchingDB>fBlockDataArray.size()-iSourceDataBlock) {
+       if (iMatchingDB>=0 && static_cast<unsigned int>(iMatchingDB)>fBlockDataArray.size()-iSourceDataBlock) {
          AliHLTComponentBlockData init;
          memset(&init, 0, sizeof(AliHLTComponentBlockData));
          fBlockDataArray.resize(iSourceDataBlock+iMatchingDB, init);
+       } else {
+         if (iMatchingDB<0) {
+           HLTError("task %s (%p): error getting no of matching data blocks from task %s (%p)", GetName(), this, pSrcTask->GetName(), pSrcTask);
+           iResult=iMatchingDB;
+         } else {
+           HLTError("task %s (%p): block data array too small to get blocks from task %s (%p)", GetName(), this, pSrcTask->GetName(), pSrcTask);
+         }
        }
        if ((iResult=pSrcTask->Subscribe(this, &fBlockDataArray[iSourceDataBlock],fBlockDataArray.size()-iSourceDataBlock))>0) {
          for (int i=0; i<iResult; i++) {
@@ -1040,7 +1047,7 @@ AliHLTConfigurationHandler::~AliHLTConfigurationHandler()
 {
   // see header file for function documentation
   TObjLink* lnk=NULL;
-  while (lnk=fgListConfigurations.FirstLink()) {
+  while ((lnk=fgListConfigurations.FirstLink())!=NULL) {
     AliHLTConfiguration* pConf=(AliHLTConfiguration*)lnk->GetObject();
     HLTDebug("delete configuration \"%s\"", pConf->GetName());
     fgListConfigurations.Remove(lnk);
index d8143df..323c1ee 100644 (file)
@@ -133,7 +133,6 @@ int AliHLTFilePublisher::DoInit( int argc, const char** argv )
        // the arguments in the wrong sequence
        TList& files=*pCurrEvent; // type conversion operator defined
        TObjLink *flnk=files.FirstLink();
-       int eventSize=0;
        while (flnk) {
          FileDesc* pFileDesc=dynamic_cast<FileDesc*>(flnk->GetObject());
          if (pFileDesc) {
@@ -163,7 +162,6 @@ int AliHLTFilePublisher::DoInit( int argc, const char** argv )
        // the arguments in the wrong sequence
        TList& files=*pCurrEvent; // type conversion operator defined
        TObjLink *flnk=files.FirstLink();
-       int eventSize=0;
        while (flnk) {
          FileDesc* pFileDesc=dynamic_cast<FileDesc*>(flnk->GetObject());
          if (pFileDesc) {
index 758b4e4..36d00bf 100644 (file)
@@ -151,12 +151,12 @@ class AliHLTFilePublisher : public AliHLTDataSource  {
     /**
      * Set data type.
      */
-    int SetDataType(AliHLTComponentDataType dt) {fDataType=dt;}
+    int SetDataType(AliHLTComponentDataType dt) {fDataType=dt; return 0;}
 
     /**
      * Set data specification
      */
-    int SetSpecification(AliHLTUInt32_t spec) {fSpecification=spec;}
+    int SetSpecification(AliHLTUInt32_t spec) {fSpecification=spec; return 0;}
 
     // implicite type conversions
     operator TFile*() const   {return fpInstance;}
index 940e342..77e141b 100755 (executable)
@@ -217,7 +217,7 @@ Int_t AliHLTTPCCalibPedestalComponent::ProcessCalibration( const AliHLTComponent
   
   const AliHLTComponentBlockData* iter = NULL;
 
-  AliHLTUInt8_t slice, patch;
+  AliHLTUInt8_t slice=0, patch=0;
   Int_t DDLid = 0;
     
   // ** Loop over all input blocks and specify which data format should be read - only select Raw Data
index 3ba272d..471842c 100644 (file)
@@ -228,7 +228,7 @@ Int_t AliHLTTPCCalibPulserComponent::ProcessCalibration( const AliHLTComponentEv
   
   const AliHLTComponentBlockData* iter = NULL;
 
-  AliHLTUInt8_t slice, patch;
+  AliHLTUInt8_t slice=0, patch=0;
   Int_t DDLid = 0;
     
   // ** Loop over all input blocks and specify which data format should be read - only select Raw Data