Coding Violations Corrected.

diff --git a/HBTAN/AliHBTParticleCut.cxx b/HBTAN/AliHBTParticleCut.cxx
index 4657a3f61236ae3b82037d39693279eab0c225bd..204430625f0652ca66c5349dd0add2c097bd6f9c 100644 (file)
--- a/HBTAN/AliHBTParticleCut.cxx
+++ b/HBTAN/AliHBTParticleCut.cxx
@@ -1,3 +1,4 @@
+#include "AliHBTParticleCut.h"
 //__________________________________________________________________________
 ////////////////////////////////////////////////////////////////////////////
 //                                                                        //
@@ -19,22 +20,21 @@
 // resonsible: Piotr Skowronski@cern.ch                                   //
 //                                                                        //
 ////////////////////////////////////////////////////////////////////////////
-#include "AliHBTParticleCut.h"
 
 #include <Riostream.h>
 
+
 ClassImp(AliHBTParticleCut)
 const Int_t AliHBTParticleCut::fgkMaxCuts = 50;
 /******************************************************************/
 
-AliHBTParticleCut::AliHBTParticleCut()
- {
-   //default ctor
-   fCuts = new AliHbtBaseCut* [fgkMaxCuts];//last property in the property
-                                         //property enum => defines number of properties
-   fNCuts = 0;
-   fPID  = 0;
- }
+AliHBTParticleCut::AliHBTParticleCut():
+ fCuts(new AliHbtBaseCut* [fgkMaxCuts]),//last property in the property enum => defines number of properties
+ fNCuts(0),
+ fPID(0)
+{
+  //default ctor
+}
 /******************************************************************/
 
 AliHBTParticleCut::AliHBTParticleCut(const AliHBTParticleCut& in):
@@ -51,9 +51,10 @@ AliHBTParticleCut::AliHBTParticleCut(const AliHBTParticleCut& in):
    }
 }
 /******************************************************************/
-const AliHBTParticleCut& AliHBTParticleCut::operator = (const AliHBTParticleCut& in)
+AliHBTParticleCut& AliHBTParticleCut::operator=(const AliHBTParticleCut& in)
 {
   //assigment operator
+  Info("operator=","operator=operator=operator=operator=\noperator=operator=operator=operator=");
   for (Int_t i = 0;i<fNCuts;i++)
    {
      delete fCuts[i];
@@ -80,10 +81,11 @@ AliHBTParticleCut::~AliHBTParticleCut()
 } 
 /******************************************************************/
 
-Bool_t AliHBTParticleCut::Pass(AliHBTParticle* p)
+Bool_t AliHBTParticleCut::Pass(AliHBTParticle* p) const
 {
 //method checks all the cuts that are set (in the list)
 //If any of the baseCuts rejects particle False(rejection) is returned
+
  if(!p) 
   {
     Warning("Pass()","No Pasaran! We never accept NULL pointers");
@@ -410,7 +412,7 @@ AliHBTLogicalOperCut::~AliHBTLogicalOperCut()
 }
 /******************************************************************/
 
-Bool_t AliHBTLogicalOperCut::AliHBTDummyBaseCut::Pass(AliHBTParticle* /*part*/)
+Bool_t AliHBTLogicalOperCut::AliHBTDummyBaseCut::Pass(AliHBTParticle* /*part*/)  const
 {
   //checks if particles passes properties defined by this cut
   Warning("Pass","You are using dummy base cut! Probobly some logical cut is not set up properly");
@@ -448,7 +450,7 @@ void AliHBTLogicalOperCut::Streamer(TBuffer &b)
 /******************************************************************/
 ClassImp(AliHBTOrCut)
 
-Bool_t AliHBTOrCut::Pass(AliHBTParticle * p)
+Bool_t AliHBTOrCut::Pass(AliHBTParticle * p) const
 {
   //returns true when rejected 
   //AND operation is a little bit misleading but is correct
@@ -461,7 +463,7 @@ Bool_t AliHBTOrCut::Pass(AliHBTParticle * p)
 
 ClassImp(AliHBTAndCut)
 
-Bool_t AliHBTAndCut::Pass(AliHBTParticle * p)
+Bool_t AliHBTAndCut::Pass(AliHBTParticle * p)  const
 {
   //returns true when rejected 
   //OR operation is a little bit misleading but is correct

diff --git a/HBTAN/AliHBTParticleCut.h b/HBTAN/AliHBTParticleCut.h
index 8fd133539962a308d1459f33142c4b793b93ff54..fbcec43432eff807f31032dff06c1e516fa4e28e 100644 (file)
--- a/HBTAN/AliHBTParticleCut.h
+++ b/HBTAN/AliHBTParticleCut.h
@@ -1,3 +1,5 @@
+#ifndef ALIHBTPARTICLECUT_H
+#define ALIHBTPARTICLECUT_H
 //__________________________________________________________________________
 ////////////////////////////////////////////////////////////////////////////
 //                                                                        //
@@ -21,9 +23,6 @@
 ////////////////////////////////////////////////////////////////////////////
 
 
-#ifndef ALIHBTPARTICLECUT_H
-#define ALIHBTPARTICLECUT_H
-
 #include <TObject.h>
 #include "AliHBTParticle.h"
 
@@ -71,9 +70,9 @@ class AliHBTParticleCut: public TObject
     AliHBTParticleCut();
     AliHBTParticleCut(const AliHBTParticleCut& in);
     virtual ~AliHBTParticleCut();
-    const AliHBTParticleCut& operator = (const AliHBTParticleCut& in);
+    AliHBTParticleCut& operator = (const AliHBTParticleCut& in);
     
-    virtual Bool_t Pass(AliHBTParticle* p);
+    virtual Bool_t Pass(AliHBTParticle* p) const;
     Bool_t IsEmpty() const {return kFALSE;}
     
     void AddBasePartCut(AliHbtBaseCut* basecut);
@@ -123,7 +122,7 @@ class AliHBTEmptyParticleCut:  public AliHBTParticleCut
     AliHBTEmptyParticleCut(){};
     virtual ~AliHBTEmptyParticleCut(){};
     
-    Bool_t Pass(AliHBTParticle*){return kFALSE;} //accept everything <<CAN NOT BE const!!!!>>
+    Bool_t Pass(AliHBTParticle*) const {return kFALSE;} //accept everything <<CAN NOT BE const!!!!>>
     Bool_t IsEmpty() const {return kTRUE;}
 
     ClassDef(AliHBTEmptyParticleCut,1)
@@ -146,7 +145,7 @@ class AliHbtBaseCut: public TObject
 
      virtual           ~AliHbtBaseCut(){}
      
-     virtual Bool_t    Pass(AliHBTParticle *p);
+     virtual Bool_t    Pass(AliHBTParticle *p) const;
      
      void              SetRange(Double_t min, Double_t max){fMin = min; fMax = max;}
 
@@ -173,7 +172,7 @@ class AliHbtBaseCut: public TObject
  };
 
 inline Bool_t
-AliHbtBaseCut::Pass(AliHBTParticle *p)
+AliHbtBaseCut::Pass(AliHBTParticle *p) const
 {
   //cjecks if particle property fits in range
   if ( (GetValue(p) < fMin) || (GetValue(p) > fMax ) ) return kTRUE; //rejected
@@ -211,7 +210,7 @@ class AliHBTEnergyCut: public AliHbtBaseCut
     AliHBTEnergyCut(Double_t min = 0.0, Double_t max = 0.0):AliHbtBaseCut(min,max,kHbtE){}
     virtual ~AliHBTEnergyCut(){}
   protected:
-    Double_t  GetValue(AliHBTParticle * p)const{return p->Energy();}
+    Double_t  GetValue(AliHBTParticle * p)const {return p->Energy();}
     ClassDef(AliHBTEnergyCut,1)
  };
 
@@ -361,7 +360,7 @@ class AliHBTLogicalOperCut:  public AliHbtBaseCut
     class  AliHBTDummyBaseCut: public AliHbtBaseCut 
      {
        Double_t  GetValue(AliHBTParticle * /*part*/) const {return 0.0;}
-       Bool_t    Pass(AliHBTParticle* /*part*/);
+       Bool_t    Pass(AliHBTParticle* /*part*/) const;
      };
      
     ClassDef(AliHBTLogicalOperCut,1)
@@ -373,7 +372,7 @@ class AliHBTOrCut: public AliHBTLogicalOperCut
      AliHBTOrCut(){}
      AliHBTOrCut(AliHbtBaseCut* first, AliHbtBaseCut* second):AliHBTLogicalOperCut(first,second){}
      virtual   ~AliHBTOrCut(){}
-     Bool_t    Pass(AliHBTParticle *p);
+     Bool_t    Pass(AliHBTParticle *p) const;
      ClassDef(AliHBTOrCut,1)
 };
 
@@ -383,7 +382,7 @@ class AliHBTAndCut: public AliHBTLogicalOperCut
      AliHBTAndCut(){}
      AliHBTAndCut(AliHbtBaseCut* first, AliHbtBaseCut* second):AliHBTLogicalOperCut(first,second){}
      virtual   ~AliHBTAndCut(){}
-     Bool_t    Pass(AliHBTParticle *p);
+     Bool_t    Pass(AliHBTParticle *p) const;
      ClassDef(AliHBTAndCut,1)
 };