Cleanup of data arrays
authorcblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 20 Nov 2000 08:56:07 +0000 (08:56 +0000)
committercblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 20 Nov 2000 08:56:07 +0000 (08:56 +0000)
TRD/AliTRDarrayF.cxx
TRD/AliTRDarrayI.cxx
TRD/AliTRDdataArray.cxx
TRD/AliTRDdataArrayF.cxx
TRD/AliTRDdataArrayI.cxx
TRD/AliTRDsegmentArrayBase.cxx

index e46a4cb..9f1ebf0 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.4  2000/06/09 11:10:07  cblume
+Compiler warnings and coding conventions, next round
+
 Revision 1.3  2000/06/08 18:32:58  cblume
 Make code compliant to coding conventions
 
@@ -42,6 +45,15 @@ Add float array
 ClassImp(AliTRDarrayF)
 
 //_____________________________________________________________________________
+AliTRDarrayF::AliTRDarrayF():TArrayF()
+{
+  //
+  // Default constructor
+  //
+
+}
+
+//_____________________________________________________________________________
 AliTRDarrayF::~AliTRDarrayF()
 {
   //
index 71af135..69d56c2 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.5  2000/11/01 14:53:20  cblume
+Merge with TRD-develop
+
 
 Revision 1.1.4.3  2000/10/06 16:49:46  cblume
 Made Getters const
@@ -67,6 +70,15 @@ Add new TRD classes
 ClassImp(AliTRDarrayI)
 
 //_____________________________________________________________________________
+AliTRDarrayI::AliTRDarrayI():TArrayI()
+{
+  //
+  // Default constructor
+  //
+
+}
+
+//_____________________________________________________________________________
 AliTRDarrayI::~AliTRDarrayI()
 {
   //
index e7002b0..89f6f95 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.7  2000/11/01 14:53:20  cblume
+Merge with TRD-develop
+
 Revision 1.1.4.3  2000/10/06 16:49:46  cblume
 Made Getters const
 
@@ -90,6 +93,8 @@ AliTRDdataArray::AliTRDdataArray(Int_t nrow, Int_t ncol, Int_t ntime)
   // The row- and column dimensions are compressible.
   //
 
+  fIndex   = 0;
+
   Allocate(nrow,ncol,ntime);
 
 }
@@ -112,7 +117,7 @@ AliTRDdataArray::~AliTRDdataArray()
   // AliTRDdataArray destructor
   //
 
-  if (fIndex) fIndex->Delete();
+  if (fIndex) delete fIndex;
   
 }
 
@@ -187,7 +192,7 @@ void AliTRDdataArray::Allocate(Int_t nrow, Int_t ncol,Int_t ntime)
   fNtime  = ntime;
 
   if (fIndex) delete fIndex;
-  fIndex = new AliTRDarrayI;
+  fIndex = new AliTRDarrayI();
   fIndex->Set(fNdim2);
   for (Int_t i = 0, k = 0; i < fNdim2; i++, k += fNdim1) { 
     (*fIndex)[i] = k;
@@ -237,7 +242,7 @@ void AliTRDdataArray::Reset()
   //
 
   if (fIndex) delete fIndex;
-  fIndex = new AliTRDarrayI;
+  fIndex = new AliTRDarrayI();
   fIndex->Set(0); 
 
   fNdim1   = -1;
index 96630e4..fbdee9b 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.6  2000/11/01 14:53:20  cblume
+Merge with TRD-develop
+
 Revision 1.1.2.3  2000/10/06 16:49:46  cblume
 Made Getters const
 
@@ -71,6 +74,8 @@ AliTRDdataArrayF::AliTRDdataArrayF(Int_t nrow, Int_t ncol, Int_t ntime)
   // The row- and column dimensions are compressible.
   //
 
+  fElements = 0;
+
   Allocate(nrow,ncol,ntime);
   
 }
@@ -93,8 +98,7 @@ AliTRDdataArrayF::~AliTRDdataArrayF()
   // Destructor
   //
 
-  if (fElements) fElements->Delete();
-  delete fElements;
+  if (fElements) delete fElements;
   
 }
 
@@ -107,11 +111,10 @@ void AliTRDdataArrayF::Allocate(Int_t nrow, Int_t ncol, Int_t ntime)
   // The row- and column dimensions are compressible.
   //
 
-
   if (fNelems < 0) AliTRDdataArray::Allocate(nrow,ncol,ntime);
 
   if (fElements) delete fElements;
-  fElements = new AliTRDarrayF;
+  fElements = new AliTRDarrayF();
   fElements->Set(fNelems);
 
 }
@@ -139,7 +142,7 @@ void AliTRDdataArrayF::Reset()
   //
   
   if (fElements) delete fElements;
-  fElements = new AliTRDarrayF;
+  fElements = new AliTRDarrayF();
   fElements->Set(0); 
 
   AliTRDdataArray::Reset();
@@ -156,7 +159,7 @@ Int_t AliTRDdataArrayF::GetSize()
   Int_t size = sizeof(this);
 
   if (fIndex)    size += sizeof(fIndex)    
-                         + fIndex->GetSize()    * sizeof(Float_t);
+                         + fIndex->GetSize()    * sizeof(Int_t);
   if (fElements) size += sizeof(fElements) 
                          + fElements->GetSize() * sizeof(Float_t);
 
@@ -341,7 +344,8 @@ void AliTRDdataArrayF::Expand1()
 
     // Negative sign counts the unwritten values (under threshold)
     if ((*fElements)[i] < 0) {
-      idx1 -= (Int_t) fElements->At(i);
+      //idx1 -= (Int_t) fElements->At(i);
+      idx1 -= TMath::Nint(fElements->At(i));
     } 
     else {
       buf[(*fIndex)[idx2] + idx1] = fElements->At(i);
@@ -370,17 +374,22 @@ void AliTRDdataArrayF::Compress1()
   // Compress a buffer of type 1
   //
 
-  AliTRDarrayF buf;  
-  buf.Set(fNelems);
-  AliTRDarrayI index;
-  index.Set(fNdim2);
+  //AliTRDarrayF buf;  
+  //buf.Set(fNelems);
+  //AliTRDarrayI index;
+  //index.Set(fNdim2);
+  AliTRDarrayF *buf   = new AliTRDarrayF();  
+  buf->Set(fNelems);
+  AliTRDarrayI *index = new AliTRDarrayI();
+  index->Set(fNdim2);
 
   Int_t icurrent = -1;
   Int_t izero;
   for (Int_t idx2 = 0; idx2 < fNdim2; idx2++){      
 
     // Set the idx2 pointer
-    index[idx2] = icurrent + 1;
+    //index[idx2] = icurrent + 1;
+    (*index)[idx2] = icurrent + 1;
 
     // Reset the zero counter 
     izero = 0;  
@@ -394,31 +403,44 @@ void AliTRDdataArrayF::Compress1()
        if (izero > 0) {
          // If we have currently izero counts under threshold
          icurrent++;     
-         if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+         //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+         if (icurrent >= buf->fN) buf->Expand(icurrent*2);
           // Store the number of entries below zero
-         buf[icurrent] = -izero;  
+         //buf[icurrent] = -izero;  
+         (*buf)[icurrent] = -izero;  
          izero = 0;
        } 
        icurrent++;
-       if (icurrent >= buf.fN) buf.Expand(icurrent*2);
-       buf[icurrent] = GetDataFast(idx1,idx2);     
+       //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+       if (icurrent >= buf->fN) buf->Expand(icurrent*2);
+       //buf[icurrent] = GetDataFast(idx1,idx2);           
+       (*buf)[icurrent] = GetDataFast(idx1,idx2);          
       } // If signal larger than threshold             
     } // End of loop over idx1
 
     if (izero > 0) {
       icurrent++;        
-      if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+      //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+      if (icurrent >= buf->fN) buf->Expand(icurrent*2);
       // Store the number of entries below zero
-      buf[icurrent] = -izero; 
+      //buf[icurrent] = -izero;  
+      (*buf)[icurrent] = -izero;  
     }
 
   }
 
-  buf.Expand(icurrent+1);
-  (*fElements) = buf;
+  //buf.Expand(icurrent+1);
+  //(*fElements) = buf;
+  //fNelems   = fElements->fN;
+  //fBufType  = 1;
+  //(*fIndex) = index;
+  buf->Expand(icurrent+1);
+  if (fElements) delete fElements;
+  fElements = buf;
   fNelems   = fElements->fN;
   fBufType  = 1;
-  (*fIndex) = index;
+  if (fIndex) delete fIndex;
+  fIndex    = index;
 
 }
 
@@ -443,7 +465,8 @@ void AliTRDdataArrayF::Expand2()
   for (i = 0; i < n; i++){
     // Negative sign counts the unwritten values (under threshold)
     if ((*fElements)[i] < 0) {
-      idx1 -= (Int_t) fElements->At(i); 
+      //idx1 -= (Int_t) fElements->At(i); 
+      idx1 -= TMath::Nint(fElements->At(i)); 
     }
     else {
       buf[(*fIndex)[idx2]+idx1] = fElements->At(i);
@@ -535,7 +558,8 @@ Bool_t AliTRDdataArrayF::First1()
   Int_t i;
   for (i = 0; i < fNelems; i++){
     if (fElements->At(i) < 0) {
-      fCurrentIdx1 -= (Int_t) fElements->At(i);
+      //fCurrentIdx1 -= (Int_t) fElements->At(i);
+      fCurrentIdx1 -= TMath::Nint(fElements->At(i));
     }
     else {     
       fCurrentIdx1++;
@@ -567,7 +591,8 @@ Bool_t AliTRDdataArrayF::Next1()
   Int_t i;
   for (i = fCurrentIndex + 1; i < fNelems; i++){
     if (fElements->At(i) < 0) {
-      fCurrentIdx1 -= (Int_t) fElements->At(i);
+      //fCurrentIdx1 -= (Int_t) fElements->At(i);
+      fCurrentIdx1 -= TMath::Nint(fElements->At(i));
     }
     else {      
       fCurrentIdx1++;
@@ -608,7 +633,8 @@ Float_t AliTRDdataArrayF::GetData1(Int_t idx1, Int_t idx2) const
  
   for (i = fIndex->At(idx2); ((i < n2) && (curidx1 < idx1)); i++){
     if (fElements->At(i) < 0) {
-      curidx1 -= (Int_t) fElements->At(i);
+      //curidx1 -= (Int_t) fElements->At(i);
+      curidx1 -= TMath::Nint(fElements->At(i));
     }
     else {      
       curidx1++;
index 35887f5..ec418d9 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.6  2000/11/01 14:53:20  cblume
+Merge with TRD-develop
+
 Revision 1.1.2.3  2000/10/06 16:49:46  cblume
 Made Getters const
 
@@ -70,6 +73,8 @@ AliTRDdataArrayI::AliTRDdataArrayI(Int_t nrow, Int_t ncol, Int_t ntime)
   // The row- and column dimensions are compressible.
   //
 
+  fElements = 0;
+
   Allocate(nrow,ncol,ntime);
   
 }
@@ -92,8 +97,7 @@ AliTRDdataArrayI::~AliTRDdataArrayI()
   // Destructor
   //
 
-  if (fElements) fElements->Delete();
-  delete fElements;
+  if (fElements) delete fElements;
   
 }
 
@@ -109,7 +113,7 @@ void AliTRDdataArrayI::Allocate(Int_t nrow, Int_t ncol, Int_t ntime)
   if (fNelems < 0) AliTRDdataArray::Allocate(nrow,ncol,ntime);
 
   if (fElements) delete fElements;
-  fElements = new AliTRDarrayI;
+  fElements = new AliTRDarrayI();
   fElements->Set(fNelems);
 
 }
@@ -137,7 +141,7 @@ void AliTRDdataArrayI::Reset()
   //
 
   if (fElements) delete fElements;
-  fElements = new AliTRDarrayI;
+  fElements = new AliTRDarrayI();
   fElements->Set(0); 
 
   AliTRDdataArray::Reset();
@@ -368,17 +372,22 @@ void AliTRDdataArrayI::Compress1()
   // Compress a buffer of type 1
   //
 
-  AliTRDarrayI  buf;  
-  buf.Set(fNelems);
-  AliTRDarrayI  index;
-  index.Set(fNdim2);
+  //AliTRDarrayI  buf;  
+  //buf.Set(fNelems);
+  //AliTRDarrayI  index;
+  //index.Set(fNdim2);
+  AliTRDarrayI *buf   = new AliTRDarrayI();  
+  buf->Set(fNelems);
+  AliTRDarrayI *index = new AliTRDarrayI();
+  index->Set(fNdim2);
 
   Int_t icurrent = -1;
   Int_t izero;
   for (Int_t idx2 = 0; idx2 < fNdim2; idx2++){      
 
     // Set the idx2 pointer
-    index[idx2] = icurrent + 1;
+    //index[idx2] = icurrent + 1;
+    (*index)[idx2] = icurrent + 1;
 
     // Reset the zero counter 
     izero = 0;  
@@ -392,31 +401,44 @@ void AliTRDdataArrayI::Compress1()
        if (izero > 0) {
          // If we have currently izero counts under threshold
          icurrent++;     
-         if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+         //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+         if (icurrent >= buf->fN) buf->Expand(icurrent*2);
           // Store the number of entries below zero
-         buf[icurrent] = -izero;  
+         //buf[icurrent] = -izero;  
+         (*buf)[icurrent] = -izero;  
          izero = 0;
        } 
        icurrent++;
-       if (icurrent >= buf.fN) buf.Expand(icurrent*2);
-       buf[icurrent] = GetDataFast(idx1,idx2);     
+       //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+       if (icurrent >= buf->fN) buf->Expand(icurrent*2);
+       //buf[icurrent] = GetDataFast(idx1,idx2);           
+       (*buf)[icurrent] = GetDataFast(idx1,idx2);          
       } // If signal larger than threshold             
     } // End of loop over idx1
 
     if (izero > 0) {
       icurrent++;        
-      if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+      //if (icurrent >= buf.fN) buf.Expand(icurrent*2);
+      if (icurrent >= buf->fN) buf->Expand(icurrent*2);
       // Store the number of entries below zero
-      buf[icurrent] = -izero; 
+      //buf[icurrent] = -izero; 
+      (*buf)[icurrent] = -izero; 
     }
 
   }
 
-  buf.Expand(icurrent+1);
-  (*fElements) = buf;
+  //buf.Expand(icurrent+1);
+  //(*fElements) = buf;
+  //fNelems   = fElements->fN;
+  //fBufType  = 1;
+  //(*fIndex) = index;
+  buf->Expand(icurrent+1);
+  if (fElements) delete fElements;
+  fElements = buf;
   fNelems   = fElements->fN;
   fBufType  = 1;
-  (*fIndex) = index;
+  if (fIndex) delete fIndex;
+  fIndex    = index;
 
 }
 
index ca55dfc..19ceb02 100644 (file)
@@ -15,6 +15,9 @@
 
 /*
 $Log$
+Revision 1.6  2000/11/01 14:53:21  cblume
+Merge with TRD-develop
+
 Revision 1.1.4.3  2000/10/06 16:49:46  cblume
 Made Getters const
 
@@ -277,7 +280,7 @@ Bool_t AliTRDsegmentArrayBase::MakeArray(Int_t n)
   if (fTreeIndex) delete fTreeIndex;  
 
   fSegment   = new TObjArray(n);
-  fTreeIndex = new AliTRDarrayI;
+  fTreeIndex = new AliTRDarrayI();
   fTreeIndex->Set(n);
   fNSegment  = n;
   if ((fSegment) && (fTreeIndex))