addressing some minor rulechecker coding violations
authordsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 10 Apr 2011 14:56:26 +0000 (14:56 +0000)
committerdsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Sun, 10 Apr 2011 14:56:26 +0000 (14:56 +0000)
PWG4/totEt/AliAnalysisEt.cxx
PWG4/totEt/AliAnalysisEtCuts.cxx
PWG4/totEt/AliAnalysisEtCuts.h
PWG4/totEt/AliAnalysisEtMonteCarlo.cxx
PWG4/totEt/AliAnalysisEtSelectionContainer.h
PWG4/totEt/AliAnalysisTaskTotEt.h

index 03d283f..81a2500 100644 (file)
@@ -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);
index 853b851..4ce2a71 100644 (file)
@@ -81,9 +81,6 @@ AliAnalysisEtCuts::AliAnalysisEtCuts() :
   ,fHistNbinsParticlePt(200) 
   ,fHistMinParticlePt(0)
   ,fHistMaxParticlePt(20)
-  
-  ,fDetectorPhos(-1)
-  ,fDetectorEmcal(1)
 { // ctor
 }
 
index 32efac8..bb00ebf 100644 (file)
@@ -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
index 1fb6865..7e4d0b9 100644 (file)
@@ -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;
 
index cb7e3ec..086889f 100644 (file)
@@ -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; }
index 2e8c4e7..456c0d4 100644 (file)
@@ -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