correcting placement of 'using' statements (Thorsten), minor coverity defect corrected
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 31 Jul 2012 08:16:56 +0000 (08:16 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 31 Jul 2012 08:16:56 +0000 (08:16 +0000)
HLT/EMCAL/AliHLTEMCALCalibrationComponent.cxx
HLT/EMCAL/AliHLTEMCALMapper.cxx
HLT/EMCAL/OnlineDisplay/AliHLTEMCALOnlineDisplayEventTab.cxx
HLT/EMCAL/OnlineDisplay/AliHLTEMCALOnlineDisplayFourierTab.cxx

index bb57406..b9afe54 100644 (file)
     @date   
     @brief  A EMCALCalibration processing component for the HLT. */
 
-#if __GNUC__ >= 3
-using namespace std;
-#endif
-
 #include "TTree.h"
 #include "TFile.h"
 #include "TBranch.h"
@@ -37,6 +33,8 @@ using namespace std;
 #include <cerrno>
 #include <string>
 
+using namespace std;
+
 // this is a global object used for automatic component registration, do not use this
 AliHLTEMCALCalibrationComponent gAliHLTEMCALCalibrationComponent;
 
index ff64b0c..a267215 100644 (file)
@@ -65,6 +65,7 @@ AliHLTEMCALMapper::InitAltroMapping(const unsigned long specification )
   int tmpXCol = 0;
   int tmpGain = 0;
   int res = 0; 
+  fIsInitializedMapping = false;
   
   if(!base.IsNull())
     {
@@ -75,7 +76,9 @@ AliHLTEMCALMapper::InitAltroMapping(const unsigned long specification )
       if(fp != 0)
        {
          res = fscanf(fp, "%d\n", &nChannels);
+         if (res!=1) return false;
          res = fscanf(fp, "%d\n", &maxaddr);
+         if (res!=1) return false;
          //      fHw2geomapPtr = new fAltromap[maxaddr +1]; 
          fHw2geomapPtr = new fAltromap[MAXHWADDR +1];
        
@@ -92,6 +95,7 @@ AliHLTEMCALMapper::InitAltroMapping(const unsigned long specification )
              for(int i=0; i<nChannels; i ++)
                {
                  res = fscanf(fp, "%d %d %d %d\n", &tmpHwaddr, &tmpXCol, &tmpZRow,  &tmpGain);
+                 if (res!=4) return false;
                  
                  if(tmpGain < 2)
                    {
index 116969a..aa562f0 100644 (file)
 #include "AliHLTCaloCoordinate.h"
 //#include "AliHLTCaloChannelRawDataStruct.h"
 
-
-using namespace std;
-
 // MT Crap
 #include <TMath.h>
 #include "AliHLTEMCALOnlineDisplayTH2D.h"
 
+using namespace std;
 
 //#include <TEveManager.h>
 //#include <TEveBoxSet.h>
index 793f9b4..20e2c3a 100644 (file)
 
 #define SAMPLINGFREQUENCY 10
 
-using namespace std;
-
 // MT Crap
 #include <TMath.h>
 //#include <TEveManager.h>
 //#include <TEveBoxSet.h>
 
+using namespace std;
+
 //TEveBoxSet* gAliEveBoxSet = 0;
 
 AliHLTEMCALOnlineDisplayFourierTab::AliHLTEMCALOnlineDisplayFourierTab()