Corrected assignment operator

[u/mrichter/AliRoot.git] / MUON / AliMUONHitMapA1.cxx

diff --git a/MUON/AliMUONHitMapA1.cxx b/MUON/AliMUONHitMapA1.cxx
index 1c874e244b4caba15ece797caf2c8850f7be8066..9573067de43d4e61b378e49ae4c634c659e82d2d 100644 (file)
--- a/MUON/AliMUONHitMapA1.cxx
+++ b/MUON/AliMUONHitMapA1.cxx
@@ -13,35 +13,7 @@
  * provided "as is" without express or implied warranty.                  *
  **************************************************************************/
 
-/*
-$Log$
-Revision 1.7  2001/01/25 11:43:48  morsch
-Add a "real" default constructor.
-
-Revision 1.6  2000/10/06 09:05:42  morsch
-Correction on upper limits for x- and y- coordinates to make code work with slat chambers.
-
-Revision 1.5  2000/10/02 21:28:09  fca
-Removal of useless dependecies via forward declarations
-
-Revision 1.4  2000/07/13 16:19:44  fca
-Mainly coding conventions + some small bug fixes
-
-Revision 1.3  2000/07/03 11:54:57  morsch
-AliMUONSegmentation and AliMUONHitMap have been replaced by AliSegmentation and AliHitMap in STEER
-The methods GetPadIxy and GetPadXxy of AliMUONSegmentation have changed name to GetPadI and GetPadC.
-
-Revision 1.2  2000/06/15 07:58:48  morsch
-Code from MUON-dev joined
-
-Revision 1.1.2.2  2000/06/12 07:58:06  morsch
-include TMath.h
-
-Revision 1.1.2.1  2000/06/09 22:01:09  morsch
-Code from AliMUONHitMap.h
-Most coding rule violations corrected.
-
-*/
+/* $Id$ */
 
 #include "AliMUONHitMapA1.h"
 #include "AliSegmentation.h"
@@ -56,31 +28,27 @@ ClassImp(AliMUONHitMapA1)
     AliMUONHitMapA1::AliMUONHitMapA1()
 {
     // Default constructor
-    fSegmentation = 0;
     fNpx          = 0;
     fNpy          = 0;
     fMaxIndex     = 0;
     
     fHitMap       = 0;
     fDigits       = 0;
-    fNdigits      = 0;
 }
 
 AliMUONHitMapA1::AliMUONHitMapA1(AliSegmentation *seg, TObjArray *dig)
 {
 // Constructor
-    fSegmentation = seg;
-    fNpx  = fSegmentation->Npx()+1;
-    fNpy  = fSegmentation->Npy()+1;
+    fNpx  = seg->Npx()+1;
+    fNpy  = seg->Npy()+1;
     fMaxIndex=2*(fNpx+1)*2*(fNpy+1)+2*fNpy;
     
     fHitMap = new Int_t[fMaxIndex];
     fDigits =  dig;
-    fNdigits = fDigits->GetEntriesFast();
     Clear();
 }
 
-AliMUONHitMapA1::AliMUONHitMapA1(const AliMUONHitMapA1 & hitMap)
+AliMUONHitMapA1::AliMUONHitMapA1(const AliMUONHitMapA1 & hitMap):AliHitMap(hitMap)
 {
 // Dummy copy constructor
     ;
@@ -90,7 +58,6 @@ AliMUONHitMapA1::AliMUONHitMapA1(const AliMUONHitMapA1 & hitMap)
 AliMUONHitMapA1::~AliMUONHitMapA1()
 {
 // Destructor
-//    if (fDigits) delete   fDigits;
     if (fHitMap) delete[] fHitMap;
 }
 
@@ -100,11 +67,21 @@ void AliMUONHitMapA1::Clear(const char *)
     memset(fHitMap,0,sizeof(int)*fMaxIndex);
 }
 
+Bool_t AliMUONHitMapA1::ValidateHit(Int_t ix, Int_t iy)
+{
+    //
+    // Check if pad coordinates are within boundaries
+    //
+//    printf("\n Validate %d %d %d %d", ix, iy, fNpx, fNpy);
+    
+    return (TMath::Abs(ix) <= fNpx && TMath::Abs(iy) <= fNpy); 
+}
+
 Int_t AliMUONHitMapA1::CheckedIndex(Int_t ix, Int_t iy) const
 {
 // Return checked indices ix, iy
     Int_t index=2*fNpy*(ix+fNpx)+(iy+fNpy);
-    if (index > fMaxIndex) {
+    if (index >= fMaxIndex) {
        printf("\n \n \n Try to read/write outside array !!!! \n \n %d %d %d %d %d %d",
               ix,iy, fMaxIndex, index, fNpx, fNpy);
        return  fMaxIndex-1;
@@ -122,7 +99,7 @@ void  AliMUONHitMapA1::FillHits()
     //printf("FindRawClusters -- ndigits %d \n",ndigits);
     if (!ndigits) return;
     AliMUONDigit *dig;
-    for (Int_t ndig=0; ndig<fNdigits; ndig++) {
+    for (Int_t ndig=0; ndig<ndigits; ndig++) {
        dig = (AliMUONDigit*)fDigits->UncheckedAt(ndig);
        SetHit(dig->PadX(),dig->PadY(),ndig);
     }
@@ -178,7 +155,7 @@ FlagType AliMUONHitMapA1::TestHit(Int_t ix, Int_t iy)
     }
 }
 
-AliMUONHitMapA1 & AliMUONHitMapA1::operator = (const AliMUONHitMapA1 & rhs) 
+AliMUONHitMapA1 & AliMUONHitMapA1::operator = (const AliMUONHitMapA1 & /*rhs*/) 
 {
 // Dummy assignment operator
     return *this;