Coding conventions
authorhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 21 Nov 2007 13:52:16 +0000 (13:52 +0000)
committerhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 21 Nov 2007 13:52:16 +0000 (13:52 +0000)
STAT/TKDPDF.cxx
STAT/TKDTree.cxx
STAT/TKDTree.h

index 9bc9e88..c8fa07e 100644 (file)
@@ -66,7 +66,7 @@ TKDPDF::TKDPDF(TTree *t, const Char_t *var, const Char_t *cut, UInt_t bsize, Lon
                        //Info("TKDPDF(TTree*, const Char_t, const Char_t, UInt_t)", Form("Allocating %d data points in %d dimensions.", fNpoints, fNDim));
                        fData = new Float_t*[fNDim];
                        for(int idim=0; idim<fNDim; idim++) fData[idim] = new Float_t[fNpoints];
-                       kDataOwner = kTRUE;
+                       fDataOwner = kTRUE;
                }
                v = t->GetV1();
                for(int ip=0; ip<fNpoints; ip++) fData[idim][ip] = (Float_t)v[ip];
index 775c602..e675392 100644 (file)
@@ -9,7 +9,7 @@ templateClassImp(TKDTree)
 //////////////////////////////////////////////////////////////////////
 // Memory setup of protected data members:
 // 
-//     kDataOwner;  // Toggle ownership of the data
+//     fDataOwner;  // Toggle ownership of the data
 //     fNnodes:
 //  size of branch node array, and index of first terminal node
 //     fNDim;       // number of variables
@@ -35,7 +35,7 @@ templateClassImp(TKDTree)
 template <typename  Index, typename Value>
 TKDTree<Index, Value>::TKDTree() :
        TObject()
-       ,kDataOwner(kFALSE)
+       ,fDataOwner(kFALSE)
        ,fNnodes(0)
        ,fNDim(0)
        ,fNDimm(0)
@@ -64,7 +64,7 @@ TKDTree<Index, Value>::TKDTree() :
 template <typename  Index, typename Value>
 TKDTree<Index, Value>::TKDTree(Index npoints, Index ndim, UInt_t bsize, Value **data) :
        TObject()
-       ,kDataOwner(kFALSE)
+       ,fDataOwner(kFALSE)
        ,fNnodes(0)
        ,fNDim(ndim)
        ,fNDimm(2*ndim)
@@ -94,6 +94,7 @@ TKDTree<Index, Value>::TKDTree(Index npoints, Index ndim, UInt_t bsize, Value **
 template <typename  Index, typename Value>
 TKDTree<Index, Value>::~TKDTree()
 {
+  // Destructor
        if (fIndBuffer) delete [] fIndBuffer;
        if (fDistBuffer) delete [] fDistBuffer;
        if (fkNNdist) delete [] fkNNdist;
@@ -103,7 +104,7 @@ TKDTree<Index, Value>::~TKDTree()
        if (fIndPoints) delete [] fIndPoints;
        if (fRange) delete [] fRange;
        if (fBoundaries) delete [] fBoundaries;
-       if (kDataOwner && fData){
+       if (fDataOwner && fData){
                for(int idim=0; idim<fNDim; idim++) delete [] fData[idim];
                delete [] fData;
        }
@@ -674,7 +675,7 @@ void TKDTree<Index, Value>::SetData(Index npoints, Index ndim, UInt_t bsize, Val
 
 //_________________________________________________________________
 template <typename  Index, typename Value>
-void TKDTree<Index, Value>::Spread(Index ntotal, Value *a, Index *index, Value &min, Value &max)
+void TKDTree<Index, Value>::Spread(Index ntotal, Value *a, Index *index, Value &min, Value &max) const
 {
   //Value min, max;
   Index i;
@@ -689,7 +690,7 @@ void TKDTree<Index, Value>::Spread(Index ntotal, Value *a, Index *index, Value &
 
 //_________________________________________________________________
 template <typename  Index, typename Value>
-Value TKDTree<Index, Value>::KOrdStat(Index ntotal, Value *a, Index k, Index *index)
+Value TKDTree<Index, Value>::KOrdStat(Index ntotal, Value *a, Index k, Index *index) const
 {
   //
    //copy of the TMath::KOrdStat because I need an Index work array
index 7d35d32..04dcb2f 100644 (file)
@@ -20,29 +20,29 @@ public:
        ~TKDTree();
        
        // getters
-       inline  Index*  GetPointsIndexes(Int_t node) const {
+       Index*  GetPointsIndexes(Int_t node) const {
                if(node < fNnodes) return 0x0;
                Int_t offset = (node >= fCrossNode) ? (node-fCrossNode)*fBucketSize : fOffset+(node-fNnodes)*fBucketSize;
                return &fIndPoints[offset];
        }
-       inline UChar_t GetNodeAxis(Int_t id) const {return (id < 0 || id >= fNnodes) ? 0 : fAxis[id];}
-       inline Value   GetNodeValue(Int_t id) const {return (id < 0 || id >= fNnodes) ? 0 : fValue[id];}
-       inline Int_t   GetNNodes() const {return fNnodes;}
-       inline Value*  GetBoundaries();
-       inline Value*  GetBoundary(const Int_t node);
+       UChar_t GetNodeAxis(Int_t id) const {return (id < 0 || id >= fNnodes) ? 0 : fAxis[id];}
+       Value   GetNodeValue(Int_t id) const {return (id < 0 || id >= fNnodes) ? 0 : fValue[id];}
+       Int_t   GetNNodes() const {return fNnodes;}
+       Value*  GetBoundaries();
+       Value*  GetBoundary(const Int_t node);
        static  Int_t   GetIndex(Int_t row, Int_t collumn){return collumn+(1<<row);}
-       static inline void GetCoord(Int_t index, Int_t &row, Int_t &collumn){for (row=0; index>=(16<<row);row+=4); for (; index>=(2<<row);row++);collumn= index-(1<<row);};
+       static void GetCoord(Int_t index, Int_t &row, Int_t &collumn){for (row=0; index>=(16<<row);row+=4); for (; index>=(2<<row);row++);collumn= index-(1<<row);};
                Bool_t  FindNearestNeighbors(const Value *point, const Int_t kNN, Index *&i, Value *&d);
                Index   FindNode(const Value * point);
                void    FindPoint(Value * point, Index &index, Int_t &iter);
                void    FindInRangeA(Value * point, Value * delta, Index *res , Index &npoints,Index & iter, Int_t bnode);
                void    FindInRangeB(Value * point, Value * delta, Index *res , Index &npoints,Index & iter, Int_t bnode);
-       inline void    FindBNodeA(Value * point, Value * delta, Int_t &inode);
-       inline Bool_t  IsTerminal(Index inode){return (inode>=fNnodes);}
-       Value           KOrdStat(Index ntotal, Value *a, Index k, Index *index);
+       void    FindBNodeA(Value * point, Value * delta, Int_t &inode);
+       Bool_t  IsTerminal(Index inode) const {return (inode>=fNnodes);}
+       Value           KOrdStat(Index ntotal, Value *a, Index k, Index *index) const;
        void            MakeBoundaries(Value *range = 0x0);
        void            SetData(Index npoints, Index ndim, UInt_t bsize, Value **data);
-       void            Spread(Index ntotal, Value *a, Index *index, Value &min, Value &max);
+       void            Spread(Index ntotal, Value *a, Index *index, Value &min, Value &max) const;
 
 protected:
        void            Build();  // build tree
@@ -54,7 +54,7 @@ private:
 
 
 protected:
-       Bool_t  kDataOwner;  //! Toggle ownership of the data
+       Bool_t  fDataOwner;  //! Toggle ownership of the data
        Int_t   fNnodes;     // size of node array
        Index   fNDim;       // number of dimensions
        Index   fNDimm;      // dummy 2*fNDim
@@ -104,6 +104,7 @@ void TKDTree<Index, Value>::FindBNodeA(Value *point, Value *delta, Int_t &inode)
 template <typename  Index, typename Value>
 Value* TKDTree<Index, Value>::GetBoundaries()
 {
+  // Get the boundaries
        if(!fBoundaries) MakeBoundaries();
        return fBoundaries;
 }
@@ -112,6 +113,7 @@ Value* TKDTree<Index, Value>::GetBoundaries()
 template <typename  Index, typename Value>
 Value* TKDTree<Index, Value>::GetBoundary(const Int_t node)
 {
+  // Get a boundary
        if(!fBoundaries) MakeBoundaries();
        return &fBoundaries[node*2*fNDim];
 }