Coding violations in HLT/MUON
authoralibrary <alibrary@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 20 Apr 2006 04:37:34 +0000 (04:37 +0000)
committeralibrary <alibrary@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 20 Apr 2006 04:37:34 +0000 (04:37 +0000)
20 files changed:
HLT/MUON/src/AliRoot/ADCStream.hpp
HLT/MUON/src/AliRoot/ADCStreamSource.cxx
HLT/MUON/src/AliRoot/ADCStreamSource.hpp
HLT/MUON/src/AliRoot/AliHLTMUONTracker.h
HLT/MUON/src/AliRoot/ClusterFinderProxy.cxx
HLT/MUON/src/AliRoot/ClusterFinderProxy.hpp
HLT/MUON/src/AliRoot/ClusterSource.hpp
HLT/MUON/src/AliRoot/MUONHLTLinkDef.hpp
HLT/MUON/src/AliRoot/MicrodHLT.cxx
HLT/MUON/src/AliRoot/Point.hpp
HLT/MUON/src/AliRoot/Region.cxx
HLT/MUON/src/AliRoot/Region.hpp
HLT/MUON/src/AliRoot/Track.cxx
HLT/MUON/src/AliRoot/Track.hpp
HLT/MUON/src/AliRoot/TrackerProxy.cxx
HLT/MUON/src/AliRoot/TrackerProxy.hpp
HLT/MUON/src/AliRoot/TriggerRecord.cxx
HLT/MUON/src/AliRoot/TriggerRecord.hpp
HLT/MUON/src/AliRoot/TriggerSource.hpp
HLT/MUON/src/Tracking/MansoTracker.cxx

index 74ae83cc8fce229d4d6352f7ec5b0c3b6be095b3..efe0a6f3fb6895e1b3c317221b1da9d147f942bf 100644 (file)
@@ -5,8 +5,8 @@
 //
 ////////////////////////////////////////////////////////////////////////////////
 
-#ifndef ALIHLTMUONSTREAM_H
-#define ALIHLTMUONSTREAM_H
+#ifndef ALIHLTMUONADCSTREAM_H
+#define ALIHLTMUONADCSTREAM_H
 
 #include <TObject.h>
 #include <TArrayI.h>
@@ -15,6 +15,9 @@
 
 class AliHLTMUONADCStream : public TObject
 {
+       // ostream operator usefull for text output.
+       friend ostream& operator << (ostream& os, const AliHLTMUONADCStream& s);
+
 public:
 
        /* Default constructor initialises everything to zero.
@@ -35,16 +38,13 @@ public:
 //     UInt_t& operator [] (const UInt_t index);
        UInt_t operator [] (UInt_t index) const;
 
-       // ostream operator usefull for text output.
-       friend ostream& operator << (ostream& os, const AliHLTMUONADCStream& s);
-
 private:
 
        // TODO: complete the ADC stream specification.
-       TArrayI fData;
+       TArrayI fData;  // The DDL raw data.
 
        ClassDef(AliHLTMUONADCStream, 1)  // ADC stream data.
 };
 
 
-#endif // ALIHLTMUONSTREAM_H
+#endif // ALIHLTMUONADCSTREAM_H
index 0a23816e8e7e1f2b30fd1e36cc7771537562144c..c3d4d61c08466742ebd87b50a4478c7abbc22fd0 100644 (file)
@@ -5,6 +5,12 @@
 //
 ////////////////////////////////////////////////////////////////////////////////
 
+/* AliHLTMUONADCStreamSource is used to store the raw DDL data generated
+   from an AliRoot simulation.
+   This class is used as a storage class for the input dataset for
+   AliHLTMUONMicrodHLT.
+ */
+
 #include "AliRoot/ADCStreamSource.hpp"
 #include "AliRoot/Base.hpp"
 #include "TSystem.h"
 // TODO: Change all the Error message statements to AliError at some stage.
 
 ClassImp(AliHLTMUONADCStreamSource)
-ClassImp(AliHLTMUONADCStreamSource::DataBlock)
+ClassImp(AliHLTMUONADCStreamSource::AliDataBlock)
 
 
-AliHLTMUONADCStreamSource::AliHLTMUONADCStreamSource() : TObject()
+AliHLTMUONADCStreamSource::AliHLTMUONADCStreamSource()
+       : TObject(), fList(AliHLTMUONADCStreamSource::AliDataBlock::Class())
 {
        fCurrentStream = -1;
 }
@@ -30,6 +37,8 @@ AliHLTMUONADCStreamSource::~AliHLTMUONADCStreamSource()
 
 void AliHLTMUONADCStreamSource::FillFromFile(const TString& filename, Int_t eventnumber)
 {
+// Fills the internal data structures from the specified file.
+
        DebugMsg(1, "Entering FillFromFile, file = " << filename.Data()
                << ", event number = " << eventnumber
        );
@@ -65,6 +74,10 @@ void AliHLTMUONADCStreamSource::FillFromFile(const TString& filename, Int_t even
 
 void AliHLTMUONADCStreamSource::FillFrom(const TString& directory, Int_t eventnumber)
 {
+// Fills the internal data structures from the specified directory.
+// FillFromFile is called for every file in the directory that is
+// prefixed with MUON_ and ends in .ddl
+
        DebugMsg(1, "Entering FillFrom, directory = " << directory.Data()
                << ", event number = " << eventnumber
        );
@@ -91,8 +104,14 @@ void AliHLTMUONADCStreamSource::FillFrom(const TString& directory, Int_t eventnu
 }
 
 
-void AliHLTMUONADCStreamSource::FillFrom(const TString& dirprefix, UInt_t firstevent, UInt_t lastevent)
+void AliHLTMUONADCStreamSource::FillFrom(
+               const TString& dirprefix, UInt_t firstevent, UInt_t lastevent
+       )
 {
+// Same as the methods above except the directory name is created as
+// dirprefix + eventnumber, where eventnumber is looped from firstevent
+// to lastevent.
+
        DebugMsg(1, "Entering FillFrom");
        
        for (UInt_t i = firstevent; i <= lastevent; i++)
@@ -108,19 +127,26 @@ void AliHLTMUONADCStreamSource::FillFrom(const TString& dirprefix, UInt_t firste
 
 void AliHLTMUONADCStreamSource::Clear(Option_t* /*option*/)
 {
+// Clears all the internal arrays.
+
        fCurrentStream = -1;
-       fList.erase( fList.begin(), fList.end() );
+       fList.Clear("C");
 }
 
 
 Int_t AliHLTMUONADCStreamSource::NumberOfStreams() const
 {
-       return fList.size();
+// Returns the number of ADC streams stored.
+
+       return fList.GetEntriesFast();
 }
 
 
 Bool_t AliHLTMUONADCStreamSource::GetStream(Int_t index) const
 {
+// Fetches the index'th ADC stream stored.
+// kTRUE is returned if the stream was found, kFALSE otherwise.
+
        if ( 0 <= index && index < NumberOfStreams() )
        {
                fCurrentStream = index;
@@ -146,6 +172,9 @@ Bool_t AliHLTMUONADCStreamSource::GetStream(Int_t index) const
 
 Bool_t AliHLTMUONADCStreamSource::FirstStream() const
 {
+// Fetches the first ADC stream stored.
+// kTRUE is returned if the stream was found, kFALSE otherwise.
+
        if (NumberOfStreams() > 0)
        {
                fCurrentStream = 0;
@@ -158,6 +187,9 @@ Bool_t AliHLTMUONADCStreamSource::FirstStream() const
 
 Bool_t AliHLTMUONADCStreamSource::NextStream() const
 {
+// Fetches the next ADC stream stored following the currently selected one.
+// kTRUE is returned if the stream was found, kFALSE otherwise.
+
        if ( 0 <= fCurrentStream && fCurrentStream < NumberOfStreams() - 1 )
        {
                fCurrentStream++;
@@ -170,10 +202,13 @@ Bool_t AliHLTMUONADCStreamSource::NextStream() const
 
 Int_t AliHLTMUONADCStreamSource::EventNumber() const
 {
+// Returns the corresponding AliRoot event number for the current stream.
+// -1 is returned if no stream is selected.
+
        if (fCurrentStream >= 0)
        {
                Assert( fCurrentStream < NumberOfStreams() );
-               return fList[fCurrentStream].fEventNumber;
+               return ((AliDataBlock*)fList[fCurrentStream])->EventNumber();
        }
        else
        {
@@ -185,10 +220,13 @@ Int_t AliHLTMUONADCStreamSource::EventNumber() const
 
 Bool_t AliHLTMUONADCStreamSource::FetchStream(AliHLTMUONADCStream& stream) const
 {
+// Returns the current ADC stream selected.
+// kFALSE is returned if there is no stream selected.
+
        if (fCurrentStream >= 0)
        {
                Assert( fCurrentStream < NumberOfStreams() );
-               stream = fList[fCurrentStream].fStream;
+               stream = ((AliDataBlock*)fList[fCurrentStream])->Stream();
                return kTRUE;
        }
        else
@@ -201,6 +239,9 @@ Bool_t AliHLTMUONADCStreamSource::FetchStream(AliHLTMUONADCStream& stream) const
 
 Bool_t AliHLTMUONADCStreamSource::FetchStream(Int_t index, AliHLTMUONADCStream& stream) const
 {
+// Returns the index'th ADC stream.
+// kTRUE is returned if the stream was found, kFALSE otherwise.
+
        if ( GetStream(index) )
                return FetchStream(stream);
        else
@@ -210,10 +251,13 @@ Bool_t AliHLTMUONADCStreamSource::FetchStream(Int_t index, AliHLTMUONADCStream&
 
 const AliHLTMUONADCStream* AliHLTMUONADCStreamSource::FetchStream() const
 {
+// Returns the current ADC stream selected.
+// A NULL pointer is returned if no ADC stream is selected.
+
        if (fCurrentStream >= 0)
        {
                Assert( fCurrentStream < NumberOfStreams() );
-               return &( fList[fCurrentStream].fStream );
+               return &( ((AliDataBlock*)fList[fCurrentStream])->Stream() );
        }
        else
        {
@@ -223,15 +267,18 @@ const AliHLTMUONADCStream* AliHLTMUONADCStreamSource::FetchStream() const
 }
 
 
-void AliHLTMUONADCStreamSource::AddStream(AliHLTMUONADCStream& stream, UInt_t eventnumber)
+void AliHLTMUONADCStreamSource::AddStream(
+               AliHLTMUONADCStream& stream, UInt_t eventnumber
+       )
 {
+// Adds a new AliHLTMUONADCStream object to the internal arrays.
+
        DebugMsg(1, "Entering AddStream");
        
-       DataBlock newdata;
-       newdata.fEventNumber = eventnumber;
-       newdata.fStream = stream;
-       fList.push_back(newdata);
-       fCurrentStream = fList.size() - 1;
+       fCurrentStream = fList.GetEntriesFast();
+       new ( fList[fCurrentStream] ) AliDataBlock();
+       ((AliDataBlock*)fList[fCurrentStream])->EventNumber() = eventnumber;
+       ((AliDataBlock*)fList[fCurrentStream])->Stream() = stream;
        
        DebugMsg(1, "Leaving AddStream");
 }
index 6fa5d444cb02c719f763a9885bf782fb8d6f0d20..956e1aab0584ae374f4c31decc6710ee604f5e18 100644 (file)
@@ -5,19 +5,42 @@
 //
 ////////////////////////////////////////////////////////////////////////////////
 
+/* AliHLTMUONADCStreamSource is used to store the raw DDL data generated
+   from an AliRoot simulation.
+   This class is used as a storage class for the input dataset for
+   AliHLTMUONMicrodHLT.
+ */
+
 #ifndef ALIHLTMUONADCSTREAMSOURCE_H
 #define ALIHLTMUONADCSTREAMSOURCE_H
 
 #include "TROOT.h"
 #include "TObject.h"
+#include "TClonesArray.h"
 #include "TString.h"
 #include "AliRoot/ADCStream.hpp"
 
-#include <vector>
-
 
 class AliHLTMUONADCStreamSource : public TObject
 {
+public:  // Unfortunately ROOT requires the following to be public.
+       
+       class AliDataBlock : public TObject
+       {
+       public:
+               virtual ~AliDataBlock() {};
+               
+               Int_t& EventNumber() { return fEventNumber; }
+               AliHLTMUONADCStream& Stream() { return fStream; }
+
+       private:
+       
+               Int_t fEventNumber;  // Event number of the stream.
+               AliHLTMUONADCStream fStream;  // The ADC stream block.
+               
+               ClassDef(AliDataBlock, 1)  // Data per event.
+       };
+
 public:
 
        AliHLTMUONADCStreamSource();
@@ -92,23 +115,8 @@ private:
        void AddStream(AliHLTMUONADCStream& stream, UInt_t eventnumber);
 
        mutable Int_t fCurrentStream;  //! The currently selected stream index.
-       
-
-public:  // Unfortunately ROOT requires the following to be public.
-       
-       struct DataBlock
-       {
-               virtual ~DataBlock() {};
-
-               Int_t fEventNumber;  // Event number of the stream.
-               AliHLTMUONADCStream fStream;  // The ADC stream block.
-               
-               ClassDef(DataBlock, 1)  // Data per event.
-       };
-
-private:
 
-       std::vector<DataBlock> fList;  // List of ADC streams.
+       TClonesArray fList;  // List of ADC streams.
 
        ClassDef(AliHLTMUONADCStreamSource, 1)  // The source of ADC stream data for dHLT.
 };
index 55ef39a7e2978faf63ae812f6ef67be131355c05..6a3ce0f812860aa6e7bacf69e39d4fc66901f40e 100644 (file)
@@ -35,6 +35,12 @@ public:
        virtual Int_t Clusters2Tracks(AliESD* event);
 
 private:
+       
+       // Do not allow copying of this object.
+       AliHLTMUONTracker(const AliHLTMUONTracker& /*object*/) : AliTracker() {}
+       AliHLTMUONTracker& operator = (const AliHLTMUONTracker& /*object*/) { return *this; }
+
+
        const AliHLTMUONTriggerRecord*
        FindTriggerRecord(const AliHLTMUONTrack* track) const;
 
index 77d54a1b6fe0d4aa8afc193b2f44ae0760d2a304..55cdb1c2066bd6e2b01b972040bf60d489e66f52 100644 (file)
 AliHLTMUONClusterFinderProxy::AliHLTMUONClusterFinderProxy(AliHLTMUONClusterFinderInterface* client)
        : AliHLTMUONCoreClusterFinder(), AliHLTMUONClusterFinderCallback()
 {
-       clusterfinder = client;
+       fClusterFinder = client;
 }
 
 
 void AliHLTMUONClusterFinderProxy::FindClusters(const AliHLTMUONCoreADCStream* /*stream*/)
 {
+// Finds clusters in the ADC stream by invoking the cluster finder.
+
        // TODO: perform conversion
        //ADCStream adc = AliHLTMUONConvert(stream);
        AliHLTMUONADCStream adc;
        DebugMsg(6, "AliHLTMUONClusterFinderProxy::FindClusters: " << adc);
-       clusterfinder->FindClusters(&adc);
+       fClusterFinder->FindClusters(&adc);
 }
 
 
 UInt AliHLTMUONClusterFinderProxy::FillClusterData(AliHLTMUONCoreClusterPoint* clusters, UInt arraysize)
 {
+// Fills the cluster points array with the clusters found.
+
        UInt result;
        AliHLTMUONPoint* points = new AliHLTMUONPoint[arraysize];
        try
        {
                DebugMsg(6, "AliHLTMUONClusterFinderProxy::FillClusterData");
-               result = clusterfinder->FillClusterData(points, arraysize);
+               result = fClusterFinder->FillClusterData(points, arraysize);
                for (UInt i = 0; i < arraysize; i++)
                {
                        clusters[i] = AliHLTMUONConvert(points[i]);
@@ -54,7 +58,7 @@ UInt AliHLTMUONClusterFinderProxy::FillClusterData(AliHLTMUONCoreClusterPoint* c
 void AliHLTMUONClusterFinderProxy::Reset()
 {
        DebugMsg(6, "AliHLTMUONClusterFinderProxy::Reset");
-       clusterfinder->Reset();
+       fClusterFinder->Reset();
 }
 
 
@@ -70,3 +74,4 @@ void AliHLTMUONClusterFinderProxy::NoClustersFound()
        DebugMsg(6, "AliHLTMUONClusterFinderProxy::NoClustersFound");
        AliHLTMUONCoreClusterFinder::NoClustersFound();
 }
+
index e15e57aa7585f4b84e77c2917be8fe7479f58d2d..d4e6128723554cb4411ad183883aa055b69907ac 100644 (file)
@@ -30,8 +30,16 @@ public:
        virtual void NoClustersFound();
 
 private:
+       // Do not allow copying.
+       AliHLTMUONClusterFinderProxy(const AliHLTMUONClusterFinderProxy& /*object*/)
+               : AliHLTMUONCoreClusterFinder(), AliHLTMUONClusterFinderCallback()
+       {}
 
-       AliHLTMUONClusterFinderInterface* clusterfinder;  // The clusterfinder we are proxying for.
+       AliHLTMUONClusterFinderProxy& operator = (const AliHLTMUONClusterFinderProxy& /*object*/)
+       { return *this; }
+
+
+       AliHLTMUONClusterFinderInterface* fClusterFinder;  // The clusterfinder we are proxying for.
 };
 
 
index 8c7285e287806e64cfe60825d4be2e9b9afbea99..3ece1d9ca51046fde0c558de1d693865284b9d16 100644 (file)
@@ -24,6 +24,45 @@ class AliMUONDataInterface;
 
 class AliHLTMUONClusterSource : public TObject
 {
+public:  // Unfortunately ROOT requires the following to be public.
+       
+       class AliBlockData : public TObject
+       {
+       public:
+               AliBlockData();
+               AliBlockData(Int_t chamber);
+               virtual ~AliBlockData();
+
+               Int_t& Chamber() { return fChamber; };
+               TClonesArray& Clusters() { return fClusters; };
+
+       private:
+
+               Int_t fChamber;  // The chamber number this block of clusters came from.
+               TClonesArray fClusters;  // The cluster points in this block.
+               
+               ClassDef(AliBlockData, 1);  // Data per block.
+       };
+       
+       class AliEventData : public TObject
+       {
+       public:
+               AliEventData();
+               AliEventData(Int_t eventnumber);
+               virtual ~AliEventData();
+
+               Int_t& EventNumber() { return fEventNumber; };
+               TClonesArray& Blocks() { return fBlocks; };
+
+       private:
+
+               Int_t fEventNumber;  // Event number in AliMUONDataInterface from which the clusters were taken.
+               TClonesArray fBlocks;  // The list of cluster blocks for this event.
+               
+               ClassDef(AliEventData, 1);  // Data per event.
+       };
+
+
 public:
 
        enum AreaType
@@ -260,47 +299,6 @@ private:
        void ResetClusterPointers() const;
        
 
-public:  // Unfortunately ROOT requires the following to be public.
-       
-       class AliBlockData : public TObject
-       {
-       public:
-               AliBlockData();
-               AliBlockData(Int_t chamber);
-               virtual ~AliBlockData();
-
-               Int_t& Chamber() { return fChamber; };
-               TClonesArray& Clusters() { return fClusters; };
-
-       private:
-
-               Int_t fChamber;  // The chamber number this block of clusters came from.
-               TClonesArray fClusters;  // The cluster points in this block.
-               
-               ClassDef(AliBlockData, 1);  // Data per block.
-       };
-       
-       class AliEventData : public TObject
-       {
-       public:
-               AliEventData();
-               AliEventData(Int_t eventnumber);
-               virtual ~AliEventData();
-
-               Int_t& EventNumber() { return fEventNumber; };
-               TClonesArray& Blocks() { return fBlocks; };
-
-       private:
-
-               Int_t fEventNumber;  // Event number in AliMUONDataInterface from which the clusters were taken.
-               TClonesArray fBlocks;  // The list of cluster blocks for this event.
-               
-               ClassDef(AliEventData, 1);  // Data per event.
-       };
-
-
-private:
-
        // Dont allow copying.
        AliHLTMUONClusterSource(const AliHLTMUONClusterSource& /*object*/) : TObject() {};
        AliHLTMUONClusterSource& operator = (const AliHLTMUONClusterSource& /*object*/) { return *this; };
index 23ff255f0766f31f2d137ec8032582921108c800..c4316292779297fc8df2125c79fa1805b6083b8a 100644 (file)
@@ -30,7 +30,7 @@
 #pragma link C++ class AliHLTMUONMicrodHLT+;
 
 #pragma link C++ class AliHLTMUONADCStreamSource+;
-#pragma link C++ struct AliHLTMUONADCStreamSource::DataBlock+;
+#pragma link C++ struct AliHLTMUONADCStreamSource::AliDataBlock+;
 
 #pragma link C++ class AliHLTMUONTriggerSource+;
 #pragma link C++ enum AliHLTMUONTriggerSource::AreaType+;
index 2d296cf9c754fb43a5c01824776a91096386f7d2..e1f2696b64255dc813abd7eecc85ada685293056 100644 (file)
@@ -55,6 +55,7 @@ class AliMicroFramework : public AliHLTMUONCoreTrackerCallback
 {
 public:
 
+       AliMicroFramework();
        virtual ~AliMicroFramework() {}
 
        virtual void RequestClusters(
@@ -96,6 +97,14 @@ public:
 
 private:
 
+       // Do not allow copying
+       AliMicroFramework(const AliMicroFramework& /*object*/)
+               : AliHLTMUONCoreTrackerCallback()
+       {}
+       
+       AliMicroFramework& operator = (const AliMicroFramework& /*object*/) { return *this; }
+
+
        void CountClusterPoints(const AliHLTMUONClusterSource* cs);
        void CreateClusterBlocks(const AliHLTMUONClusterSource* cs, Int eventnumber);
        void FreeClusterBlocks();
@@ -109,6 +118,19 @@ private:
 
 //-----------------------------------------------------------------------------
 
+AliMicroFramework::AliMicroFramework()
+{
+// Default constructor.
+
+       for (Int_t i = 0; i < 10; i++)
+       {
+               fClusterCount[i] = 0;
+               fClusters[i] = NULL;
+       }
+       fTrackOutput = NULL;
+       fCurrentTriggerNumber = 0;
+}
+
 
 void AliMicroFramework::RequestClusters(
                AliHLTMUONCoreTracker* tracker,
index 20054ba2265b2753f6838fab5a836e87435dd27f..1b0ba80d2cdc06c0d0626b24e2d364189c43af4a 100644 (file)
@@ -14,6 +14,9 @@
 
 class AliHLTMUONPoint : public TObject
 {
+       // ostream operator usefull for text output.
+       friend ostream& operator << (ostream& os, const AliHLTMUONPoint& p);
+
 public:
 
        /* Default constructor initialises everything to zero.
@@ -26,9 +29,6 @@ public:
 
        virtual ~AliHLTMUONPoint() {};
        
-       // ostream operator usefull for text output.
-       friend ostream& operator << (ostream& os, const AliHLTMUONPoint& p);
-
 
        Float_t fX;    // X coordinate of the 2D point.
        Float_t fY;    // Y coordinate of the 2D point.
index ae42348778b7d4836643fa57347a43ad75d73686..607d5fe4b4ad38f0b7f18e9451c2d63e83301e1d 100644 (file)
@@ -22,6 +22,9 @@ AliHLTMUONRegion::AliHLTMUONRegion() : TObject()
 AliHLTMUONRegion::AliHLTMUONRegion(Float_t left, Float_t right, Float_t bottom, Float_t top)
         : TObject()
 {
+// Construct a region of interest from the boundary box defined by the borders
+// 'left', 'right', 'top' and 'bottom'.
+
        if (left > right)
        {
                fLeft = fRight = fBottom = fTop = 0.0;
@@ -44,6 +47,8 @@ AliHLTMUONRegion::AliHLTMUONRegion(Float_t left, Float_t right, Float_t bottom,
 
 void AliHLTMUONRegion::Left(Float_t value)
 {
+// Set the left border of the ROI.
+
        if (value > fRight)
                Error("Left", "Trying to assign fLeft (%f) larger than fRight (%f).", value, fRight);
        else
@@ -53,6 +58,8 @@ void AliHLTMUONRegion::Left(Float_t value)
 
 void AliHLTMUONRegion::Right(Float_t value)
 {
+// Set the right border of the ROI.
+
        if (value < fLeft)
                Error("Right", "Trying to assign fRight (%f) smaller than fLeft (%f).", value, fLeft);
        else
@@ -62,6 +69,8 @@ void AliHLTMUONRegion::Right(Float_t value)
 
 void AliHLTMUONRegion::Bottom(Float_t value)
 {
+// Set the bottom border of the ROI.
+
        if (value > fTop)
                Error("Bottom", "Trying to assign fBottom (%f) larger than fTop (%f).", value, fTop);
        else
@@ -71,6 +80,8 @@ void AliHLTMUONRegion::Bottom(Float_t value)
 
 void AliHLTMUONRegion::Top(Float_t value)
 {
+// Set the top border of the ROI.
+
        if (value < fBottom)
                Error("Top", "Trying to assign fTop (%f) smaller than fBottom (%f).", value, fBottom);
        else
@@ -80,6 +91,9 @@ void AliHLTMUONRegion::Top(Float_t value)
 
 Bool_t AliHLTMUONRegion::Contains(const AliHLTMUONPoint& p) const
 {
+// Checks if the point is within this region. If it is then kTRUE is returned
+// otherwise kFALSE is returned.
+
        return 
          fLeft <= p.fX 
          && p.fX <= fRight 
index ed22f1cfd46c80415010740c53a4d5c54ac8a3e3..dda92abc2ca4f22fb8e867c3be5eb8148ca30880 100644 (file)
@@ -16,6 +16,9 @@ class AliHLTMUONPoint;
 
 class AliHLTMUONRegion : public TObject
 {
+       // ostream operator usefull for text output.
+       friend ostream& operator << (ostream& os, const AliHLTMUONRegion& r);
+
 public:
 
        /* Default constructor initialises everything to zero.
@@ -51,9 +54,6 @@ public:
         */
        Bool_t Contains(const AliHLTMUONPoint& p) const;
 
-       // ostream operator usefull for text output.
-       friend ostream& operator << (ostream& os, const AliHLTMUONRegion& r);
-
 private:
 
        Float_t fLeft;    // Left boundary of boundary box.
index eda232c35dd38c1aab5fdc80ea8e4befeb8ad3c2..7befe20e37226443b77e1f08e02b23e727516ad9 100644 (file)
@@ -14,6 +14,8 @@ ClassImp(AliHLTMUONTrack)
 
 AliHLTMUONTrack::AliHLTMUONTrack() : TObject()
 {
+// Default constructor initialises everything to zero and fTriggerID to -1.
+
        Init();
 }
 
@@ -23,6 +25,12 @@ AliHLTMUONTrack::AliHLTMUONTrack(
                const AliHLTMUONPoint hits[10], const AliHLTMUONRegion regions[10]
        ) : TObject()
 {
+// Create a track object from the given parameters.
+// This constructor checks that momentum >= pt and sign is one of the
+// following values: -1, 0 or +1. If these conditions are violated then
+// the internal data is initialised as in the default constructor and an
+// error message is displayed.
+
        if (sign < -1 || +1 < sign)
        {
                Init();
@@ -57,6 +65,8 @@ AliHLTMUONTrack::AliHLTMUONTrack(
 
 void AliHLTMUONTrack::Init()
 {
+// Internal initialisation routine used by the constructors.
+
        fTriggerID = -1;
        fParticleSign = 0;
        fP = fPt = 0.0;
@@ -65,6 +75,11 @@ void AliHLTMUONTrack::Init()
 
 void AliHLTMUONTrack::ParticleSign(Int_t value)
 {
+// Set method for the particle sign. The particle sign must be one
+// of the following values: -1, 0 or +1
+// If the new value is not in this range then an error message is
+// displayed and the internal value remain unchanged.
+
        if (-1 <= value && value <= +1)
                fParticleSign = value;
        else
@@ -77,6 +92,11 @@ void AliHLTMUONTrack::ParticleSign(Int_t value)
 
 void AliHLTMUONTrack::P(Float_t value)
 {
+// The set method for the momentum.
+// This method checks that the momentum is always equal or larger than
+// the pt. If not then the internal values are left unchanged and an
+// error message is displayed. The numbers must also be positive.
+
        if (value >= fPt)
                fP = value;
        else
@@ -88,6 +108,11 @@ void AliHLTMUONTrack::P(Float_t value)
 
 void AliHLTMUONTrack::Pt(Float_t value)
 {
+// The set method for the pt.
+// This method checks that the momentum is always equal or larger than
+// the pt. If not then the internal values are left unchanged and an
+// error message is displayed. The numbers must also be positive.
+
        if (value >= 0.0)
        {
                if (value <= fP)
@@ -105,6 +130,10 @@ void AliHLTMUONTrack::Pt(Float_t value)
 
 AliHLTMUONPoint& AliHLTMUONTrack::Hit(UInt_t chamber)
 {
+// Returns the hit point for the specified chamber.
+// If the chamber number in out of bounds the point on the first
+// chamber is returned and an error message displayed.
+
        if (chamber < 10)
                return fHit[chamber];
        else
@@ -120,6 +149,10 @@ AliHLTMUONPoint& AliHLTMUONTrack::Hit(UInt_t chamber)
 
 const AliHLTMUONPoint& AliHLTMUONTrack::Hit(UInt_t chamber) const
 {
+// Returns a constant hit object for the specified chamber.
+// If the chamber number in out of bounds the point on the first
+// chamber is returned and an error message displayed.
+
        if (chamber < 10)
                return fHit[chamber];
        else
@@ -135,6 +168,8 @@ const AliHLTMUONPoint& AliHLTMUONTrack::Hit(UInt_t chamber) const
 
 void AliHLTMUONTrack::Hit(UInt_t chamber, const AliHLTMUONPoint& value)
 {
+// Set method for hits. The chamber must be in the range [0..9]
+
        if (chamber < 10)
                fHit[chamber] = value;
        else
@@ -147,6 +182,10 @@ void AliHLTMUONTrack::Hit(UInt_t chamber, const AliHLTMUONPoint& value)
 
 AliHLTMUONRegion& AliHLTMUONTrack::RegionOfInterest(UInt_t chamber)
 {
+// Returns the region of interest for the specified chamber.
+// If the chamber number in out of bounds the region on the first
+// chamber is returned and an error message displayed.
+
        if (chamber < 10)
                return fRegionOfInterest[chamber];
        else
@@ -162,6 +201,10 @@ AliHLTMUONRegion& AliHLTMUONTrack::RegionOfInterest(UInt_t chamber)
 
 const AliHLTMUONRegion& AliHLTMUONTrack::RegionOfInterest(UInt_t chamber) const
 {
+// Returns the constatn region of interest object for the specified chamber.
+// If the chamber number in out of bounds the region on the first
+// chamber is returned and an error message displayed.
+
        if (chamber < 10)
                return fRegionOfInterest[chamber];
        else
@@ -177,6 +220,8 @@ const AliHLTMUONRegion& AliHLTMUONTrack::RegionOfInterest(UInt_t chamber) const
 
 void AliHLTMUONTrack::RegionOfInterest(UInt_t chamber, const AliHLTMUONRegion& value)
 {
+// Set method for regions. The chamber must be in the range [0..9]
+
        if (chamber < 10)
                fRegionOfInterest[chamber] = value;
        else
@@ -189,6 +234,10 @@ void AliHLTMUONTrack::RegionOfInterest(UInt_t chamber, const AliHLTMUONRegion& v
 
 Bool_t AliHLTMUONTrack::HitsInRegions() const
 {
+// Checks to see if the all the hits are within their respective regions
+// of interest for each chamber. kTRUE is returned if they are and kFALSE
+// otherwise.
+
        for (Int_t i = 0; i < 10; i++)
        {
                if ( ! fRegionOfInterest[i].Contains(fHit[i]) )
index 8a66bbcbe074cd0e18c712aa5803c6ab3dbaabc0..6945760acf28628823a67ad4732728c17c1ceb50 100644 (file)
@@ -17,6 +17,9 @@
 
 class AliHLTMUONTrack : public TObject
 {
+       // ostream operator usefull for text output.
+       friend ostream& operator << (ostream& os, const AliHLTMUONTrack& t);
+       
 public:
 
        /* Default constructor initialises everything to zero and fTriggerID to -1.
@@ -51,7 +54,7 @@ public:
        
        /* The get and set methods for the momentum and transverse momentum pt.
           These methods check that the momentum is always equal or larger than
-          the pt. If not then the internal value are left unchanged and an
+          the pt. If not then the internal values are left unchanged and an
           error message is displayed. The numbers must also be positive.
         */
        void P(Float_t value);
@@ -87,9 +90,6 @@ public:
         */
        Bool_t HitsInRegions() const;
 
-       // ostream operator usefull for text output.
-       friend ostream& operator << (ostream& os, const AliHLTMUONTrack& t);
-
 private:
 
        // Internal initialisation routine used by the constructors.
index 24649ee67ea36c1a2aee7eb85349d54d473971ec..4ddf45d9d5d3858282ada631943368e32f8e15c2 100644 (file)
 AliHLTMUONTrackerProxy::AliHLTMUONTrackerProxy(AliHLTMUONTrackerInterface* client)
        : AliHLTMUONCoreTracker(), AliHLTMUONTrackerCallback()
 {
-       tracker = client;
+       fTracker = client;
 }
 
 
 void AliHLTMUONTrackerProxy::FindTrack(const AliHLTMUONCoreTriggerRecord& trigger)
 {
+// Finds a track from the tigger seed by invoking the tracker object.
+
        AliHLTMUONTriggerRecord rec = AliHLTMUONConvert(trigger, 0);
        DebugMsg(6, "AliHLTMUONTrackerProxy::FindTrack : rec = " << rec);
-       tracker->FindTrack(rec);
+       fTracker->FindTrack(rec);
 }
 
 
 void AliHLTMUONTrackerProxy::ReturnClusters(void* tag, const AliHLTMUONCoreClusterPoint* clusters, UInt count)
 {
+// Passes the cluster points the tracker requested to the tracker.
+
        AliHLTMUONPoint* points = new AliHLTMUONPoint[count];
        try
        {
@@ -37,7 +41,7 @@ void AliHLTMUONTrackerProxy::ReturnClusters(void* tag, const AliHLTMUONCoreClust
                        points[i] = AliHLTMUONConvert(clusters[i]);
                        DebugMsg(6, "\tpoints[" << i << "] = " << points[i] );
                };
-               tracker->ReturnClusters(tag, points, count);
+               fTracker->ReturnClusters(tag, points, count);
        }
        finally
        (
@@ -48,15 +52,19 @@ void AliHLTMUONTrackerProxy::ReturnClusters(void* tag, const AliHLTMUONCoreClust
 
 void AliHLTMUONTrackerProxy::EndOfClusters(void* tag)
 {
+// Tells the tracker there are no more clusters.
+
        DebugMsg(6, "AliHLTMUONTrackerProxy::EndOfClusters");
-       tracker->EndOfClusters(tag);
+       fTracker->EndOfClusters(tag);
 }
 
 
 void AliHLTMUONTrackerProxy::FillTrackData(AliHLTMUONCoreTrack& track)
 {
+// Fills the tracker data for the found track.
+
        AliHLTMUONTrack data;
-       tracker->FillTrackData(data);
+       fTracker->FillTrackData(data);
        DebugMsg(6, "AliHLTMUONTrackerProxy::FillTrackData : data = " << data);
        track = AliHLTMUONConvert(data);
 }
@@ -65,7 +73,7 @@ void AliHLTMUONTrackerProxy::FillTrackData(AliHLTMUONCoreTrack& track)
 void AliHLTMUONTrackerProxy::Reset()
 {
        DebugMsg(6, "AliHLTMUONTrackerProxy::Reset");
-       tracker->Reset();
+       fTracker->Reset();
 }
 
 
index 057d1af40db1a2e49bc4e31add6357512b3c1174..ec344a3e368b8fee7aea8748f6b6451449689c1e 100644 (file)
@@ -38,7 +38,15 @@ public:
 
 private:
 
-       AliHLTMUONTrackerInterface* tracker;  // The tracker we are proxying for.
+       AliHLTMUONTrackerProxy(const AliHLTMUONTrackerProxy& /*object*/)
+               : AliHLTMUONCoreTracker(), AliHLTMUONTrackerCallback()
+       {}
+       
+       AliHLTMUONTrackerProxy& operator = (const AliHLTMUONTrackerProxy& /*object*/)
+       { return *this; }
+
+
+       AliHLTMUONTrackerInterface* fTracker;  // The tracker we are proxying for.
 };
 
 
index 861602dbdde5bf6b2349459748a8779acaf23140..b5a4b9089c60e016a1b6b2af1e702e7f1282a039 100644 (file)
@@ -13,6 +13,8 @@ ClassImp(AliHLTMUONTriggerRecord)
 
 AliHLTMUONTriggerRecord::AliHLTMUONTriggerRecord()
 {
+// Default constructor initialises everything to zero and the trigger number to -1.
+
        Init();
 }
 
@@ -22,6 +24,13 @@ AliHLTMUONTriggerRecord::AliHLTMUONTriggerRecord(
                const AliHLTMUONPoint& station1point, const AliHLTMUONPoint& station2point
        )
 {
+// Creates a trigger record from the specified parameters.
+// Note: the trigger number must be greater or equal to zero. The particle
+// sign must also be one of the following values: -1, 0 or +1
+// Pt must be a positive number.
+// If these conditions are not met then an error message is displayed and
+// the object is filled like it is in the default constructor. 
+
        if (triggernumber < 0)
        {
                Init();
@@ -59,6 +68,8 @@ AliHLTMUONTriggerRecord::AliHLTMUONTriggerRecord(
 
 void AliHLTMUONTriggerRecord::Init()
 {
+// Performs internal initialisation for the constructors.
+
        fTriggerNumber = -1;
        fParticleSign = 0;
        fPt = 0.0;
@@ -67,6 +78,11 @@ void AliHLTMUONTriggerRecord::Init()
 
 void AliHLTMUONTriggerRecord::TriggerNumber(Int_t value)
 {
+// Set method for the trigger number. 
+// The trigger number must be positive when assigning the trigger number.
+// If it is not then an error message is displayed and the internal value
+// remains untouched.
+
        if (value >= 0)
                fTriggerNumber = value;
        else
@@ -79,6 +95,11 @@ void AliHLTMUONTriggerRecord::TriggerNumber(Int_t value)
 
 void AliHLTMUONTriggerRecord::ParticleSign(Int_t value)
 {
+// Set method for the particle sign.
+// The particle sign must be one of the following values: -1, 0 or +1
+// If it is not then an error message is displayed and the internal value
+// remains untouched.
+
        if (-1 <= value && value <= +1)
                fParticleSign = value;
        else
@@ -91,6 +112,11 @@ void AliHLTMUONTriggerRecord::ParticleSign(Int_t value)
 
 void AliHLTMUONTriggerRecord::Pt(Float_t value)
 {
+// Set method for the particle Pt, as measured by the L0 trigger.
+// The pt must be a positive number when assigning the pt.
+// If it is not then an error message is displayed and the internal value
+// remains untouched.
+
        if (value >= 0)
                fPt = value;
        else
index 4aaa94603bc7996a126632abd9e4ea22caac34f2..e41b8a343e1687a488d63605819095aabb65074e 100644 (file)
@@ -15,6 +15,9 @@
 
 class AliHLTMUONTriggerRecord : public TObject
 {
+       // ostream operator usefull for text output.
+       friend ostream& operator << (ostream& os, const AliHLTMUONTriggerRecord& r);
+       
 public:
 
        /* Default constructor initialises everything to zero and the trigger
@@ -68,9 +71,7 @@ public:
        void Station2Point(const AliHLTMUONPoint& value) { fSt2Point = value; };
        AliHLTMUONPoint& Station2Point() { return fSt2Point; };
        const AliHLTMUONPoint& Station2Point() const { return fSt2Point; };
-
-       // ostream operator usefull for text output.
-       friend ostream& operator << (ostream& os, const AliHLTMUONTriggerRecord& r);
+       
 
 private:
 
index c0ecd6f80aee484d0ee8f276b4de93d10ac0be8a..18925958439aeeef310583bc6a87561a8f51f195 100644 (file)
@@ -27,6 +27,27 @@ class AliMUONDataInterface;
 
 class AliHLTMUONTriggerSource : public TObject
 {
+public:  // Unfortunately ROOT requires the following to be public.
+
+       class AliEventData : public TObject
+       {
+       public:
+               AliEventData();
+               AliEventData(Int_t eventnumber);
+               virtual ~AliEventData();
+               
+               Int_t& EventNumber() { return fEventNumber; }
+               TClonesArray& Blocks() { return fBlocks; }
+
+       private:
+       
+               Int_t fEventNumber;  // Event number in AliMUONDataInterface from which the triggers were taken.
+               TClonesArray fBlocks; // The list of blocks of trigger records.
+               
+               ClassDef(AliEventData, 1)  // Data per event.
+       };
+
+
 public:
 
        enum AreaType
@@ -286,28 +307,6 @@ private:
        void ResetTriggerPointers() const;
 
 
-public:  // Unfortunately ROOT requires the following to be public.
-
-       class AliEventData : public TObject
-       {
-       public:
-               AliEventData();
-               AliEventData(Int_t eventnumber);
-               virtual ~AliEventData();
-               
-               Int_t& EventNumber() { return fEventNumber; }
-               TClonesArray& Blocks() { return fBlocks; }
-
-       private:
-       
-               Int_t fEventNumber;  // Event number in AliMUONDataInterface from which the triggers were taken.
-               TClonesArray fBlocks; // The list of blocks of trigger records.
-               
-               ClassDef(AliEventData, 1)  // Data per event.
-       };
-       
-private:
-
        // Do not allow copying of this object.
        AliHLTMUONTriggerSource(const AliHLTMUONTriggerSource& /*object*/) : TObject() {}
        AliHLTMUONTriggerSource& operator = (const AliHLTMUONTriggerSource& /*object*/) { return *this; }
index b4b1fadd1b131c62be3cd6db4902a2f9ebd48e60..228a34a582d962ca90e311760e675486ac83ba5b 100644 (file)
@@ -80,7 +80,7 @@ void AliHLTMUONCoreMansoTracker::AliRegionOfInterest::Create(AliHLTMUONCorePoint
 
        fCentre = p;
        // Compute the radius Rp
-       Float Rp = (Float) sqrt( p.fX * p.fX + p.fY * p.fY );
+       Float rp = (Float) sqrt( p.fX * p.fX + p.fY * p.fY );
 
        // The radius Rs for the region of interest is computed from the
        // specification given in the document:
@@ -89,7 +89,7 @@ void AliHLTMUONCoreMansoTracker::AliRegionOfInterest::Create(AliHLTMUONCorePoint
        //   equation:
        //     Rs = a * Rp + b
        //   given on page 3 section 4.
-       fRs = a * Rp + b;
+       fRs = a * rp + b;
 }