Coding violations
authoralibrary <alibrary@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 29 Apr 2004 20:50:31 +0000 (20:50 +0000)
committeralibrary <alibrary@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 29 Apr 2004 20:50:31 +0000 (20:50 +0000)
31 files changed:
STEER/AliCluster.h
STEER/AliDebugVolume.h
STEER/AliDigit.h
STEER/AliESDCaloTrack.h
STEER/AliESDPmdTrack.cxx
STEER/AliESDPmdTrack.h
STEER/AliFieldMap.h
STEER/AliGausCorr.h
STEER/AliGenEventHeader.h
STEER/AliHeader.h
STEER/AliHit.h
STEER/AliLego.cxx
STEER/AliMCQA.cxx
STEER/AliMagF.h
STEER/AliMagFC.h
STEER/AliMagFCM.cxx
STEER/AliMagFCM.h
STEER/AliMagFDM.cxx
STEER/AliMagFDM.h
STEER/AliMagFMaps.cxx
STEER/AliMagFMaps.h
STEER/AliMemoryWatcher.h
STEER/AliMergeCombi.h
STEER/AliPDG.h
STEER/AliPoints.cxx
STEER/AliPoints.h
STEER/AliReconstructor.h
STEER/AliTrackMapper.cxx
STEER/AliTrackReference.cxx
STEER/AliTrackReference.h
STEER/AliVertexer.cxx

index 54cb37c..4320266 100644 (file)
@@ -7,7 +7,7 @@
 
 //-------------------------------------------------------------------------
 //                          Class AliCluster
-//
+//                Generic class for clusters
 //       Origin: Iouri Belikov, CERN, Jouri.Belikov@cern.ch 
 //-------------------------------------------------------------------------
 
index 22f0c90..f3f82e0 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//-----------------------------------------------------------------------
+//    Class to debug entry and exit from a volume
+//    Used by AliLego class
+//    Author: A.Morsch
+//-----------------------------------------------------------------------
+
 #include "TNamed.h"
 class AliDebugVolume : public TNamed  {
 
index 9c33568..5839821 100644 (file)
@@ -17,7 +17,7 @@ public:
   AliDigit();
   AliDigit(Int_t *track);
   virtual ~AliDigit() {}
-  virtual Int_t *GetTracks() {return &fTracks[0];}
+  virtual Int_t *GetTracks() {return fTracks;}
   virtual Int_t GetTrack(Int_t i) const 
   {assert(0<=i&&i<=2); return fTracks[i];}
   
index 32a0349..dfa0375 100644 (file)
@@ -19,17 +19,21 @@ class AliESDCaloTrack : public TObject {
 
 public:
   AliESDCaloTrack(): TObject(), fRecParticle(0) {}
+  AliESDCaloTrack(const AliESDCaloTrack &act): TObject(act)
+    {fRecParticle = act.fRecParticle;}
   virtual ~AliESDCaloTrack() {
     //PH    delete fRecParticle;
   }
   AliESDCaloTrack(TParticle* recpart);
-  Float_t Px() { return fRecParticle->Px(); }
-  Float_t Py() { return fRecParticle->Py(); }
-  Float_t Pz() { return fRecParticle->Pz(); }
+  Float_t Px() const { return fRecParticle->Px(); }
+  Float_t Py() const { return fRecParticle->Py(); }
+  Float_t Pz() const { return fRecParticle->Pz(); }
 
-  TParticle * GetRecParticle() {return fRecParticle;}
+  TParticle * GetRecParticle() const {return fRecParticle;}
 
 private:
+  AliESDCaloTrack & operator=(const AliESDCaloTrack &)
+    {Fatal("= operator","Not implemented\n"); return *this;}
   TParticle *fRecParticle; // reconstructed particle from PHOS or EMCAL
 
   ClassDef(AliESDCaloTrack,2)  //ESD calorimeter track class 
index e93a8e5..45dadc8 100644 (file)
@@ -1,5 +1,28 @@
+/**************************************************************************
+ * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
+ *                                                                        *
+ * Author: The ALICE Off-line Project.                                    *
+ * Contributors are mentioned in the code where appropriate.              *
+ *                                                                        *
+ * Permission to use, copy, modify and distribute this software and its   *
+ * documentation strictly for non-commercial purposes is hereby granted   *
+ * without fee, provided that the above copyright notice appears in all   *
+ * copies and that both the copyright notice and this permission notice   *
+ * appear in the supporting documentation. The authors make no claims     *
+ * about the suitability of this software for any purpose. It is          *
+ * provided "as is" without express or implied warranty.                  *
+ **************************************************************************/
+
+/* $Id$ */
+
 #include "AliESDPmdTrack.h"
 
+// Event Data Summary Class
+// For pmd tracks
+// This is part of the reconstructed
+// ESD events
+// for the PMD detector
+
 ClassImp(AliESDPmdTrack)
 
 AliESDPmdTrack::AliESDPmdTrack (const AliESDPmdTrack& PMDTrack):TObject(PMDTrack)
index 4982b86..e2b7b84 100644 (file)
@@ -1,6 +1,15 @@
 #ifndef ALIESDPMDTRACK_H
 #define ALIESDPMDTRACK_H
 
+/* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
+ * See cxx source for full Copyright notice                               */
+
+/* $Id$ */
+
+// Event Data Summary Class for pmd tracks
+// This is part of the reconstructed ESD events
+// for the PMD detector
+
 #include "TObject.h"
 
 class AliESDPmdTrack : public TObject {
index 76a913d..d42ab3e 100644 (file)
@@ -9,7 +9,7 @@
 // Author: Andreas Morsch <andreas.morsch@cern.ch>
 //
 #include <TNamed.h>
-class TVector;
+#include <TVector.h>
 
 class AliFieldMap : public TNamed
 {
index 31d9cca..4c25128 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+////////////////////////////////////////////////////////////////////////
+// Class used to generate correlated gaussian numbers with mean
+// zero and known covariance matrix.
+// M.Masera 15.03.2001 9:30 - modified on 26.02.2002 17:40
+////////////////////////////////////////////////////////////////////////
+
 #include <TMatrixD.h>
 class TArrayD;
 
index 33345d1..c40ef93 100644 (file)
@@ -6,6 +6,12 @@
 
 /* $Id$ */
 
+//---------------------------------------------------------------------
+// Event header base class for generator. 
+// Stores generated event information
+// Author: andreas.morsch@cern.ch
+//---------------------------------------------------------------------
+
 #include <TNamed.h>
 #include <TArrayF.h>
 
index 114838e..68d2447 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//-----------------------------------------------------------------------
+//     Simulation event header class
+//     Collaborates with AliRun, AliStack, and AliGenReaderTreeK classes
+//     Many other classes depend on it
+//-----------------------------------------------------------------------
+
 #include <TObject.h>
 
 class AliStack;
index 3985ab4..b0e1886 100644 (file)
@@ -5,6 +5,10 @@
 
 /* $Id$ */
 
+// Base class for hits
+// This class is used as a base class for 
+// hits in the different detectors
+
 #include "TObject.h"
 
 class AliHit : public TObject {
index 4a4bff9..81e9ccd 100644 (file)
 #include "TString.h"
 #include "TVirtualMC.h"
 
-#include "AliConst.h"
 #include "AliDebugVolume.h"
 #include "AliLego.h"
 #include "AliLegoGenerator.h"
-#include "AliRun.h"
 #include "AliMC.h"
+#include "AliRun.h"
 
 ClassImp(AliLego)
 
index 735e43d..a3e1b80 100644 (file)
 #include <TLorentzVector.h>
 #include <TMath.h>
 #include <TObjArray.h>
-#include <TPad.h>
 #include <TPaveLabel.h>
 #include <TROOT.h>
 #include <TVirtualMC.h>
 
+#include "AliMC.h"
 #include "AliMCQA.h"
 #include "AliModule.h"
 #include "AliRun.h"
-#include "AliMC.h"
 
 ClassImp(AliMCQA)
 
index 7b54cdf..6bf3de6 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//----------------------------------------------------------------------
+// Basic magnetic field class
+// Used in all the detectors, and also in the traking classes
+// Author:
+//----------------------------------------------------------------------
+
 #include "TNamed.h"
 
 enum Field_t {kUndef=1, kConst=1, kConMesh=2, kDipoMap=3};
index f8418cb..3d50bce 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//-------------------------------------------------------------------------
+//     Constant magnetic field class
+//     Used by AliRun class
+//     Author:
+//-------------------------------------------------------------------------
+
 #include "AliMagF.h"
 
 class AliMagFC  : public AliMagF
index c5742e0..4b0a1fd 100644 (file)
@@ -20,6 +20,7 @@
 //  Used in the configuration macros (macros/Config.C, etc.)
 //  Author:
 //-----------------------------------------------------------------------
+
 #include "TSystem.h"
 
 #include "TVector.h"
index e2bdc19..18ea959 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//-----------------------------------------------------------------------
+//  Class for Alice magnetic field with constant mesh
+//  Used in the configuration macros (macros/Config.C, etc.)
+//  Author:
+//-----------------------------------------------------------------------
+
 #include "AliMagFC.h"
 class TVector;
 
index 5604727..7e9bfa2 100644 (file)
@@ -21,8 +21,6 @@
 //   Author:
 //-------------------------------------------------------------------------
 
-#include <stdlib.h>
-
 #include "TSystem.h"
 
 #include "AliMagFDM.h"
@@ -375,10 +373,10 @@ Double_t AliMagFDM::Ba(Int_t kaai,Double_t zaa1, Double_t zaa2,
   //
   // Calculation of field componet for case (keps <r0<= fRdel) at a given axis
   //
-  Double_t fa11,fa12,fa13;
-  Double_t fa21,fa22,fa23;
-  Double_t faY1,faY2;
-  Double_t bba;
+  Double_t fa11=0,fa12=0,fa13=0;
+  Double_t fa21=0,fa22=0,fa23=0;
+  Double_t faY1=0,faY2=0;
+  Double_t bba=0;
 
   switch (kaai) {
   case 0:
@@ -407,7 +405,6 @@ Double_t AliMagFDM::Ba(Int_t kaai,Double_t zaa1, Double_t zaa2,
     break;
   default:
     Fatal("Ba","Invalid value of kaai %d\n",kaai);
-    exit(1);
   }                            
   faY1=alf1*fa11+alf2*fa12+alf3*fa13;
   faY2=alf1*fa21+alf2*fa22+alf3*fa23;
@@ -423,11 +420,11 @@ Double_t AliMagFDM::Bb(Double_t z1,Double_t z2, Double_t y1,Double_t y2,
   //
   // Calculation of field componet at a given axis (general case)
   //
-  Double_t fy1, fy2, ffy;
-  Double_t gy1,gy2,ggy;
-  Double_t bbi;
-  Double_t bf11,bf12,bf21,bf22;
-  Double_t bg11,bg12,bg21,bg22;
+  Double_t fy1=0,fy2=0,ffy=0;
+  Double_t gy1=0,gy2=0,ggy=0;
+  Double_t bbi=0;
+  Double_t bf11=0,bf12=0,bf21=0,bf22=0;
+  Double_t bg11=0,bg12=0,bg21=0,bg22=0;
 
   
   /*-----------------Polar part ------------------*/
@@ -508,7 +505,6 @@ Double_t AliMagFDM::Bb(Double_t z1,Double_t z2, Double_t y1,Double_t y2,
 
   default:
     Fatal("Bb","Invalid value of kv %d\n",kv);
-    exit(1);
   }  
   
   
@@ -687,7 +683,6 @@ printf("fZpdl %e, fPhid %e, fRdel %e, fZpmx %e, fZpmn %e,fRmax %e,fRmin %e \n",
     }
 //
   } else { 
-    printf("File %s not found !\n",fTitle.Data());
-    exit(1);
+    Fatal("ReadField","File %s not found !\n",fTitle.Data());
   }
 }
index 170a0e5..4e4ed93 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//-------------------------------------------------------------------------
+//   Field with Magnetic Field map
+//   Used by AliRun class
+//   Author:
+//-------------------------------------------------------------------------
+
 #include "AliMagFC.h"
 //
 class AliMagFDM : public AliMagFC
index aa43984..59a933e 100644 (file)
@@ -24,7 +24,6 @@
 
 #include <TFile.h>
 #include <TSystem.h>
-#include <TVector.h>
 
 #include "AliFieldMap.h"
 #include "AliMagFMaps.h"
index 1b36b12..5546c0c 100644 (file)
@@ -6,6 +6,8 @@
 /* $Id$ */
 
 //
+// Magnetic field composed by 3 maps: the L3 magnet, extended region, and
+// dipole magnet
 // Author: Andreas Morsch <andreas.morsch@cern.ch>
 //
 
index 090ef0d..6b529f7 100644 (file)
@@ -2,40 +2,44 @@
 #define ALIMEMORYWATCHER_H
 /* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice     */
+
 /* $Id$ */
+
 //_________________________________________________________________________
-/*Basic Memory Leak utility.
-    
-    You can use this tiny class to *see* if your program is leaking.
-    Usage:
-    AliMemoryWatcher memwatcher;
-    some program loop on events here {
-      if ( nevents % x == 0 ) 
-      {
-      // take a sample every x events
-        memwatcher.watch(nevents);
-      }
-    }
-    TFile f("out.root","RECREATE");
-    memwatcher.write();
-    f.Close();
-    In the output root file you'll get 3 graphs representing
-    the evolAliPHOSon, as a function of the number of events, of :
-    - VSIZE is the virtual size (in KBytes) of your program, that is sort of
-    the total memory used
-    - RSSIZE is the resident size (in KBytes), that is, the part of your 
-    program which is really in physical memory.
-    - TIME is an estimate of time per event (really it's the time elasped
-    between two calls to watch method)
-    WARNING: this is far from a bulletproof memory report (it's basically 
-    using UNIX command ps -h -p [PID] -o vsize,rssize to do its job).
-    It has only been tested on Linux so far.
-    
-    But by fitting the VSIZE by a pol1 under ROOT, you'll see right away
-    by how much your program is leaking.
-*/             
+//Basic Memory Leak utility.
+//  
+//  You can use this tiny class to *see* if your program is leaking.
+//  Usage:
+//  AliMemoryWatcher memwatcher;
+//  some program loop on events here {
+//    if ( nevents % x == 0 ) 
+//    {
+//    // take a sample every x events
+//      memwatcher.watch(nevents);
+//    }
+//  }
+//  TFile f("out.root","RECREATE");
+//  memwatcher.write();
+//  f.Close();
+//  In the output root file you'll get 3 graphs representing
+//  the evolAliPHOSon, as a function of the number of events, of :
+//  - VSIZE is the virtual size (in KBytes) of your program, that is sort of
+//  the total memory used
+//  - RSSIZE is the resident size (in KBytes), that is, the part of your 
+//  program which is really in physical memory.
+//  - TIME is an estimate of time per event (really it's the time elasped
+//  between two calls to watch method)
+//  WARNING: this is far from a bulletproof memory report (it's basically 
+//  using UNIX command ps -h -p [PID] -o vsize,rssize to do its job).
+//  It has only been tested on Linux so far.
+//  
+//  But by fitting the VSIZE by a pol1 under ROOT, you'll see right away
+//  by how much your program is leaking.
+//
 //*-- Author: Laurent Aphecetche(SUBATECH)
+
 // --- ROOT system ---
+
 #include "TObject.h" 
 class TH2;
 class TGraph;
index c9980a5..782d804 100644 (file)
@@ -8,7 +8,7 @@
 ////////////////////////////////////////////////////////////////////////
 //
 //  Class to make combination for merging
-//                  
+//  Returns combination of input event numbers                
 //  Author: Jiri Chudoba (CERN), 2001
 //
 ////////////////////////////////////////////////////////////////////////
index b4cbd84..b5f62d0 100644 (file)
@@ -7,6 +7,10 @@
 
 #include "TObject.h"
 
+// Class to encapsulate the ALICE updates to TDatabasePDG.h
+// Can be used by TGeant3 and TGeant4
+// Comments to: andreas.morsch@cern.ch 
+
 class AliPDG : public TObject {
 public:
     static void AddParticlesToPdgDataBase();
index 15d535d..8e7e9ff 100644 (file)
@@ -33,9 +33,9 @@
 #include "TView.h"
 
 #include "AliDetector.h"
+#include "AliMC.h"
 #include "AliPoints.h"
 #include "AliRun.h"
-#include "AliMC.h"
  
 ClassImp(AliPoints)
 
index 5dbd214..90a15d5 100644 (file)
@@ -5,6 +5,10 @@
 
 /* $Id$ */
 
+// This class contains the points
+// that are used for the ALICE
+// event display
+
 #include "TPolyMarker3D.h"
 class AliDetector;
 class TParticle;
index afcffdc..6419dd4 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+//
+// base class for reconstruction algorithm
+// Derived classes should implement a default constructor and
+// the virtual methods
+//
+
 #include <TObject.h>
 
 class AliRunLoader;
index cf2c2fb..f644ff0 100644 (file)
 
 #include <Riostream.h>
 
-#include "TTree.h"
+#include "TError.h"
 #include "TFile.h"
 #include "TStopwatch.h"
-#include "TError.h"
+#include "TTree.h"
 
 #include "AliDetector.h"
-#include "AliTrackMapper.h"
-#include "AliTrackMap.h"
+#include "AliHit.h"
+#include "AliLoader.h"
 #include "AliRun.h"
 #include "AliRunLoader.h"
-#include "AliLoader.h"
-#include "AliHit.h"
+#include "AliTrackMap.h"
+#include "AliTrackMapper.h"
 
 ClassImp(AliTrackMapper)
 
index 7c3fb26..5049601 100644 (file)
  * provided "as is" without express or implied warranty.                  *
  **************************************************************************/
 
+/* $Id$ */
 
-#include "AliTrackReference.h"
+#include "TLorentzVector.h"
 #include "TVirtualMC.h"
-#include "TParticle.h"
+
 #include "AliRun.h"
-#include "TLorentzVector.h"
+#include "AliTrackReference.h"
 
 // 
 // Track Reference object is created every time particle is 
index fc8117b..4ab5940 100644 (file)
@@ -5,6 +5,12 @@
 
 /* $Id$ */
 
+// 
+// Track Reference object is created every time particle is 
+// crossing detector bounds. 
+// The object is created by Step Manager
+//
+
 #include "TObject.h"
 #include "TMath.h"
 
index 1125ff7..c4c9d48 100644 (file)
@@ -1,4 +1,20 @@
-#include <AliESDVertex.h>
+/**************************************************************************
+ * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
+ *                                                                        *
+ * Author: The ALICE Off-line Project.                                    *
+ * Contributors are mentioned in the code where appropriate.              *
+ *                                                                        *
+ * Permission to use, copy, modify and distribute this software and its   *
+ * documentation strictly for non-commercial purposes is hereby granted   *
+ * without fee, provided that the above copyright notice appears in all   *
+ * copies and that both the copyright notice and this permission notice   *
+ * appear in the supporting documentation. The authors make no claims     *
+ * about the suitability of this software for any purpose. It is          *
+ * provided "as is" without express or implied warranty.                  *
+ **************************************************************************/
+
+/* $Id$ */
+
 #include <AliVertexer.h>
 
 ClassImp(AliVertexer)