bugfix: missing function implemented; next strike against warnings
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Mar 2008 19:54:18 +0000 (19:54 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Mar 2008 19:54:18 +0000 (19:54 +0000)
HLT/BASE/AliHLTOUT.cxx
HLT/TPCLib/comp/AliHLTTPCCompDumpComponent.cxx
HLT/TPCLib/comp/AliHLTTPCCompModelAnalysis.h
HLT/TPCLib/comp/AliHLTTPCCompModelConverter.cxx
HLT/TPCLib/comp/AliHLTTPCCompModelDeconverterComponent.cxx
HLT/TPCLib/comp/AliHLTTPCCompModelDeflaterComponent.cxx
HLT/comp/AliHLTCOMPHuffmanAltro.cxx
HLT/comp/AliHLTCOMPHuffmanAltroComponent.cxx

index 7c66804..61e143f 100644 (file)
@@ -76,7 +76,7 @@ int AliHLTOUT::GetNofDataBlocks()
 }
 
 int AliHLTOUT::SelectFirstDataBlock(AliHLTComponentDataType dt, AliHLTUInt32_t spec,
-                                   AliHLTModuleAgent::AliHLTOUTHandlerType handlerType)
+                                   AliHLTModuleAgent::AliHLTOUTHandlerType /*handlerType*/)
 {
   // see header file for class documentation
   if (CheckStatusFlag(kLocked)) return -EPERM;
index 3c1da17..5a97671 100644 (file)
@@ -96,7 +96,7 @@ AliHLTComponent* AliHLTTPCCompDumpComponent::Spawn()
     {
       // see header file for class documentation
       return new AliHLTTPCCompDumpComponent;
-    };
+    }
 
 void AliHLTTPCCompDumpComponent::InitBitDataInput( AliHLTUInt8_t* input, UInt_t inputSize )
     {
@@ -106,7 +106,7 @@ void AliHLTTPCCompDumpComponent::InitBitDataInput( AliHLTUInt8_t* input, UInt_t
       fBitDataCurrentInput = fBitDataCurrentInputStart = input;
       fBitDataCurrentInputEnd = input+inputSize;
       fBitDataCurrentWord = *fBitDataCurrentInput;
-    };
+    }
 
  bool AliHLTTPCCompDumpComponent::InputBit( AliHLTUInt8_t & value )
     {
@@ -126,7 +126,7 @@ void AliHLTTPCCompDumpComponent::InitBitDataInput( AliHLTUInt8_t* input, UInt_t
            }
        }
       return true;
-    };
+    }
 
 bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt8_t & value, UInt_t const & bitCount )
     {
@@ -141,7 +141,7 @@ bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt8_t & value, UInt_t const
        return false;
       value = (AliHLTUInt8_t)( temp & (AliHLTUInt64_t)0xFFFFFFFFULL );
       return true;
-    };
+    }
 
 bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt16_t & value, UInt_t const & bitCount )
    {
@@ -156,7 +156,22 @@ bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt16_t & value, UInt_t const
        return false;
      value = (AliHLTUInt16_t)( temp & (AliHLTUInt64_t)0xFFFFFFFFULL );
      return true;
-   };
+   }
+
+bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt32_t & value, UInt_t const & bitCount )
+   {
+     // see header file for class documentation
+     if ( bitCount>32 )         
+       {        
+        HLTFatal( "Internal error: Attempt to write more than 32 bits (%u)", (unsigned)bitCount );      
+        return false;   
+       }        
+     AliHLTUInt64_t temp;       
+     if ( !InputBits( temp, bitCount ) )        
+       return false;    
+     value = (AliHLTUInt32_t)( temp & (AliHLTUInt64_t)0xFFFFFFFFULL );          
+     return true;       
+   }
 
 bool AliHLTTPCCompDumpComponent::InputBits( Int_t & value, UInt_t const & bitCount )
    {
@@ -171,7 +186,7 @@ bool AliHLTTPCCompDumpComponent::InputBits( Int_t & value, UInt_t const & bitCou
        return false;
      value = (Int_t)( temp & (AliHLTUInt64_t)0xFFFFFFFFULL );
      return true;
-   };
+   }
 
 bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt64_t & value, UInt_t const & bitCount )
    {
@@ -207,7 +222,7 @@ bool AliHLTTPCCompDumpComponent::InputBits( AliHLTUInt64_t & value, UInt_t const
         bitsToRead -= curBitCount;
        }
      return true;
-   };
+   }
 
 void AliHLTTPCCompDumpComponent::Pad8Bits()
    {
@@ -220,7 +235,7 @@ void AliHLTTPCCompDumpComponent::Pad8Bits()
         fBitDataCurrentWord = *fBitDataCurrentInput;
         fBitDataCurrentPosInWord = 7;
        }
-   };
+   }
 
 bool AliHLTTPCCompDumpComponent::InputBytes( AliHLTUInt8_t* data, UInt_t const & byteCount )
    {
@@ -236,7 +251,7 @@ bool AliHLTTPCCompDumpComponent::InputBytes( AliHLTUInt8_t* data, UInt_t const &
         fBitDataCurrentPosInWord = 7;
        }
      return true;
-   };
+   }
 
 int AliHLTTPCCompDumpComponent::DoInit( int argc, const char** argv )
     {
index 354177c..d9cb0d9 100644 (file)
@@ -73,7 +73,7 @@ public:
   Bool_t GetfTrackAnalysis() {return fTrackAnalysis;};
 
   /** fill track arrays with track data from original and secondary tracking 
-   * @param firsttracklets      pointer to track array to be filled
+   * @param tracklets           pointer to track array to be filled
    * @param fillingfirsttracks  boolean to decide which track array is to be filled (1 for first, 0 for second)
    * @return 0 upon success
    */
@@ -141,7 +141,7 @@ private:
   Bool_t GetTrashTrackPythiaInfo(AliHLTTPCTrack* discardedtrack);
 
   /** compare information of a cluster not assigned to any track with its Pythia information
-   * @param discarded cluster  pointer to discarded cluster
+   * @param discardedcluster  pointer to discarded cluster
    * @return 0 upon correct decision (cluster not assigned to any track is true in Pythia, i.e. cluster = noise cluster)
    * @return 1 upon wrong decision (cluster wrongly discarded, i.e. it belongs to a valuable track according to Pythia)
    */
index b82ce4e..c9e45f6 100644 (file)
@@ -53,8 +53,8 @@ AliHLTTPCCompModelConverter::AliHLTTPCCompModelConverter():
 AliHLTTPCCompModelConverter::AliHLTTPCCompModelConverter(AliHLTTPCCompModelAnalysis* modelanalysis):
   fInputTrackArray(),
   fOutputTrackArray("AliHLTTPCModelTrack"),
-  fMinHits(0),
-  fModelAnalysisInstance(modelanalysis)
+  fModelAnalysisInstance(modelanalysis),
+  fMinHits(0)
     {
       // see header file for class documentation
       for ( UInt_t slice=0; slice<36; slice++ )
index 269300c..cd75951 100644 (file)
@@ -44,7 +44,8 @@ AliHLTTPCCompModelDeconverterComponent gAliHLTTPCCompClusterModelDeconverterComp
 ClassImp(AliHLTTPCCompModelDeconverterComponent)
     
 AliHLTTPCCompModelDeconverterComponent::AliHLTTPCCompModelDeconverterComponent():
-    fOutputTracks(kTRUE)
+  fDeconverter(),
+  fOutputTracks(kTRUE)
     {
       // see header file for class documentation
     }
index 1c659f2..9e85622 100644 (file)
@@ -36,9 +36,9 @@ AliHLTTPCCompModelDeflaterComponent gAliHLTTPCCompClusterModelDeflaterComponent;
 ClassImp(AliHLTTPCCompModelDeflaterComponent);
     
 AliHLTTPCCompModelDeflaterComponent::AliHLTTPCCompModelDeflaterComponent():
-  fForwardIfUncompressed(true),
   fModelDeflater(),
-  fConverter()
+  fConverter(),
+  fForwardIfUncompressed(true)
     {
       // see header file for class documentation
     }
index 0c58c84..07166fb 100644 (file)
@@ -1243,8 +1243,8 @@ Int_t AliHLTCOMPHuffmanAltro::CreateCodeTable()
   //  }
   
   // findout maximal and minimal codelength and print them out
-    Int_t maxcodelength = fTranslationTable[0].fvalidcodelength;
-    Int_t mincodelength = TIMEBINS;
+    UInt_t maxcodelength = fTranslationTable[0].fvalidcodelength;
+    UInt_t mincodelength = TIMEBINS;
 
     for (Int_t kk = 0; kk < TIMEBINS; kk++)
       {
index 731b797..857a8b9 100644 (file)
@@ -40,13 +40,12 @@ AliHLTCOMPHuffmanAltroComponent::AliHLTCOMPHuffmanAltroComponent(bool compressio
   fHuffmanCompressor(NULL),
   fCompressionSwitch(compression),
   fTrainingMode(kFALSE),
-  fHuffmanData(NULL),
   fOrigin(kAliHLTVoidDataOrigin),
   fRunNumber(0),
   fDataSpec(0),
   fTablePath(),
-  fNrcuTrailerwords(0)
+  fNrcuTrailerwords(0),
+  fHuffmanData(NULL) 
 {
    // see header file for class documentation
   // or