Fixes for bug #71984 ENUM_AS_BOOLEAN problem reported by Coverity

diff --git a/EMCAL/AliCaloCalibPedestal.cxx b/EMCAL/AliCaloCalibPedestal.cxx
index c3ec750367404ad74baaa53a2d674858b3610aa2..f1a215c6bfe91ef13cb34a3bafc0f1e8566bc709 100644 (file)
--- a/EMCAL/AliCaloCalibPedestal.cxx
+++ b/EMCAL/AliCaloCalibPedestal.cxx
@@ -448,7 +448,7 @@ void AliCaloCalibPedestal::SetParametersFromFile(const char *parameterFile)
       s >> keyValue;
       
       // check stream status
-      if( s.rdstate() & ios::failbit ) break;
+      if( ( s.rdstate() & ios::failbit ) == ios::failbit ) break;
                        
       // skip rest of line if comments found
       if( keyValue.substr( 0, 2 ) == "//" ) break;

diff --git a/EMCAL/AliCaloCalibSignal.cxx b/EMCAL/AliCaloCalibSignal.cxx
index 0977c4efcf6280c372eb7ca4f51ffe8a42ac923f..83e812a7f173a6ab5126dfa629e66130d31b3fe7 100644 (file)
--- a/EMCAL/AliCaloCalibSignal.cxx
+++ b/EMCAL/AliCaloCalibSignal.cxx
@@ -334,7 +334,7 @@ void AliCaloCalibSignal::SetParametersFromFile(const char *parameterFile)
       s >> keyValue;
       
       // check stream status
-      if( s.rdstate() & ios::failbit ) break;
+      if( ( s.rdstate() & ios::failbit ) == ios::failbit ) break;
                        
       // skip rest of line if comments found
       if( keyValue.substr( 0, 2 ) == "//" ) break;

diff --git a/EVE/EveBase/AliEveMacroExecutor.cxx b/EVE/EveBase/AliEveMacroExecutor.cxx
index 0a5a9af887dcf1b3c18be19d0ef2cfe55d94e8f2..ffd608f39e5cf0d83648c13582da806e2ac9c453 100644 (file)
--- a/EVE/EveBase/AliEveMacroExecutor.cxx
+++ b/EVE/EveBase/AliEveMacroExecutor.cxx
@@ -136,7 +136,7 @@ void AliEveMacroExecutor::ExecMacros()
        G__security_recover(0);
       }
 
-      if (error)
+      if (error != TInterpreter::kNoError)
       {
         mac->SetExecError();
         Error("ExecMacros", "Executing %s::%s, CINT error ... hopefully recovered.",

diff --git a/FASTSIM/AliQuenchingWeights.h b/FASTSIM/AliQuenchingWeights.h
index 81e5d18b7d549bd724699aaa859d33b67e356c15..9348ac031dbc5729691fb4443934f705cd6401f6 100644 (file)
--- a/FASTSIM/AliQuenchingWeights.h
+++ b/FASTSIM/AliQuenchingWeights.h
@@ -120,7 +120,7 @@ class AliQuenchingWeights : public TObject {
   Float_t GetMu()           const {return fMu;}
   Float_t GetQTransport()   const {return fQTransport;}
   Float_t GetK()            const {return fK;}
-  Bool_t  GetECMethod()     const {return fECMethod;}
+  kECMethod  GetECMethod()  const {return fECMethod;}
   Bool_t  GetTablesLoaded() const {return fTablesLoaded;}
   Bool_t  GetMultSoft()     const {return fMultSoft;}
   Int_t   GetLengthMax()    const {return fLengthMax;}

diff --git a/ITS/AliITSvSDD03.cxx b/ITS/AliITSvSDD03.cxx
index 2bde3773b69bb33f78864b77cb5d74ec0e7c33f3..41ecc69941bb5057c8365a12c3acc6af434cd517 100644 (file)
--- a/ITS/AliITSvSDD03.cxx
+++ b/ITS/AliITSvSDD03.cxx
@@ -339,7 +339,7 @@ void AliITSvSDD03::CreateGeometry(){
     gMC->Gspos("IDET",2,"ITSV",pdet2X,pdet2Y,pdet2Z,idrotm[0],"ONLY");// Detector2
 
 // Target definition and placement
-    if(fTarg){
+    if(fTarg != kNoTarg){
       data[0] = 30*kmm;
       data[1] = fTargThick*kmm;  // Target thickness
       data[2] = 30*kmm;

diff --git a/STEER/AliGeomManager.cxx b/STEER/AliGeomManager.cxx
index f0bad5de470c8f048d1fcd6d572e79e0354aad40..a193e711e056ecd67b0dd577629386310cff2451 100644 (file)
--- a/STEER/AliGeomManager.cxx
+++ b/STEER/AliGeomManager.cxx
@@ -293,7 +293,7 @@ AliGeomManager::ELayerID AliGeomManager::VolUIDToLayerSafe(UShort_t voluid, Int_
   // Checks the validity of the given voluid
   //
   ELayerID layId = VolUIDToLayerSafe(voluid);
-  if(layId){
+  if(layId != AliGeomManager::kInvalidLayer){
     Int_t mId = Int_t(voluid & 0x7ff);
     if( mId>=0 && mId<LayerSize(layId)){
       modId = mId;