Coding conventions (E. Fragiacomo)
authormasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 8 Dec 2011 17:09:21 +0000 (17:09 +0000)
committermasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 8 Dec 2011 17:09:21 +0000 (17:09 +0000)
13 files changed:
ITS/AliITSBadChannelsSSD.h
ITS/AliITSBadChannelsSSDv2.h
ITS/AliITSCalibrationSSD.h
ITS/AliITSGainSSD.h
ITS/AliITSGainSSDv2.h
ITS/AliITSHandleDaSSD.cxx
ITS/AliITSNoiseSSD.h
ITS/AliITSNoiseSSDv2.h
ITS/AliITSPedestalSSD.h
ITS/AliITSPedestalSSDv2.h
ITS/AliITSQASSDDataMakerRec.cxx
ITS/AliITSQASSDDataMakerRec.h
ITS/AliITSQASSDDataMakerSim.h

index 0a43226..610680c 100644 (file)
@@ -25,7 +25,7 @@ class AliITSBadChannelsSSD : public TObject {
     //
 
     void SetMod(UShort_t mod) {fMod = mod;}
-    UShort_t GetMod() { return fMod;}
+    UShort_t GetMod() const { return fMod;}
 
  protected:
 
index ce0e5a0..5088175 100644 (file)
@@ -21,24 +21,24 @@ class AliITSBadChannelsSSDv2 : public TObject {
     void AddBadChannelP(Int_t module, Int_t strip, Char_t value) { 
       fBadChannels[module*2*fgkDefaultNStripsSSD+strip] = value;
     }       
-    Char_t GetBadChannelP(Int_t module, Int_t strip) {
+    Char_t GetBadChannelP(Int_t module, Int_t strip) const {
       return fBadChannels[module*2*fgkDefaultNStripsSSD+strip]; 
     }
 
     void AddBadChannelN(Int_t module, Int_t strip, Char_t value) { 
       fBadChannels[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip] = value;
     }       
-    Char_t GetBadChannelN(Int_t module, Int_t strip) {
+    Char_t GetBadChannelN(Int_t module, Int_t strip) const  {
       return fBadChannels[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip]; 
     }
 
  protected:
 
-    static const Int_t fgkDefaultNModulesSSD = 1698;
-    static const Int_t fgkDefaultNStripsSSD = 768;
+    static const Int_t fgkDefaultNModulesSSD = 1698; // # of default SSD modules
+    static const Int_t fgkDefaultNStripsSSD = 768;  // # of default SSD strips
     //static const Int_t fgkDefaultNModulesSSD; // Total numbers of SSD modules
    //   static const Int_t fgkDefaultNStripsSSD; // Total numbers of SSD modules
-   Char_t fBadChannels[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD];
+    Char_t fBadChannels[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; // tot # of default SSD strips
     //   Char_t *fBadChannels; 
 
  private:
index 049417e..1468adc 100644 (file)
@@ -108,10 +108,10 @@ class AliITSCalibrationSSD : public AliITSCalibration {
     Int_t   fNPar;            // Number of detector param 
     Double_t *fDetPar;         //[fNPar] Array of parameters
 
-    AliITSNoiseSSDv2 *fNoise;
-    AliITSPedestalSSDv2 *fPedestal;
-    AliITSGainSSDv2 *fGain;
-    AliITSBadChannelsSSDv2 *fBadChannels;
+    AliITSNoiseSSDv2 *fNoise; // pointer to noise calib object    
+    AliITSPedestalSSDv2 *fPedestal; // pointer to pedestal calib object
+    AliITSGainSSDv2 *fGain; // pointer to gain calib object
+    AliITSBadChannelsSSDv2 *fBadChannels; // pointer to bad channels calib object
 
     Bool_t   fIsBad;                         // module is dead or alive ?
     Bool_t   fIsChipBad[fgkChipsPerModule];  // chip is dead or alive ?
index 9291347..857c452 100644 (file)
@@ -9,6 +9,7 @@
 // Date: 23/08/2007
 //                                          //
 //////////////////////////////////////////////
+
 class AliITSGainSSD : public TObject {
 
  public:
@@ -26,7 +27,7 @@ class AliITSGainSSD : public TObject {
     Float_t GetGainN(Int_t n) {return fGainN.At(n); }
 
     void SetMod(UShort_t mod) {fMod = mod;}
-    UShort_t GetMod() { return fMod;}
+    UShort_t GetMod() const { return fMod;}
 
 protected:
 
index 5711c4c..0e09a2c 100644 (file)
@@ -21,7 +21,7 @@ class AliITSGainSSDv2 : public TObject {
     void AddGainP(Int_t module, Int_t strip, Float_t value) { 
       fGain[module*2*fgkDefaultNStripsSSD+strip] = (UShort_t) (1000.*value);
     }       
-    Float_t GetGainP(Int_t module, Int_t strip) {
+    Float_t GetGainP(Int_t module, Int_t strip) const {
       return ((Float_t) ( (Float_t) fGain[module*2*fgkDefaultNStripsSSD+strip] ) /1000.); 
     }
 
@@ -29,19 +29,19 @@ class AliITSGainSSDv2 : public TObject {
       fGain[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip] = 
        (UShort_t) (1000.*value);
     }       
-    Float_t GetGainN(Int_t module, Int_t strip) {
+    Float_t GetGainN(Int_t module, Int_t strip) const {
       return ( (Float_t) ((Float_t) fGain[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip])/1000.); 
     }
 
  protected:
 
-    static const Int_t fgkDefaultNModulesSSD = 1698;
-    static const Int_t fgkDefaultNStripsSSD = 768;
+    static const Int_t fgkDefaultNModulesSSD = 1698; // # of default SSD modules
+    static const Int_t fgkDefaultNStripsSSD = 768;// # of default SSD strips
 
     //static const Int_t fgkDefaultNModulesSSD; // Total numbers of SSD modules
     //static const Int_t fgkDefaultNStripsSSD; // Total numbers of SSD modules
 
-    UShort_t fGain[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; 
+    UShort_t fGain[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; // tot # of default SSD strips
     //   UShort_t *fGain; 
 
  private:
index fc01e7b..9ad823a 100644 (file)
@@ -28,7 +28,7 @@
 #include <new>
 #include <Riostream.h> 
 #include "AliITSHandleDaSSD.h"
-#include <math.h>
+//#include <math.h>
 #include <limits.h>
 #include "event.h"
 #include "TFile.h"
@@ -707,7 +707,7 @@ Bool_t AliITSHandleDaSSD::CalculatePedestal(const AliITSModuleDaSSD *const modul
       if (SignalOutOfRange(signal[ev])) ovev += 1;
       else nsum += pow((signal[ev] - strip->GetPedestal()), 2);
     } 
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  sqrt(nsum / (Double_t)(n));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  TMath::Sqrt(nsum / (Double_t)(n));
     else  noise = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetNoise(static_cast<Float_t>(noise));
 //************* pedestal second pass ****************
@@ -748,7 +748,7 @@ Bool_t AliITSHandleDaSSD::CalculateNoise(const AliITSModuleDaSSD *const module)
       if (SignalOutOfRange(signal[ev])) ovev += 1;
       else nsum += pow((signal[ev] - strip->GetPedestal()), 2);
     } 
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  static_cast<Float_t>(sqrt(nsum / (Float_t)(n)));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  static_cast<Float_t>(TMath::Sqrt(nsum / (Float_t)(n)));
     else  noise = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetNoise(noise);
   }
@@ -784,7 +784,7 @@ Bool_t AliITSHandleDaSSD::CalculateNoiseCM(AliITSModuleDaSSD *const module)
       if (SignalOutOfRange(signal[ev])) ovev += 1;
       else nsum += pow((signal[ev] - strip->GetPedestal() - module->GetCM(chipind, ev)), 2);
     } 
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = static_cast<Float_t>(sqrt(nsum / (Double_t)(n)));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = static_cast<Float_t>(TMath::Sqrt(nsum / (Double_t)(n)));
     else  noise = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetNoiseCM(noise);
   }
@@ -835,7 +835,7 @@ Bool_t AliITSHandleDaSSD::CalculateCM(AliITSModuleDaSSD *const module)
         if ((SignalOutOfRange(signal[ev])) || (strip->GetPedestal() == AliITSChannelDaSSD::GetUndefinedValue())) ovstr += 1;
         else cmsigma += pow((cm0 - (signal[ev] - strip->GetPedestal())), 2);
       }
-      if ((n = AliITSModuleDaSSD::GetStripsPerChip() - ovstr)) cmsigma = sqrt(cmsigma / (Float_t)(n));
+      if ((n = AliITSModuleDaSSD::GetStripsPerChip() - ovstr)) cmsigma = TMath::Sqrt(cmsigma / (Float_t)(n));
       else { module->SetCM(0.0f, chipind, ev); continue; }
    // calculate cm with threshold
       Double_t cmsum = 0.0L;
@@ -1525,7 +1525,7 @@ Bool_t AliITSHandleDaSSD::CalculatePedNoiseW(const AliITSModuleDaSSD *const modu
     if (strip->GetEventsNumber() == ovev) pedestal = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetPedestal(static_cast<Float_t>(pedestal));
     if ((n = strip->GetEventsNumber() - ovev - 1) > 0) 
-      strip->SetNoise( static_cast<Float_t>(sqrt(noise / static_cast<Double_t>(n))) );
+      strip->SetNoise( static_cast<Float_t>(TMath::Sqrt(noise / static_cast<Double_t>(n))) );
     else {
       strip->SetNoise(AliITSChannelDaSSD::GetUndefinedValue());
       continue;
@@ -1547,7 +1547,7 @@ Bool_t AliITSHandleDaSSD::CalculatePedNoiseW(const AliITSModuleDaSSD *const modu
           noise = s0;
         }
     }      
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = sqrt(noise / static_cast<Double_t>(n));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = TMath::Sqrt(noise / static_cast<Double_t>(n));
     else  noise = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetNoise(static_cast<Float_t>(noise));
     if (strip->GetEventsNumber() == ovev) pedestal = AliITSChannelDaSSD::GetUndefinedValue();
@@ -1591,7 +1591,7 @@ Bool_t AliITSHandleDaSSD::CalculateCMW(AliITSModuleDaSSD *const module)
           cmsigma = cms1;
         } }
       }
-      if ((n = AliITSModuleDaSSD::GetStripsPerChip() - ovstr - 1) > 0) cmsigma = sqrt(cmsigma / static_cast<Double_t>(n));
+      if ((n = AliITSModuleDaSSD::GetStripsPerChip() - ovstr - 1) > 0) cmsigma = TMath::Sqrt(cmsigma / static_cast<Double_t>(n));
       else {
         AliWarning(Form("AliITSHandleDaSSD: Too little number of strips have a signal for module:chip:event : [%d]:[%d]:[%ld]\n",
                    module->GetModuleId(), chipind, ev));
@@ -1657,7 +1657,7 @@ Bool_t AliITSHandleDaSSD::CalculateNoiseCMW(AliITSModuleDaSSD *const module)
           noise = s0;
         }
     }
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = sqrt(noise / static_cast<Double_t>(n));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise = TMath::Sqrt(noise / static_cast<Double_t>(n));
     else  {
          strip->SetNoiseCM(AliITSChannelDaSSD::GetUndefinedValue());
          continue;
@@ -1671,7 +1671,7 @@ Bool_t AliITSHandleDaSSD::CalculateNoiseCMW(AliITSModuleDaSSD *const module)
           || TMath::Abs(signal[ev] - pedestal) > (fPedestalThresholdFactor * noise)) ovev += 1;
       else nsum += pow((signal[ev] - strip->GetPedestal() - module->GetCM(chipind, ev)), 2);
     } 
-    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  sqrt(nsum / static_cast<Double_t>(n));
+    if ((n = strip->GetEventsNumber() - ovev - 1) > 0) noise =  TMath::Sqrt(nsum / static_cast<Double_t>(n));
     else  noise = AliITSChannelDaSSD::GetUndefinedValue();
     strip->SetNoiseCM(static_cast<Float_t>(noise));
   }
index e389cf1..86b8040 100644 (file)
@@ -28,7 +28,7 @@ class AliITSNoiseSSD : public TObject {
     Float_t GetNoiseN(Int_t n) {return fNoisN.At(n); }
 
     void SetMod(UShort_t mod) {fMod = mod;}
-    UShort_t GetMod() { return fMod;}
+    UShort_t GetMod() const { return fMod;}
 
 protected:
 
index 3ddd232..f80f8fe 100644 (file)
@@ -21,26 +21,26 @@ class AliITSNoiseSSDv2 : public TObject {
     void AddNoiseP(Int_t module, Int_t strip, Float_t value) { 
       fNois[module*2*fgkDefaultNStripsSSD+strip] = value;
     }       
-    Float_t GetNoiseP(Int_t module, Int_t strip) {
+    Float_t GetNoiseP(Int_t module, Int_t strip) const {
       return fNois[module*2*fgkDefaultNStripsSSD+strip]; 
     }
 
     void AddNoiseN(Int_t module, Int_t strip, Float_t value) { 
       fNois[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip] = value;
     }       
-    Float_t GetNoiseN(Int_t module, Int_t strip) {
+    Float_t GetNoiseN(Int_t module, Int_t strip) const {
       return fNois[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip]; 
     }
 
  protected:
 
-    static const Int_t fgkDefaultNModulesSSD = 1698;
-    static const Int_t fgkDefaultNStripsSSD = 768;
+    static const Int_t fgkDefaultNModulesSSD = 1698;// # of default SSD modules
+    static const Int_t fgkDefaultNStripsSSD = 768;// # of default SSD strips
 
     //   static const Int_t fgkDefaultNModulesSSD; // Total numbers of SSD modules
     //static const Int_t fgkDefaultNStripsSSD; // Total numbers of SSD modules
 
-     Float_t fNois[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; 
+     Float_t fNois[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; // tot # of default SSD strips
      //Float_t *fNois;
 
  private:
index 7afc2e7..4e2d1db 100644 (file)
@@ -31,7 +31,7 @@ class AliITSPedestalSSD : public TObject {
     Float_t GetPedestalN(Int_t n) {return fPedN.At(n); }
 
     void SetMod(UShort_t mod) {fMod = mod;}
-    UShort_t GetMod() { return fMod;}
+    UShort_t GetMod() const { return fMod;}
 
 protected:
 
index 7fcf0fe..9e84cd6 100644 (file)
@@ -21,26 +21,26 @@ class AliITSPedestalSSDv2 : public TObject {
     void AddPedestalP(Int_t module, Int_t strip, Float_t value) { 
       fPedestal[module*2*fgkDefaultNStripsSSD+strip] = value;
     }       
-    Float_t GetPedestalP(Int_t module, Int_t strip) {
+    Float_t GetPedestalP(Int_t module, Int_t strip) const {
       return fPedestal[module*2*fgkDefaultNStripsSSD+strip]; 
     }
 
     void AddPedestalN(Int_t module, Int_t strip, Float_t value) { 
       fPedestal[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip] = value;
     }       
-    Float_t GetPedestalN(Int_t module, Int_t strip) {
+    Float_t GetPedestalN(Int_t module, Int_t strip) const {
       return fPedestal[module*2*fgkDefaultNStripsSSD+fgkDefaultNStripsSSD+strip]; 
     }
 
  protected:
 
-    static const Int_t fgkDefaultNModulesSSD = 1698;
-    static const Int_t fgkDefaultNStripsSSD = 768;
+    static const Int_t fgkDefaultNModulesSSD = 1698;// # of default SSD modules
+    static const Int_t fgkDefaultNStripsSSD = 768;// # of default SSD strips
 
     //   static const Int_t fgkDefaultNModulesSSD; // Total numbers of SSD modules
     //static const Int_t fgkDefaultNStripsSSD; // Total numbers of SSD modules
 
-Float_t fPedestal[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD]; 
+Float_t fPedestal[2*fgkDefaultNModulesSSD*fgkDefaultNStripsSSD];  // tot # of default SSD strips
 
  private:
     
index 90e0808..6e6f0c3 100644 (file)
@@ -184,6 +184,7 @@ AliITSQASSDDataMakerRec::~AliITSQASSDDataMakerRec() {
 //____________________________________________________________________________ 
 void AliITSQASSDDataMakerRec::StartOfDetectorCycle()
 {
+  // start of detector cycle
 
   if(fAliITSQADataMakerRec->ListExists(AliQAv1::kRAWS)==kFALSE)return;
 
index 3dcfe30..3786425 100644 (file)
@@ -1,5 +1,5 @@
-#ifndef AliITSQASSDDataMakerRec_H
-#define AliITSQASSDDataMakerRec_H
+#ifndef AliITSQASSDDATAMAKERREC_H
+#define AliITSQASSDDATAMAKERREC_H
 /* Copyright(c) 2007-2009, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */
 
index d4b905b..5e18aa9 100644 (file)
@@ -1,5 +1,5 @@
-#ifndef AliITSQASSDDataMakerSim_H
-#define AliITSQASSDDataMakerSim_H
+#ifndef AliITSQASSDDATAMAKERSIM_H
+#define AliITSQASSDDATAMAKERSIM_H
 /* Copyright(c) 2007-2009, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */