addressing some minor rulechecker coding violations

diff --git a/PWG4/totEt/AliAnalysisEt.cxx b/PWG4/totEt/AliAnalysisEt.cxx
index 03d283fd1bc4145041491a30a5d1f7cf48efb562..81a2500e2a88df937de807100a7e39171ff80091 100644 (file)
--- a/PWG4/totEt/AliAnalysisEt.cxx
+++ b/PWG4/totEt/AliAnalysisEt.cxx
@@ -570,7 +570,7 @@ void AliAnalysisEt::ResetEventValues()
 }
 
 Double_t AliAnalysisEt::CalculateTransverseEnergy(AliESDCaloCluster* cluster)
-{
+{ // based on cluster energy and cluster pos
   
   Float_t pos[3];
   cluster->GetPosition(pos);

diff --git a/PWG4/totEt/AliAnalysisEtCuts.cxx b/PWG4/totEt/AliAnalysisEtCuts.cxx
index 853b851188de4ec3c1572ec92fe962b50c970d35..4ce2a7192973c6a8cc31b1c0d39025a0ae5472b8 100644 (file)
--- a/PWG4/totEt/AliAnalysisEtCuts.cxx
+++ b/PWG4/totEt/AliAnalysisEtCuts.cxx
@@ -81,9 +81,6 @@ AliAnalysisEtCuts::AliAnalysisEtCuts() :
   ,fHistNbinsParticlePt(200) 
   ,fHistMinParticlePt(0)
   ,fHistMaxParticlePt(20)
-  
-  ,fDetectorPhos(-1)
-  ,fDetectorEmcal(1)
 { // ctor
 }
 

diff --git a/PWG4/totEt/AliAnalysisEtCuts.h b/PWG4/totEt/AliAnalysisEtCuts.h
index 32efac82ffe32c7d0007a4c94e25f328574e043f..bb00ebf605b0dfdc7c5c64342fd804d8eb7f920f 100644 (file)
--- a/PWG4/totEt/AliAnalysisEtCuts.h
+++ b/PWG4/totEt/AliAnalysisEtCuts.h
@@ -83,8 +83,8 @@ class AliAnalysisEtCuts : public TNamed
   Double_t GetHistMinParticlePt() const { return fHistMinParticlePt; }
   Double_t GetHistMaxParticlePt() const { return fHistMaxParticlePt; }
   
-  Short_t GetDetectorPhos() const { return fDetectorPhos; }
-  Short_t GetDetectorEmcal() const { return fDetectorEmcal; }
+  Short_t GetDetectorPhos() const { return fgkDetectorPhos; }
+  Short_t GetDetectorEmcal() const { return fgkDetectorEmcal; }
 
   // Setters
   // Common
@@ -226,8 +226,8 @@ class AliAnalysisEtCuts : public TNamed
   Double_t fHistMaxParticlePt; // maximum value in particle Pt histograms
 
 // Detector definition
-  const Short_t fDetectorPhos; // PHOS 
-  const Short_t fDetectorEmcal; // PHOS 
+  static const Short_t fgkDetectorPhos = -1; // PHOS 
+  static const Short_t fgkDetectorEmcal = 1; // EMCAL 
 
 private:
   //Declare private to avoid compilation warning

diff --git a/PWG4/totEt/AliAnalysisEtMonteCarlo.cxx b/PWG4/totEt/AliAnalysisEtMonteCarlo.cxx
index 1fb6865a822cfdc51b10355d10989c3ca07c8742..7e4d0b90ce693d942e66c38610cfceb0fd18178c 100644 (file)
--- a/PWG4/totEt/AliAnalysisEtMonteCarlo.cxx
+++ b/PWG4/totEt/AliAnalysisEtMonteCarlo.cxx
@@ -1370,7 +1370,7 @@ bool AliAnalysisEtMonteCarlo::TrackHitsCalorimeter(TParticle* part, Double_t mag
 }
 
 void AliAnalysisEtMonteCarlo::FillHistograms()
-{
+{ // let base class fill its histograms, and us fill the local ones
     AliAnalysisEt::FillHistograms();
     //std::cout << fEtNonRemovedPiPlus << " " << fCentClass << std::endl;
 

diff --git a/PWG4/totEt/AliAnalysisEtSelectionContainer.h b/PWG4/totEt/AliAnalysisEtSelectionContainer.h
index cb7e3ec104b97f365d315f826a4f5cdf6eee5ff4..086889faf48b33aa8dd2662de67d98e7db5df594 100644 (file)
--- a/PWG4/totEt/AliAnalysisEtSelectionContainer.h
+++ b/PWG4/totEt/AliAnalysisEtSelectionContainer.h
@@ -29,10 +29,10 @@ public:
   virtual ~AliAnalysisEtSelectionContainer();
   
   /** Return the physics selection for the current run */
-  AliPhysicsSelection* GetPhysicsSelection(Int_t runNumber){ return fPhysicsSelectionMap[runNumber]; }
+  AliPhysicsSelection* GetPhysicsSelection(Int_t runNumber) { return fPhysicsSelectionMap[runNumber]; }
   
   /** Return the physics selection for the current run */
-  AliPhysicsSelection* GetDefaultPhysicsSelection(){ return fPhysicsSelectionMap[0]; }
+  AliPhysicsSelection* GetDefaultPhysicsSelection() { return fPhysicsSelectionMap[0]; }
   
   /** Get the map */
   std::map<int, AliPhysicsSelection*> GetPhysicsSelectionMap() const { return fPhysicsSelectionMap; }

diff --git a/PWG4/totEt/AliAnalysisTaskTotEt.h b/PWG4/totEt/AliAnalysisTaskTotEt.h
index 2e8c4e7fde453b0fce50bb6a06e5fb6bda66eacd..456c0d436d13fd86654b454b835e5f6e7626fab6 100644 (file)
--- a/PWG4/totEt/AliAnalysisTaskTotEt.h
+++ b/PWG4/totEt/AliAnalysisTaskTotEt.h
@@ -36,8 +36,8 @@ private:
   AliAnalysisEtReconstructed *fRecAnalysis; // Rec \r
   AliAnalysisEtMonteCarlo *fMCAnalysis; // MC\r
   \r
-  THnSparseD *fSparseHistRecVsMc;\r
-  Double_t *fSparseRecVsMc;\r
+  THnSparseD *fSparseHistRecVsMc; // Hist Rec vs Mc\r
+  Double_t *fSparseRecVsMc; // Rec vs Mc\r
   \r
   ClassDef(AliAnalysisTaskTotEt, 2) \r
 };\r