Make usage of mask=0 possible
authorlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Jan 2010 14:02:00 +0000 (14:02 +0000)
committerlaphecet <laphecet@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 13 Jan 2010 14:02:00 +0000 (14:02 +0000)
MUON/AliMUONPadStatusMaker.cxx
MUON/AliMUONPadStatusMapMaker.cxx

index d3ee831..f4936b5 100644 (file)
@@ -115,6 +115,11 @@ AliMUONPadStatusMaker::AsString(Int_t status)
 {
   /// return a human readable version of the integer status
   
+  if ( status == 0 ) 
+  {
+    return "Brave New World";
+  }
+  
   Int_t pedStatus;
   Int_t gainStatus;
   Int_t hvStatus;
@@ -709,7 +714,7 @@ AliMUONPadStatusMaker::Report(UInt_t mask)
         
         Int_t status = PadStatus(detElemId,manuId,i);          
         
-        if ( ( status & mask) || (!mask && status) )
+        if ( mask && ( status & mask) ) // note that if mask == 0, all pads are good...
         {
           ++nBadPads;
           log.Log(AsString(status));
index 8c6785f..73469f7 100644 (file)
@@ -242,7 +242,7 @@ AliMUONPadStatusMapMaker::ComputeStatusMap(Int_t detElemId, Int_t manuId) const
       {
         status = statusParam->ValueAsIntFast(c); //fkStatusMaker.PadStatus(detElemId,m,c);
       }
-      if ( ( fMask==0 && status !=0 ) || ( (status & fMask) != 0 ) )
+      if ( ( fMask != 0 ) && ( (status & fMask) != 0 ) )
       {
         statusMap |= (1<<i);
       }