added -Wshadow flag; corrected first bunch of corresponding warnings
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 20 May 2008 10:18:53 +0000 (10:18 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 20 May 2008 10:18:53 +0000 (10:18 +0000)
HLT/BASE/AliHLTComponent.cxx
HLT/BASE/AliHLTProcessor.cxx
HLT/BASE/AliHLTTask.cxx
HLT/BASE/AliHLT_C_Component_WrapperInterface.cxx
HLT/BASE/interface/AliHLTExternalInterface.cxx
HLT/BASE/util/AliHLTFileWriter.cxx
HLT/TPCLib/AliHLTTPCConfMapFit.cxx
HLT/TPCLib/AliHLTTPCFileHandler.cxx
HLT/TPCLib/AliHLTTPCTransform.cxx
HLT/configure.ac
HLT/rec/AliHLTEsdManager.cxx

index 501c7d0..d675df7 100644 (file)
@@ -126,13 +126,13 @@ int AliHLTComponent::UnsetGlobalComponentHandler()
   return SetGlobalComponentHandler(NULL,1);
 }
 
-int AliHLTComponent::Init( AliHLTComponentEnvironment* environ, void* environParam, int argc, const char** argv )
+int AliHLTComponent::Init( AliHLTComponentEnvironment* comenv, void* environParam, int argc, const char** argv )
 {
   // see header file for function documentation
   HLTLogKeyword(GetComponentID());
   int iResult=0;
-  if (environ) {
-    memcpy(&fEnvironment, environ, sizeof(AliHLTComponentEnvironment));
+  if (comenv) {
+    memcpy(&fEnvironment, comenv, sizeof(AliHLTComponentEnvironment));
     fEnvironment.fParam=environParam;
   }
   const char** pArguments=NULL;
@@ -1228,7 +1228,9 @@ int AliHLTComponent::ProcessEvent( const AliHLTComponentEventData& evtData,
   
   AliHLTComponentBlockDataList blockData;
   { // dont delete, sets the scope for the stopwatch guard
-    ALIHLTCOMPONENT_DA_STOPWATCH();
+    // do not use ALIHLTCOMPONENT_DA_STOPWATCH(); macro
+    // in order to avoid 'shadowed variable' warning
+    AliHLTStopwatchGuard swguard2(fpStopwatches!=NULL?reinterpret_cast<TStopwatch*>(fpStopwatches->At((int)kSWDA)):NULL);
     iResult=DoProcessing(evtData, blocks, trigData, outputPtr, size, blockData, edd);
   } // end of the scope of the stopwatch guard
   if (iResult>=0 && !bSkipDataProcessing) {
index 0221f56..93a81e5 100644 (file)
@@ -46,11 +46,11 @@ AliHLTProcessor::~AliHLTProcessor()
   // see header file for class documentation
 }
 
-int AliHLTProcessor::Init( AliHLTComponentEnvironment* environ, void* environParam, int argc, const char** argv )
+int AliHLTProcessor::Init( AliHLTComponentEnvironment* comenv, void* environParam, int argc, const char** argv )
 {
   // see header file for class documentation
   int iResult=0;
-  iResult=AliHLTComponent::Init(environ, environParam, argc, argv);
+  iResult=AliHLTComponent::Init(comenv, environParam, argc, argv);
   return iResult;
 }
 
index b0f8caf..e55d204 100644 (file)
@@ -346,6 +346,7 @@ int AliHLTTask::StartRun()
   AliHLTComponent* pComponent=GetComponent();
   if (pComponent) {
     // determine the number of input data blocks provided from the source tasks
+    { // set scope for lnk as a local variable
     TObjLink* lnk=fListDependencies.FirstLink();
     while (lnk && iResult>=0) {
       AliHLTTask* pSrcTask=(AliHLTTask*)lnk->GetObject();
@@ -361,6 +362,7 @@ int AliHLTTask::StartRun()
       }
       lnk=lnk->Next();
     }
+    }
     if (iResult>=0) {
       if (fBlockDataArray.size()>0) {
        HLTWarning("block data array for task %s (%p) was not cleaned", GetName(), this);
index 1027f02..4101f4a 100644 (file)
@@ -35,13 +35,13 @@ using namespace std;
 static AliHLTComponentHandler *gComponentHandler_C = NULL;
 
 
-int AliHLT_C_Component_InitSystem( AliHLTComponentEnvironment* environ )
+int AliHLT_C_Component_InitSystem( AliHLTComponentEnvironment* comenv )
 {
   if ( gComponentHandler_C )
     {
       return EINPROGRESS;
     }
-  gComponentHandler_C = new AliHLTComponentHandler(environ);
+  gComponentHandler_C = new AliHLTComponentHandler(comenv);
   if ( !gComponentHandler_C )
     return EFAULT;
   gComponentHandler_C->InitAliLogTrap(gComponentHandler_C);
@@ -83,7 +83,7 @@ int AliHLT_C_Component_UnloadLibrary( const char* /*libraryPath*/ )
   return 0;
 }
 
-int AliHLT_C_CreateComponent( const char* componentType, void* environ_param, int argc, const char** argv, AliHLTComponentHandle* handle )
+int AliHLT_C_CreateComponent( const char* componentType, void* environParam, int argc, const char** argv, AliHLTComponentHandle* handle )
 {
   if ( !gComponentHandler_C )
     return ENXIO;
@@ -91,7 +91,7 @@ int AliHLT_C_CreateComponent( const char* componentType, void* environ_param, in
   AliHLTComponent* comp=NULL;
   const char* cdbPath = getenv("ALIHLT_HCDBDIR");
   if (!cdbPath) cdbPath = getenv("ALICE_ROOT");
-  int ret = gComponentHandler_C->CreateComponent( componentType, environ_param, argc, argv, comp, cdbPath);
+  int ret = gComponentHandler_C->CreateComponent( componentType, environParam, argc, argv, comp, cdbPath);
   *handle = reinterpret_cast<AliHLTComponentHandle>( comp );
 
   return ret;
@@ -107,11 +107,11 @@ void AliHLT_C_DestroyComponent( AliHLTComponentHandle handle )
   delete pComp;
 }
 
-int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponent_EventData* evtData, const AliHLTComponent_BlockData* blocks, 
-                           AliHLTComponent_TriggerData* trigData, AliHLTUInt8_t* outputPtr,
+int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponentEventData* evtData, const AliHLTComponentBlockData* blocks, 
+                           AliHLTComponentTriggerData* trigData, AliHLTUInt8_t* outputPtr,
                            AliHLTUInt32_t* size, AliHLTUInt32_t* outputBlockCnt, 
-                           AliHLTComponent_BlockData** outputBlocks,
-                           AliHLTComponent_EventDoneData** edd )
+                           AliHLTComponentBlockData** outputBlocks,
+                           AliHLTComponentEventDoneData** edd )
 {
   if ( !handle )
     return ENXIO;
@@ -119,7 +119,7 @@ int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponent_E
   return comp->ProcessEvent( *evtData, blocks, *trigData, outputPtr, *size, *outputBlockCnt, *outputBlocks, *edd );
 }
 
-int AliHLT_C_GetOutputDataType( AliHLTComponentHandle handle, AliHLTComponent_DataType* dataType )
+int AliHLT_C_GetOutputDataType( AliHLTComponentHandle handle, AliHLTComponentDataType* dataType )
 {
   if ( !handle )
     return ENXIO;
index ff3f915..fb5602c 100644 (file)
@@ -36,13 +36,13 @@ using namespace std;
 static AliHLTComponentHandler *gComponentHandler_C = NULL;
 
 
-int AliHLT_C_Component_InitSystem( AliHLTComponentEnvironment* environ )
+int AliHLT_C_Component_InitSystem( AliHLTComponentEnvironment* comenv )
 {
   if ( gComponentHandler_C )
     {
       return EINPROGRESS;
     }
-  gComponentHandler_C = new AliHLTComponentHandler(environ);
+  gComponentHandler_C = new AliHLTComponentHandler(comenv);
   if ( !gComponentHandler_C )
     return EFAULT;
   gComponentHandler_C->InitAliLogTrap(gComponentHandler_C);
@@ -74,12 +74,12 @@ int AliHLT_C_Component_UnloadLibrary( const char* libraryPath )
   return gComponentHandler_C->UnloadLibrary( libraryPath );
 }
 
-int AliHLT_C_CreateComponent( const char* componentType, void* environ_param, int argc, const char** argv, AliHLTComponentHandle* handle )
+int AliHLT_C_CreateComponent( const char* componentType, void* environParam, int argc, const char** argv, AliHLTComponentHandle* handle )
 {
   if ( !gComponentHandler_C )
     return ENXIO;
   AliHLTComponent* comp;
-  int ret = gComponentHandler_C->CreateComponent( componentType, environ_param, argc, argv, comp );
+  int ret = gComponentHandler_C->CreateComponent( componentType, environParam, argc, argv, comp );
   *handle = reinterpret_cast<AliHLTComponentHandle>( comp );
   return ret;
 }
@@ -94,11 +94,11 @@ void AliHLT_C_DestroyComponent( AliHLTComponentHandle handle )
   delete pComp;
 }
 
-int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponent_EventData* evtData, const AliHLTComponent_BlockData* blocks, 
-                           AliHLTComponent_TriggerData* trigData, AliHLTUInt8_t* outputPtr,
+int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponentEventData* evtData, const AliHLTComponentBlockData* blocks, 
+                           AliHLTComponentTriggerData* trigData, AliHLTUInt8_t* outputPtr,
                            AliHLTUInt32_t* size, AliHLTUInt32_t* outputBlockCnt, 
-                           AliHLTComponent_BlockData** outputBlocks,
-                           AliHLTComponent_EventDoneData** edd )
+                           AliHLTComponentBlockData** outputBlocks,
+                           AliHLTComponentEventDoneData** edd )
 {
   if ( !handle )
     return ENXIO;
@@ -106,7 +106,7 @@ int AliHLT_C_ProcessEvent( AliHLTComponentHandle handle, const AliHLTComponent_E
   return comp->ProcessEvent( *evtData, blocks, *trigData, outputPtr, *size, *outputBlockCnt, *outputBlocks, *edd );
 }
 
-int AliHLT_C_GetOutputDataType( AliHLTComponentHandle handle, AliHLTComponent_DataType* dataType )
+int AliHLT_C_GetOutputDataType( AliHLTComponentHandle handle, AliHLTComponentDataType* dataType )
 {
   if ( !handle )
     return ENXIO;
index 00349eb..8cdf1e6 100644 (file)
@@ -100,12 +100,12 @@ int AliHLTFileWriter::DoInit( int argc, const char** argv )
       if (pTokens) {
        int iEntries=pTokens->GetEntries();
        if (iEntries>1) {
-         int i=0;
-         fBaseName=((TObjString*)pTokens->At(i++))->GetString();
-         while (i<iEntries-1) {
-           fBaseName+="." + ((TObjString*)pTokens->At(i++))->GetString();
+         int n=0;
+         fBaseName=((TObjString*)pTokens->At(n++))->GetString();
+         while (n<iEntries-1) {
+           fBaseName+="." + ((TObjString*)pTokens->At(n++))->GetString();
          }
-         fExtension=((TObjString*)pTokens->At(i))->GetString();
+         fExtension=((TObjString*)pTokens->At(n))->GetString();
        }
        delete pTokens;
       }
index 22f3a14..3efbdaa 100644 (file)
@@ -715,12 +715,15 @@ Int_t AliHLTTPCConfMapFit::FitLineSZ ( ){
     for(fTrack->StartLoop(); fTrack->LoopDone(); fTrack->GetNextHit()) {
        AliHLTTPCConfMapPoint *currentHit = (AliHLTTPCConfMapPoint*)fTrack->GetCurrentHit();
 
-       Double_t sPrime =  currentHit->GetS() - sav;
-       sPrime   += currentHit->GetZWeight();
-       ssPrime  += currentHit->GetZWeight() * sPrime;
-       sssPrime += currentHit->GetZWeight() * sPrime * sPrime;
+       // Matthias 20.05.2008
+       // here was a shadowed variable, sPrime is formerly defined
+       // renamed it to lsPrime ('local')
+       Double_t lsPrime =  currentHit->GetS() - sav;
+       lsPrime   += currentHit->GetZWeight();
+       ssPrime  += currentHit->GetZWeight() * lsPrime;
+       sssPrime += currentHit->GetZWeight() * lsPrime * lsPrime;
        szPrime  += currentHit->GetZWeight() * currentHit->GetZ();
-       sszPrime += currentHit->GetZWeight() * sPrime * currentHit->GetZ();
+       sszPrime += currentHit->GetZWeight() * lsPrime * currentHit->GetZ();
     }
 
     Double_t det = sPrime*sssPrime + ssPrime*ssPrime;
index e91d8b7..287646a 100644 (file)
@@ -1040,9 +1040,8 @@ AliHLTTPCDigitRowData * AliHLTTPCFileHandler::AliAltroDigits2Memory(UInt_t & nro
       }
     }
     Byte_t *tmp = (Byte_t*)tempPt;
-    Int_t size = sizeof(AliHLTTPCDigitRowData)
+    tmp += sizeof(AliHLTTPCDigitRowData)
       + ndigits[r]*sizeof(AliHLTTPCDigitData);
-    tmp += size;
     tempPt = (AliHLTTPCDigitRowData*)tmp;
   }
   delete [] ndigits;
@@ -1104,13 +1103,13 @@ void AliHLTTPCFileHandler::AliDigits2RootFile(AliHLTTPCDigitRowData *rowPt,Char_
     return;
   }
   tpcLoader->LoadDigits();
-  TTree *t=tpcLoader->TreeD();
+  TTree *treeD=tpcLoader->TreeD();
 
   AliTPCDigitsArray *oldArray = new AliTPCDigitsArray();
   oldArray->Setup(fParam);
   oldArray->SetClass("AliSimDigits");
 
-  Bool_t ok = oldArray->ConnectTree(t);
+  Bool_t ok = oldArray->ConnectTree(treeD);
   if(!ok)
     {
       LOG(AliHLTTPCLog::kError,"AliHLTTPCFileHandler::AliDigits2RootFile","File")
@@ -1168,21 +1167,21 @@ void AliHLTTPCFileHandler::AliDigits2RootFile(AliHLTTPCDigitRowData *rowPt,Char_
          digcounter++;
          
          //Tricks to get and set the correct track id's. 
-         for(Int_t t=0; t<3; t++)
+         for(Int_t track=0; track<3; track++)
            {
-             Int_t label = oldDig->GetTrackIDFast(time,pad,t);
+             Int_t label = oldDig->GetTrackIDFast(time,pad,track);
              if(label > 1)
-               trackID[t] = label - 2;
+               trackID[track] = label - 2;
              else if(label==0)
-               trackID[t] = -2;
+               trackID[track] = -2;
              else
-               trackID[t] = -1;
+               trackID[track] = -1;
            }
          
          dig->SetDigitFast(charge,time,pad);
          
-         for(Int_t t=0; t<3; t++)
-           ((AliSimDigits*)dig)->SetTrackIDFast(trackID[t],time,pad,t);
+         for(Int_t track=0; track<3; track++)
+           ((AliSimDigits*)dig)->SetTrackIDFast(trackID[track],time,pad,track);
          
        }
       //cout<<"Wrote "<<digcounter<<" on row "<<i<<endl;
index 0dffa84..d54f838 100644 (file)
@@ -1,3 +1,4 @@
+
 // @(#) $Id$
 // Original: AliHLTTransform.cxx,v 1.53 2005/06/14 10:55:21 cvetan 
 
@@ -1241,11 +1242,11 @@ Bool_t AliHLTTPCTransform::MakeInitFile(Char_t *rootfilename,Char_t *filename)
        <<"Could not open file: "<<rootfilename<<ENDLOG;
       return kFALSE;
     }
-  AliRun *gAlice = (AliRun*)rootfile->Get("gAlice");
-  if(!gAlice)
+  AliRun *lgAlice = (AliRun*)rootfile->Get("gAlice");
+  if(!lgAlice)
     {
       LOG(AliHLTTPCLog::kError,"AliHLTTPCTransform::MakeInitFile","File")
-       <<"No gAlice in file: "<<rootfilename<<ENDLOG;
+       <<"No lgAlice in file: "<<rootfilename<<ENDLOG;
       return kFALSE;
     }  
   AliTPCParamSR *param=(AliTPCParamSR*)rootfile->Get(GetParamName());
@@ -1317,10 +1318,10 @@ Bool_t AliHLTTPCTransform::MakeInitFile(Char_t *rootfilename,Char_t *filename)
   }
   
   fgVersion=kValiroot;
-  SetBFieldFactor((Double_t)gAlice->Field()->Factor());
+  SetBFieldFactor((Double_t)lgAlice->Field()->Factor());
   SetSolenoidBField(-
-                   (Double_t)gAlice->Field()->SolenoidField()/
-                   (Double_t)gAlice->Field()->Factor()
+                   (Double_t)lgAlice->Field()->SolenoidField()/
+                   (Double_t)lgAlice->Field()->Factor()
                    );
   fgPadPitchWidthLow=param->GetInnerPadPitchWidth();
   fgPadPitchWidthUp=param->GetOuterPadPitchWidth();
@@ -1364,7 +1365,7 @@ Bool_t AliHLTTPCTransform::MakeInitFile(Char_t *rootfilename,Char_t *filename)
     else fgSectorLow[i]=0;
   }
 
-  delete gAlice;
+  delete lgAlice;
   rootfile->Close();
   delete rootfile;
 
index 19f140e..789b3b4 100644 (file)
@@ -484,7 +484,7 @@ AC_ARG_ENABLE(strict,
       [disable coding convention checks ])],
   [],[enable_strict=yes])
 if test "x$enable_strict" = "xyes" ; then
-   CPPFLAGS="$CPPFLAGS -W -Weffc++ -Wall"
+   CPPFLAGS="$CPPFLAGS -W -Weffc++ -Wall -Wshadow"
 fi
 AC_MSG_RESULT([$enable_strict])
 
index f63df7b..9bf6fa9 100644 (file)
@@ -512,7 +512,7 @@ int AliHLTEsdManager::AliHLTEsdListEntry::WriteESD(AliESDEvent* pSrcESD, int eve
 TString AliHLTEsdManager::AliHLTEsdListEntry::WriteTempFile(AliESDEvent* pESD) const
 {
   // see header file for class documentation
-  int iResult;
+  int iResult=0;
   TString tmpfilename;
   FILE* pTmpFile=gSystem->TempFileName(tmpfilename);
   if (pTmpFile) {