Fixing coding conventions
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 30 Jun 2010 15:00:50 +0000 (15:00 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 30 Jun 2010 15:00:50 +0000 (15:00 +0000)
42 files changed:
MUON/AliMUONAlignmentTask.cxx
MUON/AliMUONCalibrationData.cxx
MUON/AliMUONCalibrationData.h
MUON/AliMUONChamberCalibrationTask.cxx
MUON/AliMUONChamberCalibrationTask.h
MUON/AliMUONClusterSplitterMLEM.cxx
MUON/AliMUONClusterSplitterMLEM.h
MUON/AliMUONClusterStoreV2.cxx
MUON/AliMUONGain.cxx
MUON/AliMUONGeometryBuilder.cxx
MUON/AliMUONGeometryBuilder.h
MUON/AliMUONGlobalCrateConfig.h
MUON/AliMUONLocalTrigger.cxx
MUON/AliMUONLocalTrigger.h
MUON/AliMUONPedestalEventGenerator.cxx
MUON/AliMUONRawCluster.h
MUON/AliMUONRawStream.cxx
MUON/AliMUONRawStream.h
MUON/AliMUONRawStreamTracker.cxx
MUON/AliMUONRawStreamTracker.h
MUON/AliMUONRawStreamTrigger.cxx
MUON/AliMUONRawStreamTrigger.h
MUON/AliMUONRecoCheck.h
MUON/AliMUONResponseTriggerV1.cxx
MUON/AliMUONResponseTriggerV1.h
MUON/AliMUONSDigitizerV2.cxx
MUON/AliMUONTrackLight.cxx
MUON/AliMUONTrackLight.h
MUON/AliMUONTriggerChamberEfficiency.h
MUON/AliMUONTriggerDisplay.h
MUON/AliMUONTriggerEfficiencyCells.cxx
MUON/AliMUONTriggerEfficiencyCells.h
MUON/AliMUONTriggerSubprocessor.cxx
MUON/AliMUONVPainter.h
MUON/AliMUONVQADataMakerRec.h
MUON/AliMUONVRawStreamTracker.h
MUON/mapping/AliMpLocalBoard.h
MUON/mapping/AliMpMotifType.cxx
MUON/mapping/AliMpPCBPadIterator.cxx
MUON/mapping/AliMpPCBPadIterator.h
MUON/mapping/AliMpTriggerReader.cxx
MUON/mapping/AliMpTriggerReader.h

index 8bbd9f9..7f98dfa 100644 (file)
@@ -357,49 +357,49 @@ void AliMUONAlignmentTask::FinishTaskOutput()
 //   fMSDEp = (TGraphErrors*)fList->At(3);
 
   // Store results
-  Double_t DEid[156] = {0};
-  Double_t MSDEx[156] = {0};
-  Double_t MSDEy[156] = {0};
-  Double_t MSDEz[156] = {0};
-  Double_t MSDEp[156] = {0};
-  Double_t DEidErr[156] = {0};
-  Double_t MSDExErr[156] = {0};
-  Double_t MSDEyErr[156] = {0};
-  Double_t MSDEzErr[156] = {0};
-  Double_t MSDEpErr[156] = {0};
+  Double_t deId[156] = {0};
+  Double_t msdEx[156] = {0};
+  Double_t msdEy[156] = {0};
+  Double_t msdEz[156] = {0};
+  Double_t msdEp[156] = {0};
+  Double_t deIdErr[156] = {0};
+  Double_t msdExErr[156] = {0};
+  Double_t msdEyErr[156] = {0};
+  Double_t msdEzErr[156] = {0};
+  Double_t msdEpErr[156] = {0};
   Int_t lNDetElem = 4*2+4*2+18*2+26*2+26*2;
   Int_t lNDetElemCh[10] = {4,4,4,4,18,18,26,26,26,26};
   // Int_t lSNDetElemCh[10] = {4,8,12,16,34,52,78,104,130,156};
   Int_t idOffset = 0; // 400
   Int_t lSDetElemCh = 0;
   for(Int_t iDE=0; iDE<lNDetElem; iDE++){
-    DEidErr[iDE] = 0.;
-    DEid[iDE] = idOffset+100;
-    DEid[iDE] += iDE; 
+    deIdErr[iDE] = 0.;
+    deId[iDE] = idOffset+100;
+    deId[iDE] += iDE; 
     lSDetElemCh = 0;
     for(Int_t iCh=0; iCh<9; iCh++){
       lSDetElemCh += lNDetElemCh[iCh];
       if (iDE>=lSDetElemCh) {
-       DEid[iDE] += 100;
-       DEid[iDE] -= lNDetElemCh[iCh];
+       deId[iDE] += 100;
+       deId[iDE] -= lNDetElemCh[iCh];
       }
     }
-    MSDEx[iDE]=fParameters[3*iDE+0];
-    MSDEy[iDE]=fParameters[3*iDE+1];
-    MSDEz[iDE]=fParameters[3*iDE+3];
-    MSDEp[iDE]=fParameters[3*iDE+2];
-    MSDExErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+0);
-    MSDEyErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+1);
-    MSDEzErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+3);
-    MSDEpErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+2);
-    fMSDEx->SetPoint(iDE,DEid[iDE],fParameters[3*iDE+0]);
-    fMSDEx->SetPoint(iDE,DEidErr[iDE],(Double_t)fAlign->GetParError(3*iDE+0));
-    fMSDEy->SetPoint(iDE,DEid[iDE],fParameters[3*iDE+1]);
-    fMSDEy->SetPoint(iDE,DEidErr[iDE],(Double_t)fAlign->GetParError(3*iDE+1));
-    fMSDEp->SetPoint(iDE,DEid[iDE],fParameters[3*iDE+2]);
-    fMSDEp->SetPoint(iDE,DEidErr[iDE],(Double_t)fAlign->GetParError(3*iDE+2));
-    fMSDEz->SetPoint(iDE,DEid[iDE],fParameters[3*iDE+3]);
-    fMSDEz->SetPoint(iDE,DEidErr[iDE],(Double_t)fAlign->GetParError(3*iDE+3));
+    msdEx[iDE]=fParameters[3*iDE+0];
+    msdEy[iDE]=fParameters[3*iDE+1];
+    msdEz[iDE]=fParameters[3*iDE+3];
+    msdEp[iDE]=fParameters[3*iDE+2];
+    msdExErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+0);
+    msdEyErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+1);
+    msdEzErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+3);
+    msdEpErr[iDE]=(Double_t)fAlign->GetParError(3*iDE+2);
+    fMSDEx->SetPoint(iDE,deId[iDE],fParameters[3*iDE+0]);
+    fMSDEx->SetPoint(iDE,deIdErr[iDE],(Double_t)fAlign->GetParError(3*iDE+0));
+    fMSDEy->SetPoint(iDE,deId[iDE],fParameters[3*iDE+1]);
+    fMSDEy->SetPoint(iDE,deIdErr[iDE],(Double_t)fAlign->GetParError(3*iDE+1));
+    fMSDEp->SetPoint(iDE,deId[iDE],fParameters[3*iDE+2]);
+    fMSDEp->SetPoint(iDE,deIdErr[iDE],(Double_t)fAlign->GetParError(3*iDE+2));
+    fMSDEz->SetPoint(iDE,deId[iDE],fParameters[3*iDE+3]);
+    fMSDEz->SetPoint(iDE,deIdErr[iDE],(Double_t)fAlign->GetParError(3*iDE+3));
   }
 
   // Post final data. Write histo list to a file with option "RECREATE"
index 64eca8e..0534147 100644 (file)
@@ -54,8 +54,8 @@
 ClassImp(AliMUONCalibrationData)
 /// \endcond
 
-AliMUONVStore* AliMUONCalibrationData::fBypassPedestals(0x0);
-AliMUONVStore* AliMUONCalibrationData::fBypassGains(0x0);
+AliMUONVStore* AliMUONCalibrationData::fgBypassPedestals(0x0);
+AliMUONVStore* AliMUONCalibrationData::fgBypassGains(0x0);
 
 //_____________________________________________________________________________
 AliMUONCalibrationData::AliMUONCalibrationData(Int_t runNumber, 
@@ -288,7 +288,7 @@ AliMUONVStore*
 AliMUONCalibrationData::Gains() const
 {
   /// Create (if needed) and return the internal store for gains.
-  if (fBypassGains) return fBypassGains;
+  if (fgBypassGains) return fgBypassGains;
   
   if (!fGains)
   {
@@ -419,8 +419,8 @@ void
 AliMUONCalibrationData::BypassStores(AliMUONVStore* ped, AliMUONVStore* gain)
 {
   /// Force the use of those pedestals and gains
-  fBypassPedestals = ped;
-  fBypassGains = gain;
+  fgBypassPedestals = ped;
+  fgBypassGains = gain;
   
 }
 
@@ -430,7 +430,7 @@ AliMUONCalibrationData::Pedestals() const
 {
   /// Return pedestals
   
-  if (fBypassPedestals) return fBypassPedestals;
+  if (fgBypassPedestals) return fgBypassPedestals;
   
   if (!fPedestals)
   {
index 5541af3..c112024 100644 (file)
@@ -170,8 +170,8 @@ private:
   
   mutable AliMUONRejectList* fRejectList; //!< reject list
 
-  static AliMUONVStore* fBypassPedestals;
-  static AliMUONVStore* fBypassGains;
+  static AliMUONVStore* fgBypassPedestals;
+  static AliMUONVStore* fgBypassGains;
   
   mutable AliMUONVStore* fConfig; //!< configuration of the tracker
   
index 7146155..5b458ee 100644 (file)
@@ -77,7 +77,7 @@ ClassImp( AliMUONChamberCalibrationTask )
 AliMUONChamberCalibrationTask::AliMUONChamberCalibrationTask():
   AliAnalysisTaskSE( "AliMUONChamberCalibrationTask" ),
   fOCDBPath( "local://$ALICE_ROOT/OCDB" ),
-  fCalibChoice(NOGAIN),
+  fCalibChoice(kNOGAIN),
   fClusterInfoTree(0x0),
   fMuonRecoParam(0x0),
   fClusterInfo(0x0),
@@ -99,7 +99,7 @@ AliMUONChamberCalibrationTask::AliMUONChamberCalibrationTask( const char* name,
                                                              const Int_t my_calib_option ):
   AliAnalysisTaskSE( name ),
   fOCDBPath( "local://$ALICE_ROOT/OCDB" ),
-  fCalibChoice(NOGAIN),
+  fCalibChoice(kNOGAIN),
   fClusterInfoTree(0x0),
   fMuonRecoParam(0x0),
   fClusterInfo(0x0),
@@ -114,12 +114,12 @@ AliMUONChamberCalibrationTask::AliMUONChamberCalibrationTask( const char* name,
   //
 
   fOCDBPath = ocdbpath;
-  if ( (my_calib_option >= ((Int_t)NOGAIN)) && (my_calib_option <= ((Int_t)INJECTIONGAIN)) ) 
+  if ( (my_calib_option >= ((Int_t)kNOGAIN)) && (my_calib_option <= ((Int_t)kINJECTIONGAIN)) ) 
     fCalibChoice = (Calibration_t)my_calib_option;
   else {
     AliWarning( Form("Wrong value of the calibration option %d not within [%d, %d] !!! Will use NOGAIN", 
-                    my_calib_option, (Int_t)NOGAIN, (Int_t)INJECTIONGAIN ) );
-    fCalibChoice = NOGAIN;
+                    my_calib_option, (Int_t)kNOGAIN, (Int_t)kINJECTIONGAIN ) );
+    fCalibChoice = kNOGAIN;
   }
 }
 
@@ -204,9 +204,9 @@ void AliMUONChamberCalibrationTask::LocalInit()
   TString caliboption4 = "INJECTIONGAIN";
 
   TString caliboption = caliboption1;
-  if ( fCalibChoice == GAINCONSTANTCAPA ) caliboption = caliboption2;
-  if ( fCalibChoice == GAIN ) caliboption = caliboption3;  
-  if ( fCalibChoice == INJECTIONGAIN ) caliboption = caliboption4;
+  if ( fCalibChoice == kGAINCONSTANTCAPA ) caliboption = caliboption2;
+  if ( fCalibChoice == kGAIN ) caliboption = caliboption3;  
+  if ( fCalibChoice == kINJECTIONGAIN ) caliboption = caliboption4;
   fMuonRecoParam->SetCalibrationMode(caliboption.Data());
 
   for (Int_t iCh=0; iCh<10; iCh++) {
index 9e1685c..34488a2 100644 (file)
@@ -35,10 +35,10 @@ class AliMUONChamberCalibrationTask : public AliAnalysisTaskSE
 
   /// enumerate the list of the available modes for the calibration
   enum Calibration_t { 
-    NOGAIN, 
-    GAINCONSTANTCAPA, 
-    GAIN, 
-    INJECTIONGAIN 
+    kNOGAIN, 
+    kGAINCONSTANTCAPA, 
+    kGAIN, 
+    kINJECTIONGAIN 
   }; 
 
   // Constructors
index ee8f727..0468e05 100644 (file)
@@ -238,7 +238,7 @@ AliMUONClusterSplitterMLEM::Fcn1(const AliMUONCluster& cluster,
 }
 
 //_____________________________________________________________________________
-Double_t AliMUONClusterSplitterMLEM::Param2Coef(Int_t icand, Double_t coef, Double_t *par)
+Double_t AliMUONClusterSplitterMLEM::Param2Coef(Int_t icand, Double_t coef, Double_t *par) const
 {
   /// Extract hit contribution scale factor from fit parameters
   
index 00e0f68..efb093c 100644 (file)
@@ -84,7 +84,7 @@ private:
   AliMUONClusterSplitterMLEM(const AliMUONClusterSplitterMLEM&);
   /// will not be implemented
   AliMUONClusterSplitterMLEM& operator=(const AliMUONClusterSplitterMLEM&);
-  Double_t Param2Coef(Int_t icand, Double_t coef, Double_t *par);
+  Double_t Param2Coef(Int_t icand, Double_t coef, Double_t *par) const;
 
 private:
   
index e8331b9..e4c0552 100644 (file)
@@ -149,7 +149,7 @@ AliMUONVCluster* AliMUONClusterStoreV2::Add(const AliMUONVCluster& vCluster)
   // check that there is no cluster with the same Id
   AliMUONVCluster *c = FindObject(cluster->GetUniqueID());
   if (c) {
-    AliError("cluster store already contains a cluster with the same ID --> add() aborted:");
+    AliError("cluster store already contains a cluster with the same ID --> add() exited:");
     c->Print("FULL");
     return 0x0;
   }
@@ -176,7 +176,7 @@ AliMUONVCluster* AliMUONClusterStoreV2::Add(Int_t chamberId, Int_t detElemId, In
   // check that there is no cluster with the same Id
   AliMUONVCluster *c = FindObject(AliMUONVCluster::BuildUniqueID(chamberId, detElemId, clusterIndex));
   if (c) {
-    AliError("cluster store already contains a cluster with the same ID --> add() aborted:");
+    AliError("cluster store already contains a cluster with the same ID --> add() exited:");
     c->Print("FULL");
     return 0x0;
   }
index b42e01d..b010e79 100644 (file)
@@ -496,14 +496,14 @@ void AliMUONGain::MakeGainStore(TString shuttleFile)
 
          Int_t fitproceed=1;
          Int_t nbpf2Dynamic=nbpf2;
-         Int_t ADClimit=4090; // when RMS < 0.5 (in other cases mean values forced to 4095, see DA_PED)
+         Int_t adcLimit=4090; // when RMS < 0.5 (in other cases mean values forced to 4095, see DA_PED)
          for (Int_t j = 0; j < nbs; ++j)
            {
              Int_t k = j + fnInit;
              x[j]    = pedMean[k];
              if(x[j]<=0.){fitproceed=0; break;}
              //              if(x[j]>= ADCMax())
-             if(x[j]>= ADClimit)
+             if(x[j]>= adcLimit)
                {
                  if(j < nbs-1){fitproceed=0; break;}
                  else { nbpf2Dynamic=nbpf2-1; break;}
index 713e71b..4da5cf4 100644 (file)
@@ -492,7 +492,7 @@ void AliMUONGeometryBuilder::CreateGeometryWithoutTGeo()
 }
 
 //_____________________________________________________________________________
-void AliMUONGeometryBuilder::SetAlign(AliMUONVGeometryBuilder* builder)
+void AliMUONGeometryBuilder::SetAlignToBuilder(AliMUONVGeometryBuilder* builder) const
 {
 /// Set align option to all geometry modules associated with the builder
 
@@ -523,7 +523,7 @@ void AliMUONGeometryBuilder::AddBuilder(AliMUONVGeometryBuilder* geomBuilder)
   if (geomBuilder->ApplyGlobalTransformation())
     geomBuilder->SetReferenceFrame(fGlobalTransformation);
   
-  SetAlign(geomBuilder);
+  SetAlignToBuilder(geomBuilder);
 }
 
 //______________________________________________________________________________
@@ -635,7 +635,7 @@ void AliMUONGeometryBuilder::SetAlign(Bool_t align)
     AliMUONVGeometryBuilder* builder
       = (AliMUONVGeometryBuilder*)fGeometryBuilders->At(i);
     
-    SetAlign(builder); 
+    SetAlignToBuilder(builder); 
   }   
 }
 
@@ -652,6 +652,6 @@ void AliMUONGeometryBuilder::SetAlign(const TString& fileName, Bool_t align)
     AliMUONVGeometryBuilder* builder
       = (AliMUONVGeometryBuilder*)fGeometryBuilders->At(i);
     
-    SetAlign(builder); 
+    SetAlignToBuilder(builder); 
   }   
 }
index 88c1b8a..5a47bef 100644 (file)
@@ -72,7 +72,7 @@ class AliMUONGeometryBuilder : public TObject
             const char* only, Bool_t makeAssembly = false) const;
     void CreateGeometryWithTGeo();
     void CreateGeometryWithoutTGeo();
-    void SetAlign(AliMUONVGeometryBuilder* builder);        
+    void SetAlignToBuilder(AliMUONVGeometryBuilder* builder) const;         
 
     // static data members
     static const TString  fgkDefaultVolPathsFileName;  ///< default volume paths file name                                        
index 4e88e0f..639235a 100644 (file)
@@ -28,7 +28,7 @@ class AliMUONGlobalCrateConfig : public  TNamed {
           /// set global crate enbale
     void SetGlobalCrateEnable(UInt_t enable) {fGlobalCrateEnable = enable;}
          /// Get global crate enbale
-    UInt_t GetGlobalCrateEnable() {return fGlobalCrateEnable;}
+    UInt_t GetGlobalCrateEnable() const {return fGlobalCrateEnable;}
     
     // Jtag
             /// Get Jtag board VME address
index 0b3a1db..9965615 100644 (file)
@@ -333,7 +333,7 @@ AliMUONLocalTrigger::GetName() const
 
 
 //----------------------------------------------------------------------
-Bool_t AliMUONLocalTrigger::IsTrigX()
+Bool_t AliMUONLocalTrigger::IsTrigX() const
 {
 /// Trigger response X strips
   Bool_t xTrig;
@@ -345,7 +345,7 @@ Bool_t AliMUONLocalTrigger::IsTrigX()
 
 
 //----------------------------------------------------------------------
-Bool_t AliMUONLocalTrigger::IsTrigY()
+Bool_t AliMUONLocalTrigger::IsTrigY() const
 {
 /// Trigger response Y strips
   Bool_t yTrig;
index 31cac4a..575d815 100644 (file)
@@ -73,9 +73,9 @@ class AliMUONLocalTrigger : public TObject {
   void  SetDeviation(Int_t deviation);
 
   /// Trigger response X strips
-  Bool_t IsTrigX();
+  Bool_t IsTrigX() const;
   /// Trigger response Y strips
-  Bool_t IsTrigY();
+  Bool_t IsTrigY() const;
 
   /// Hit pattern from the re-calculated trigger response after removing chambers one-by-one
   UShort_t GetHitPatternFromResponse() const {return fHitPatternFromResponse; }
index a63b55b..f7365f9 100644 (file)
@@ -306,7 +306,7 @@ AliMUONPedestalEventGenerator::Exec(Option_t*)
     Bool_t dateOutput = ConvertRawFilesToDate();
     if (!dateOutput) 
     {
-      AliError("DATE output failed. Aborting.");
+      AliError("DATE output failed. Exiting.");
       return;
     }    
   }
index 0bc48ee..5a15f05 100644 (file)
@@ -84,7 +84,7 @@ public:
   virtual Int_t    GetMCLabel() const {return GetTrack(0);}
   
   /// Return radius
-   Float_t      GetRadius(Int_t i) {return TMath::Sqrt(fX[i]*fX[i]+fY[i]*fY[i]);}
+   Float_t      GetRadius(Int_t i) const {return TMath::Sqrt(fX[i]*fX[i]+fY[i]*fY[i]);}
    /// Return true as the function Compare() is implemented
    Bool_t       IsSortable() const {return kTRUE;}
    Int_t        Compare(const TObject *obj) const;
index a908ea1..a46ef39 100644 (file)
@@ -78,10 +78,10 @@ void AliMUONRawStream::Swap(UInt_t* buffer, Int_t size) const
 
     for (int i = 0 ; i < size; i++) {
       temp = *(((RawWord*)buffer)+i);
-      word->b1 = temp.b4;
-      word->b2 = temp.b3;
-      word->b3 = temp.b2;
-      word->b4 = temp.b1;
+      word->fB1 = temp.fB4;
+      word->fB2 = temp.fB3;
+      word->fB3 = temp.fB2;
+      word->fB4 = temp.fB1;
       word++;
     }
 
index 6a12e30..b600483 100644 (file)
@@ -56,10 +56,10 @@ class AliMUONRawStream: public TObject {
     AliMUONRawStream& operator = (const AliMUONRawStream& stream);
 
     typedef struct {
-     UInt_t b1:8; ///< first byte word
-     UInt_t b2:8; ///< second byte word
-     UInt_t b3:8; ///< third byte word
-     UInt_t b4:8; ///< fourth byte word
+     UInt_t fB1:8; ///< first byte word
+     UInt_t fB2:8; ///< second byte word
+     UInt_t fB3:8; ///< third byte word
+     UInt_t fB4:8; ///< fourth byte word
     } RawWord;
 
     AliRawReader* fRawReader;    //!< object for reading the raw data  
index 4f493c3..41c6e1d 100644 (file)
@@ -50,9 +50,6 @@
 ClassImp(AliMUONRawStreamTracker)
 /// \endcond
 
-const Int_t AliMUONRawStreamTracker::fgkMaxDDL = 20;
-
-
 //___________________________________________
 AliMUONRawStreamTracker::AliMUONRawStreamTracker()
  : AliMUONVRawStreamTracker(),
index 59c94c6..ae0339d 100644 (file)
@@ -49,8 +49,6 @@ class AliMUONRawStreamTracker: public AliMUONVRawStreamTracker {
     
     virtual Bool_t NextDDL();
 
-    /// Return maximum number of DDLs
-    Int_t GetMaxDDL() const {return fgkMaxDDL;}
     /// Return maximum number of block per DDL in DATE file
     Int_t GetMaxBlock() const {return  fPayload->GetMaxBlock();}
     /// Return maximum number of Dsp per block in DATE file
@@ -124,7 +122,6 @@ class AliMUONRawStreamTracker: public AliMUONVRawStreamTracker {
     Int_t fCurrentBusStructIndex;            //!< for iterator: current bus index    
     Int_t fCurrentDataIndex;                 //!< for iterator: current data index
     Int_t  fDDL;                             //!< number of DDL
-    static const Int_t  fgkMaxDDL;           //!< maximum number of DDLs
 
     ClassDef(AliMUONRawStreamTracker, 4)    // base class for reading MUON raw digits
 };
index 7378c4e..c0c48f9 100644 (file)
@@ -64,8 +64,7 @@ AliMUONRawStreamTrigger::AliMUONRawStreamTrigger()
     fCurrentLocalStructIndex(0),
     fLocalStructRead(kFALSE),
     fDDL(0),
-    fNextDDL(kFALSE),
-    fEnableErrorLogger(kFALSE)
+    fNextDDL(kFALSE)
 {
   ///
   /// create an object to read MUON raw digits
@@ -88,8 +87,7 @@ AliMUONRawStreamTrigger::AliMUONRawStreamTrigger(AliRawReader* rawReader)
     fCurrentLocalStructIndex(0),
     fLocalStructRead(kFALSE),
     fDDL(0),
-    fNextDDL(kFALSE),
-    fEnableErrorLogger(kFALSE)
+    fNextDDL(kFALSE)
 {
   ///
   /// ctor with AliRawReader as argument
index 27174a1..07bb522 100644 (file)
@@ -114,7 +114,6 @@ class AliMUONRawStreamTrigger: public AliMUONVRawStreamTrigger {
 
 
     Bool_t fNextDDL;      ///< flag for next DDL to be read
-    Bool_t fEnableErrorLogger; //!< flag to enable the error info logger
 
     static const Int_t  fgkMaxDDL;       ///< maximum number of DDLs
 
index e02fdc4..b47ed98 100644 (file)
@@ -57,10 +57,10 @@ public:
   Int_t NumberOfEvents() const;
   
   /// Return the reconstructed data of current event
-  const AliESDEvent* GetESDEvent() { return fESDEvent; }
+  const AliESDEvent* GetESDEvent() const { return fESDEvent; }
   
   /// Return the interface to the Monte Carlo data of current event
-  const AliMCEventHandler* GetMCEventHandler() { return fMCEventHandler; }
+  const AliMCEventHandler* GetMCEventHandler() const { return fMCEventHandler; }
   
   /// Return the track from the store matched with the given track (or 0x0) and the fraction of matched clusters.
   static AliMUONTrack* FindCompatibleTrack(AliMUONTrack &track, AliMUONVTrackStore &trackStore,
index ffd19b2..22eb9fb 100644 (file)
@@ -233,7 +233,7 @@ void AliMUONResponseTriggerV1::DisIntegrate(const AliMUONHit& hit, TList& digits
 //------------------------------------------------------------------  
 void AliMUONResponseTriggerV1::Neighbours(const Int_t cath, 
                                          const Int_t ix, const Int_t iy, 
-                                         Int_t Xlist[10], Int_t Ylist[10]) 
+                                         Int_t Xlist[10], Int_t Ylist[10]) const
 {
     ///-----------------BENDING-----------------------------------------      /n
     /// Returns list of 10 next neighbours for given X strip (ix, iy)         /n
index 772721f..74761be 100644 (file)
@@ -37,7 +37,7 @@ class AliMUONResponseTriggerV1 : public AliMUONResponseTrigger
     void SetParameters(Float_t hv);
     // parametrization of the cluster-size
     Float_t FireStripProb(Float_t x4, Float_t theta) const;
-    void Neighbours(const Int_t cath, const Int_t iX, const Int_t iY, Int_t Xlist[10], Int_t Ylist[10]);
+    void Neighbours(const Int_t cath, const Int_t iX, const Int_t iY, Int_t Xlist[10], Int_t Ylist[10]) const;
     
   ClassDef(AliMUONResponseTriggerV1,1) // Implementation of RPC response
     
index 801a505..a4d596d 100644 (file)
@@ -144,7 +144,7 @@ AliMUONSDigitizerV2::Exec(Option_t*)
     runLoader->GetEvent(iEvent);
   
     // for pile up studies
-    float T0=fgkMaxIntTime;  int AA=0;
+    float t0=fgkMaxIntTime;  int aa=0;
     AliHeader* header = runLoader->GetHeader();   
     AliGenCocktailEventHeader* cocktailHeader =
       dynamic_cast<AliGenCocktailEventHeader*>(header->GenEventHeader());
@@ -155,15 +155,15 @@ AliMUONSDigitizerV2::Exec(Option_t*)
       AliGenEventHeader *entry; 
       while((entry = (AliGenEventHeader*)nextH())) {
        float t = entry->InteractionTime();     
-       if (TMath::Abs(t)<TMath::Abs(T0)) T0 = t;      
-       AA++;
+       if (TMath::Abs(t)<TMath::Abs(t0)) t0 = t;      
+       aa++;
       }
     } else {
       AliGenEventHeader* evtHeader = 
        (AliGenEventHeader*)(header->GenEventHeader());
       float t = evtHeader->InteractionTime();          
-      if (TMath::Abs(t)<TMath::Abs(T0)) T0 = t;           
-      AA++;
+      if (TMath::Abs(t)<TMath::Abs(t0)) t0 = t;           
+      aa++;
     }
 
     loader->MakeSDigitsContainer();
@@ -196,14 +196,14 @@ AliMUONSDigitizerV2::Exec(Option_t*)
       while ( ( hit = static_cast<AliMUONHit*>(next()) ) )       
       {
        Int_t chamberId = hit->Chamber()-1;
-       Float_t age = hit->Age()-T0;
+       Float_t age = hit->Age()-t0;
 
         AliMUONChamber& chamber = muon->Chamber(chamberId);
         AliMUONResponse* response = chamber.ResponseModel();
         
         // This is the heart of this method : the dis-integration
         TList digits;        
-       if (AA>1){  // if there are pileup events
+       if (aa>1){  // if there are pileup events
          Float_t chamberTime = AliMUONConstants::AverageChamberT(chamberId);
          Float_t timeDif=age-chamberTime;        
          if (timeDif>fgkMaxPosTimeDif || timeDif<fgkMaxNegTimeDif) {
index 7582223..f15460c 100644 (file)
@@ -380,7 +380,7 @@ Bool_t AliMUONTrackLight::IsParentPionOrKaon(Int_t idparent){
   else return kFALSE;
 }
 //====================================
-Bool_t AliMUONTrackLight::IsDiquark(Int_t pdg){
+Bool_t AliMUONTrackLight::IsDiquark(Int_t pdg) const{
   /// check if the provided pdg code corresponds to a diquark 
   pdg = TMath::Abs(pdg);
   if((pdg > 1000) && (pdg%100 < 10)) return kTRUE;
index 1152912..b51b127 100644 (file)
@@ -49,21 +49,21 @@ class AliMUONTrackLight : public TObject {
   /// Set primary vertex position from the ITS
   void SetVertex(Double_t *xyz) {for (Int_t i=0; i<3; i++) fXYZ[i]=xyz[i];}
   /// Return primary vertex x position from the ITS 
-  Double_t GetX() { return fXYZ[0]; } 
+  Double_t GetX() const { return fXYZ[0]; } 
   /// Return primary vertex y position from the ITS 
-  Double_t GetY() { return fXYZ[1]; } 
+  Double_t GetY() const { return fXYZ[1]; } 
   /// Return primary vertex z position from the ITS 
-  Double_t GetZ() { return fXYZ[2]; } 
+  Double_t GetZ() const { return fXYZ[2]; } 
   /// Return  primary vertex position from the ITS
   Double_t* GetVertex() { return fXYZ; } 
   /// Set chi2 / ndf in the MUON track fit
   void SetChi2(Double_t chi2) {fChi2=chi2;}
   /// Return chi2 / ndf in the MUON track fit 
-  Double_t GetChi2() { return fChi2; } 
+  Double_t GetChi2() const { return fChi2; } 
   /// Set weight assigned to the muon
   void SetWeight(Double_t w) {fWeight=w;}
   /// Return weight assigned to the muon 
-  Double_t GetWeight() { return fWeight; } 
+  Double_t GetWeight() const { return fWeight; } 
   
   /// Set muon charge 
   void SetCharge(Int_t charge) {fCharge = charge;}
@@ -106,7 +106,7 @@ class AliMUONTrackLight : public TObject {
   //"K" muon kinematics
   //"A" all variables
   Int_t GetParentFlavour(Int_t idParent=0) const;
-  Bool_t IsDiquark(Int_t pdg);
+  Bool_t IsDiquark(Int_t pdg) const;
 protected:
   static const Int_t fgkNParentsMax = 5;   ///< maximum number of parents
   TLorentzVector fPrec; ///< reconstructed 4-momentum
index b36bbff..e542ab4 100755 (executable)
@@ -9,8 +9,8 @@
 ///
 //  Author: Diego Stocco; Subatech, Nantes
 
-#ifndef ALIMUONTRIGGEREFFICIENCY_H
-#define ALIMUONTRIGGEREFFICIENCY_H
+#ifndef ALIMUONTRIGGERCHAMBEREFFICIENCY_H
+#define ALIMUONTRIGGERCHAMBEREFFICIENCY_H
 
 #include "TObject.h"
 class TH1;
index 0401b06..5af7efa 100644 (file)
@@ -68,7 +68,7 @@ public:
 private:
   Bool_t AddSortedPoint(Float_t currVal, TArrayD& position, const Float_t kResetValue);
   /// Return index
-  Int_t GetIndex(Int_t chamber, Int_t cathode) { return 2*chamber + cathode;}
+  Int_t GetIndex(Int_t chamber, Int_t cathode) const { return 2*chamber + cathode;}
 
   Bool_t InitOrDisplayTriggerInfo(TObject* inputHisto, TH2* displayHisto,
                                  EDisplayType displayType,
index 2d06c8a..3189558 100755 (executable)
@@ -255,7 +255,7 @@ AliMUONTriggerEfficiencyCells::GetHistoName(Int_t histoType, Int_t countType,
 
 //__________________________________________________________________________
 TH1F* AliMUONTriggerEfficiencyCells::GetOldEffHisto(Int_t histoType,
-                                                   Int_t ich, Int_t icath)
+                                                   Int_t ich, Int_t icath) const
 {
   //
   /// Compatibility with old class
index 28f8a50..4df354b 100755 (executable)
@@ -48,7 +48,7 @@ public:
   /// Get list of histograms
   TList* GetHistoList() { return fCountHistoList; }
 
-  TH1F* GetOldEffHisto(Int_t hType, Int_t ich, Int_t icath); // obsolete
+  TH1F* GetOldEffHisto(Int_t hType, Int_t ich, Int_t icath) const; // obsolete
 
 protected:
     void ResetHistos(Bool_t deleteObjects = kFALSE);
index bea0114..a0e6e20 100644 (file)
@@ -149,7 +149,7 @@ AliMUONTriggerSubprocessor::Initialize(Int_t run, UInt_t startTime, UInt_t endTi
 
   if ( check ) 
   {
-    Master()->Log("Could not read some input file(s). Aborting");
+    Master()->Log("Could not read some input file(s). Exiting");
     Master()->Invalidate();
     return kFALSE;
   }
index bfb67cb..fe16b5f 100644 (file)
@@ -271,20 +271,20 @@ protected:
     
   virtual TCollection* Children() const;
 
-  mutable TH1* fHistogram; //!< histogram
-  
+  void CreateGroups();
+
   AliMUONVTrackerData* InteractiveReadOutConfig() const;
   
-private:
+  mutable TH1* fHistogram; //!< histogram
   
-    void FlatList(TList& list);
+  TMap* fPainterGroups; ///< map of groups
+  AliMUONPainterGroup* fResponderGroup; ///< the responder group
 
-  AliMUONPainterGroup* CreateGroup(const char* type, Int_t depth);
+private:
   
-protected:
-  void CreateGroups();
+  void FlatList(TList& list);
 
-private:
+  AliMUONPainterGroup* CreateGroup(const char* type, Int_t depth);
   
   void GetBoundingBox(Double_t& x1, Double_t& y1, 
                       Double_t& x2, Double_t& y2) const;
@@ -295,21 +295,13 @@ private:
   
   void GetIROCManuList(TObjArray& manuList);
 
-private:
-  
   TString fName; ///< our (short) name
   TString fPathName; ///< our long name
   TString fType; ///< our type (DE, Chamber, MANU, etc...)
   AliMUONVPainter* fMother;  ///< our mother
   AliMUONPainterGroup* fGroup; ///< our group
   AliMUONContour* fContour;  ///< our contour
-protected:
-  TMap* fPainterGroups; ///< map of groups
-private:
   TObjArray* fChildren; ///< our children
-protected:
-  AliMUONPainterGroup* fResponderGroup; ///< the responder group
-private:
   AliMUONPainterGroup* fPlotterGroup; ///< the plotter group
   Double_t fBorderFactor; ///< border factor for visu 
   TVirtualPad* fPad; ///< the pad we're drawn in
index 7a62763..9b83e61 100644 (file)
@@ -94,15 +94,14 @@ protected:
   Int_t Add2RecPointsList(TH1 * hist, const Int_t index, const Bool_t expert = kFALSE, const Bool_t image = kFALSE);                 
   Int_t Add2RawsList(TH1 * hist, const Int_t index, const Bool_t expert = kFALSE, const Bool_t image = kFALSE, const Bool_t saveForCorr = kFALSE);  
   
+  AliQADataMakerRec* fMaster; ///< master to get access to its methods
+
 private:
   /// Not implemented
   AliMUONVQADataMakerRec(const AliMUONVQADataMakerRec& rhs);
   /// Not implemented
   AliMUONVQADataMakerRec& operator=(const AliMUONVQADataMakerRec& rhs);
   
-protected:
-  AliQADataMakerRec* fMaster; ///< master to get access to its methods
-  
   ClassDef(AliMUONVQADataMakerRec,1) // Interface for a MUON QADataMakerRec
 };
 
index 62e343a..49f7d24 100644 (file)
@@ -69,6 +69,10 @@ public:
                kParityErr      = 3  ///< parity error
        };
 
+protected:     
+
+       static const Int_t fgkMaxDDL;   //!< maximum number of DDLs
+
 private:
 
        // Do not allow copying of this class.
@@ -77,8 +81,6 @@ private:
         /// Not implemented
        AliMUONVRawStreamTracker& operator = (const AliMUONVRawStreamTracker& stream);
 
-       static const Int_t fgkMaxDDL;   //!< maximum number of DDLs
-
        ClassDef(AliMUONVRawStreamTracker, 0) // Base class for reading MUON raw digits from tracking chambers.
 };
 
index add3114..4f434a8 100644 (file)
@@ -91,13 +91,13 @@ class AliMpLocalBoard : public TNamed
     // Id to be copy to or from
     
     /// Get Id from where the X input are copied
-    Int_t GetInputXfrom() {return fInputXfrom;}
+    Int_t GetInputXfrom() const {return fInputXfrom;}
     /// Get Id to where the X input are copied
-    Int_t GetInputXto()   {return fInputXto;}
+    Int_t GetInputXto() const   {return fInputXto;}
     /// Get Id from where the Y input are copied
-    Int_t GetInputYfrom() {return fInputYfrom;}
+    Int_t GetInputYfrom() const {return fInputYfrom;}
     /// Get Id to where the Y input are copied
-    Int_t GetInputYto()   {return fInputYto;}
+    Int_t GetInputYto() const  {return fInputYto;}
 
     /// Set Id from where the X input are copied 
     void SetInputXfrom(Int_t id) {fInputXfrom = id;}
index 5b16950..58e6bcb 100755 (executable)
@@ -497,7 +497,7 @@ AliMpMotifType::Save(const char* motifName) const
     << "#connecteur_berg kapton padname not_used" << endl
     << "#for slats there's no kapton connector, so it's always 1" 
     << " (zero make the reader" << endl
-    << "#abort, so it's not a valid value here)." << endl
+    << "#exit, so it's not a valid value here)." << endl
     << "#" << endl;
   
   for ( Int_t ix = 0; ix < GetNofPadsX(); ++ix ) 
index 4160d37..4cf9e3c 100644 (file)
@@ -183,7 +183,7 @@ AliMpPCBPadIterator::First()
 
 //_____________________________________________________________________________
 Bool_t
-AliMpPCBPadIterator::GetNextPosition(Int_t& ix, Int_t& iy)
+AliMpPCBPadIterator::GetNextPosition(Int_t& ix, Int_t& iy) const
 {
   /// Get the next iteration position. 
   /// On input, fOffset must be a valid position (i.e. within iteration
index c99391e..c5b27dd 100644 (file)
@@ -39,7 +39,7 @@ private:
   /// Not implemented
   AliMpPCBPadIterator&  operator = (const AliMpPCBPadIterator& right);
   
-  Bool_t GetNextPosition(Int_t& ix, Int_t& iy);
+  Bool_t GetNextPosition(Int_t& ix, Int_t& iy) const;
   Bool_t CropArea(const AliMpArea& area);
   void SetPad(AliMpPad& pad, Int_t ix, Int_t iy);
   
index a7d433a..82efbca 100644 (file)
@@ -337,7 +337,7 @@ AliMpTriggerReader::FlipLines(TList& lines, Bool_t flipX, Bool_t flipY,
 
 //___________________________________________________________________________
 Int_t
-AliMpTriggerReader::IsLayerLine(const TString& sline)
+AliMpTriggerReader::IsLayerLine(const TString& sline) const
 {
   /// Whether sline contains LAYER keyword
 
index f9ce61d..5f9440c 100644 (file)
@@ -68,7 +68,7 @@ private:
   
   Int_t GetLine(const TString& slatType);
   
-  Int_t IsLayerLine(const TString& sline);
+  Int_t IsLayerLine(const TString& sline) const;
     
   int LocalBoardNumber(const char* localBoardName);