Coding violation (Marian)

diff --git a/PWG1/AliGenInfoMaker.cxx b/PWG1/AliGenInfoMaker.cxx
index 24310d20b9fd7e9c535bf54d3646b99da24b3516..3f43c91b0d73256967ea2f9e951244c4e5966155 100644 (file)
--- a/PWG1/AliGenInfoMaker.cxx
+++ b/PWG1/AliGenInfoMaker.cxx
@@ -36,15 +36,15 @@ t->Exec();
 #include "Rtypes.h"
 #include "TFile.h"
 #include "TTree.h"
-#include "TChain.h"
-#include "TCut.h"
-#include "TString.h"
+//#include "TChain.h"
+//#include "TCut.h"
+//#include "TString.h"
 #include "TStopwatch.h"
 #include "TParticle.h"
-#include "TSystem.h"
-#include "TCanvas.h"
-#include "TGeometry.h"
-#include "TPolyLine3D.h"
+//#include "TSystem.h"
+//#include "TCanvas.h"
+//#include "TGeometry.h"
+//#include "TPolyLine3D.h"
 
 //ALIROOT includes
 #include "AliRun.h"
@@ -53,12 +53,12 @@ t->Exec();
 #include "AliTPCParam.h"
 #include "AliTPC.h"
 #include "AliTPCLoader.h"
-#include "AliDetector.h"
+//#include "AliDetector.h"
 #include "AliTrackReference.h"
 #include "AliTPCParamSR.h"
 #include "AliTracker.h"
-#include "AliMagF.h"
-#include "AliHelix.h"
+//#include "AliMagF.h"
+//#include "AliHelix.h"
 #include "AliTrackPointArray.h"
 
 #endif
@@ -172,6 +172,8 @@ AliGenInfoMaker::AliGenInfoMaker(const char * fnGalice, const char* fnRes,
 AliMCInfo * AliGenInfoMaker::MakeInfo(UInt_t i)
 {
   // 
+  // Make info structure for given particle index
+  //
   if (i<fNParticles) {
     if (fGenInfo[i]) return  fGenInfo[i];
     fGenInfo[i] = new AliMCInfo;  
@@ -185,6 +187,9 @@ AliMCInfo * AliGenInfoMaker::MakeInfo(UInt_t i)
 ////////////////////////////////////////////////////////////////////////
 AliGenInfoMaker::~AliGenInfoMaker()
 {
+  //
+  // Destructor
+  //
   
   if (fLoader){
     fLoader->UnloadgAlice();
@@ -196,7 +201,8 @@ AliGenInfoMaker::~AliGenInfoMaker()
 Int_t  AliGenInfoMaker::SetIO()
 {
   //
-  // 
+  // Set IO for given event 
+  //
   CreateTreeGenTracks();
   if (!fTreeGenTracks) return 1;
   //  AliTracker::SetFieldFactor(); 
@@ -230,6 +236,9 @@ Int_t AliGenInfoMaker::SetIO(Int_t eventNr)
 
 Int_t AliGenInfoMaker::CloseIOEvent()
 {
+  //
+  // Close IO for current event
+  //
   fLoader->UnloadHeader();
   fLoader->UnloadKinematics();
   fLoader->UnloadTrackRefs();
@@ -249,6 +258,11 @@ Int_t AliGenInfoMaker::CloseIO()
 ////////////////////////////////////////////////////////////////////////
 Int_t AliGenInfoMaker::Exec(Int_t nEvents, Int_t firstEventNr)
 {
+  //
+  // Execute action for 
+  // nEvents
+  // firstEventNr - first event number
+  //
   fNEvents = nEvents;
   fFirstEventNr = firstEventNr;
   return Exec();
@@ -257,6 +271,10 @@ Int_t AliGenInfoMaker::Exec(Int_t nEvents, Int_t firstEventNr)
 ////////////////////////////////////////////////////////////////////////
 Int_t AliGenInfoMaker::Exec()  
 {
+  //
+  // Make a comparision MC tree
+  // Connect MC information -TPArticle - AliTrackRefernces ...
+  //
   TStopwatch timer;
   timer.Start();
   Int_t status =SetIO();
@@ -310,6 +328,9 @@ Int_t AliGenInfoMaker::Exec()
 ////////////////////////////////////////////////////////////////////////
 void AliGenInfoMaker::CreateTreeGenTracks() 
 {
+  //
+  //
+  //
   fFileGenTracks = TFile::Open(fFnRes,"RECREATE");
   if (!fFileGenTracks) {
     cerr<<"Error in CreateTreeGenTracks: cannot open file "<<fFnRes<<endl;
@@ -349,6 +370,9 @@ void AliGenInfoMaker::CreateTreeGenTracks()
 ////////////////////////////////////////////////////////////////////////
 void AliGenInfoMaker::CloseOutputFile() 
 {
+  //
+  // Close Output files
+  //
   if (!fFileGenTracks) {
     cerr<<"File "<<fFnRes<<" not found as an open file."<<endl;
     return;
@@ -952,6 +976,9 @@ Int_t AliGenInfoMaker::TreeTRLoopNew()
 Float_t AliGenInfoMaker::TR2LocalX(AliTrackReference *trackRef,
                                    AliTPCParam *paramTPC) const {
 
+  //
+  // Transformation from Gloabal to local tacking system
+  //
   Float_t x[3] = { trackRef->X(),trackRef->Y(),trackRef->Z()};
   Int_t index[4];
   paramTPC->Transform0to1(x,index);
@@ -963,6 +990,9 @@ Float_t AliGenInfoMaker::TR2LocalX(AliTrackReference *trackRef,
 
 
 AliTPCParam * AliGenInfoMaker::GetTPCParam(){
+  //
+  // create default TPC parameters
+  //
   AliTPCParamSR * par = new AliTPCParamSR;
   par->Update();
   return par;

diff --git a/PWG1/AliRecInfoMaker.cxx b/PWG1/AliRecInfoMaker.cxx
index df273c11514d392a7317863ca8ec3dcba3c1a0f8..4092664dec26db2e8637df51fc63cf0c9bbf6043 100644 (file)
--- a/PWG1/AliRecInfoMaker.cxx
+++ b/PWG1/AliRecInfoMaker.cxx
@@ -141,7 +141,7 @@ TProfile prof("prof","prof",10,0.5,5);
 #include "TTree.h"
 #include "TStopwatch.h"
 #include "TVector3.h"
-#include "Getline.h"
+//#include "Getline.h"
 //
 //ALIROOT includes
 //
@@ -281,6 +281,9 @@ AliRecInfoMaker::AliRecInfoMaker(const char* fnGenTracks,
 ////////////////////////////////////////////////////////////////////////
 AliRecInfoMaker::~AliRecInfoMaker()
 {
+  //
+  // Destructor
+  //
   if (fLoader) {
     delete fLoader;
   }
@@ -290,7 +293,8 @@ AliRecInfoMaker::~AliRecInfoMaker()
 Int_t AliRecInfoMaker::SetIO()
 {
   //
-  // 
+  // SetIO  - Create the input trees
+  //
   CreateTreeCmp();
   if (!fTreeCmp) return 1;
   fParamTPC = GetTPCParam();
@@ -588,12 +592,14 @@ Int_t AliRecInfoMaker::TreeTLoop()
   //
   // load kinks to the memory
   for (Int_t i=0; i<nKinks;i++){
-    AliESDkink * kink =fEvent->GetKink(i);
+    //    AliESDkink * kink =
+    fEvent->GetKink(i);
     fSignedKinks[i]=0;
   }
   //
   for (Int_t i=0; i<nV0MIs;i++){
-    AliV0 * v0MI = (AliV0*)fEvent->GetV0(i);
+    //AliV0 * v0MI = 
+    (AliV0*)fEvent->GetV0(i);
     fSignedV0[i]=0;
   }
   

diff --git a/PWG1/AliRecInfoMaker.h b/PWG1/AliRecInfoMaker.h
index 1ab4b4944a30d1768d86d0ca578c252c6e3d2a4b..395cff843883d6fa8d28513e9203c8be2aeab0e0 100644 (file)
--- a/PWG1/AliRecInfoMaker.h
+++ b/PWG1/AliRecInfoMaker.h
@@ -75,8 +75,8 @@ private:
   TTree *fTreeCmpV0;                //! output tree with cmp V0
   //
   char  fFnGenTracks[1000];             //! input file name with gen tracks
-  TFile *fFileGenTracks;
-  TTree *fTreeGenTracks;
+  TFile *fFileGenTracks;                //! input files with generated tracks   
+  TTree *fTreeGenTracks;           //! tree with generated tracks
   TTree *fTreeGenKinks;            // tree with gen kinks
   TTree *fTreeGenV0;            // tree with gen V0
   //