Fix coding rule violations
authorcblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 21 Oct 2009 12:54:33 +0000 (12:54 +0000)
committercblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 21 Oct 2009 12:54:33 +0000 (12:54 +0000)
TRD/AliTRDqaBlackEvents.cxx
TRD/AliTRDqaBlackEvents.h

index d8fc4cf..31e7b6c 100644 (file)
@@ -27,7 +27,7 @@
 #include "TH1D.h"
 #include "TH2D.h"
 #include "TH2S.h"
-#include "TH3F.h"
+//#include "TH3F.h"
 #include "TF1.h"
 #include "TFile.h"
 #include "TCanvas.h"
 #include "TStyle.h"
 #include "TGraph.h"
 
-#include "AliLog.h"
+#include "AliRawReader.h"
 
-#include "AliTRDgeometry.h"
 #include "AliTRDrawStream.h"
 #include "AliTRDqaBlackEvents.h"
 
-#include "AliRawReader.h"
-
-
 ClassImp(AliTRDqaBlackEvents)
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
@@ -294,7 +290,10 @@ void AliTRDqaBlackEvents::SetRefFile(const char *filename) {
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 void AliTRDqaBlackEvents::ReadRefHists(Int_t det) {
-  
+  //  
+  // Read the reference histograms
+  //
+
   fRefHistPed = 0;
   fRefHistNoise = 0;
   
@@ -345,7 +344,7 @@ void AliTRDqaBlackEvents::StartEvent()
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
-void AliTRDqaBlackEvents::AddBuffer(AliTRDrawStream *data, AliRawReader *reader) 
+void AliTRDqaBlackEvents::AddBuffer(AliTRDrawStream *data, AliRawReader * const reader) 
 {
   
   //printf ("try to read data\n");
@@ -608,6 +607,9 @@ void AliTRDqaBlackEvents::AddBuffer(AliTRDrawStream *data, AliRawReader *reader)
 
 void AliTRDqaBlackEvents::FinishEvent()
 {
+  //
+  // Processing at the end of the current event
+  //
 
   for(Int_t i=0; i<3; i++) {
     fGraphPP[i]->SetPoint(fnEvents, fnEvents, fnPP[i]);
@@ -931,8 +933,11 @@ void AliTRDqaBlackEvents::Process(const char *filename)
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
-Int_t AliTRDqaBlackEvents::CheckMCM(Int_t index) {
-  
+Int_t AliTRDqaBlackEvents::CheckMCM(Int_t index) const {
+  //
+  // Checks a single MCM
+  //  
+
   return 1;
   
   static Int_t data[21][3] = {
@@ -1079,6 +1084,9 @@ void AliTRDqaBlackEvents::DrawSm(const char *filename, Int_t sm, Int_t w, Int_t
 ///////////////////////////////////////////////////////////////////////////////////////////////////
 
 Int_t AliTRDqaBlackEvents::FillBits(TH1D *hist, Int_t code, Int_t offset) {
+  //
+  // Fill bits
+  //
 
   Int_t nb = 0;
   UInt_t test = 1;
index 64ed262..088d968 100644 (file)
@@ -39,16 +39,16 @@ class AliTRDqaBlackEvents : public TObject {
   //Int_t AddEvent(AliTRDrawStream *data, AliRawReader *reader);
 
   void StartEvent();
-  void AddBuffer(AliTRDrawStream *data, AliRawReader *reader);
+  void AddBuffer(AliTRDrawStream *data, AliRawReader * const reader);
   void FinishEvent();
 
   void Process(const char* filename);
   
   //TH2D *GetChamberPedestal(Int_t sm, Int_t layer, Int_t stack) {return 0;}
-  TH2D *GetChamberPedestal(Int_t det) {return fChPed[det];}
+  TH2D *GetChamberPedestal(Int_t det) const {return fChPed[det];}
   
   //TH2D *GetChamberNoise(Int_t sm, Int_t layer, Int_t stack) {return 0;}
-  TH2D *GetChamberNoise(Int_t det) {return fChNoise[det];}
+  TH2D *GetChamberNoise(Int_t det) const {return fChNoise[det];}
   
   void SetNoiseLevel(Double_t min, Double_t max) {fMinNoise = min; fMaxNoise = max;}
   void SetFitMethod(Int_t fit) {fFitType = fit;} 
@@ -110,8 +110,8 @@ class AliTRDqaBlackEvents : public TObject {
   TH1D *fnEntriesRMDist[kDET]; // distribtion of number of entries per ROB-MCM
 
   // direct access to data
-  Float_t  fDataDirect[kDET][kROW][kPAD][kCH];
-  Double_t fSignalDirect[kDET][kCH]; 
+  Float_t  fDataDirect[kDET][kROW][kPAD][kCH];  // data array
+  Double_t fSignalDirect[kDET][kCH]; // signal array
 
   // after reference subtraction
   TH2D *fChPedRes[kDET];    // histograms after reference subtraction
@@ -139,26 +139,26 @@ class AliTRDqaBlackEvents : public TObject {
   TH2D *fErrorLocADC[kDET];      // errors in ADC
 
   // error fraction
-  TGraph *fErrorGraphHC;
-  TGraph *fErrorGraphMCM;
-  TGraph *fErrorGraphADC;
+  TGraph *fErrorGraphHC;     // HC errors
+  TGraph *fErrorGraphMCM;    // MCM errors
+  TGraph *fErrorGraphADC;    // ADC errors
 
   TGraph *fGraphMCM;         // number of strange MCMs detected 
-  TGraph *fGraphPP[3];
+  TGraph *fGraphPP[3];       // number of strange points
   
 
-  // mcm trackles
-  TObjArray *fMcmTracks;
+  // mcm tracklets
+  TObjArray *fMcmTracks;     // MCM tracklets
 
   // problematic MCMs
-  TH2D *fMapMCM;
-  TH1D *fFracMCM;
+  TH2D *fMapMCM;             // map of problematic MCMs
+  TH1D *fFracMCM;            // fraction of problematicc MCMs
   
   // full detector view
-  TH2D *fSMHCped;
-  TH2D *fSMHCerr;
-  TH2D *fSMLink[3];
-  TGraph *fGrLink[3];
+  TH2D *fSMHCped;            // full detector view of pedestals
+  TH2D *fSMHCerr;            // full detector view of errors
+  TH2D *fSMLink[3];          // full detector view
+  TGraph *fGrLink[3];        // full detector view
   
   //TH1D *fZSsize;
   
@@ -168,27 +168,27 @@ class AliTRDqaBlackEvents : public TObject {
   
   //Int_t fChkDe
   
-  TH1D *fNoiseTotal;
-  TH1D *fPP;
+  TH1D *fNoiseTotal;         // total noise
+  TH1D *fPP;                 // points
   
-  TH1D *fSmNoiseRms[kSM];
-  TH1D *fSmNoiseFit[kSM];
-  TH1D *fSmPP[kSM];    
+  TH1D *fSmNoiseRms[kSM];    // noise RMS
+  TH1D *fSmNoiseFit[kSM];    // noise fit
+  TH1D *fSmPP[kSM];          // points
 
 
-  TH1D *fEvNoDist[1000];
+  TH1D *fEvNoDist[1000];     // event numbers
 
   //
   Double_t fMinNoise;   // Minimum noise
   Double_t fMaxNoise;   // Maximum noise
-  Int_t fFitType;
+  Int_t fFitType;       // fit type
   
   // variables keeping info in one event
   Int_t fnErrorHC[2];  // 0 good, 1 error
-  Int_t fnErrorMCM[2]; //
-  Int_t fnErrorADC[2]; // 
-    
-  Int_t fppThresh[3];      // thersholds for storing pp
+  Int_t fnErrorMCM[2]; // 0 good, 1 error
+  Int_t fnErrorADC[2]; // 0 good, 1 error
+   
+  Int_t fppThresh[3];      // thresholds for storing pp
   Int_t fnPP[3];           // number of entries above the thershold
   Int_t fnLink[3];         // links present, beaf-beaf, good
   Int_t fnADCinSM[kSM+1];  // number of ADC channels in a SuperModule
@@ -198,7 +198,7 @@ class AliTRDqaBlackEvents : public TObject {
 
   // private function
   void  ReadRefHists(Int_t det);
-  Int_t CheckMCM(Int_t index);
+  Int_t CheckMCM(Int_t index) const;
   
   Int_t FillBits(TH1D *hist, Int_t code, Int_t offset);