Fix coding rule violations
authorcblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 16 Dec 2009 08:27:10 +0000 (08:27 +0000)
committercblume <cblume@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 16 Dec 2009 08:27:10 +0000 (08:27 +0000)
TRD/AliTRDmcmSim.cxx
TRD/AliTRDmcmSim.h
TRD/AliTRDtrapConfig.cxx
TRD/AliTRDtrapConfig.h

index 9841912..ddb1044 100644 (file)
@@ -1881,7 +1881,7 @@ void AliTRDmcmSim::Sort6To2Worst(UShort_t  idx1i, UShort_t  idx2i, UShort_t  idx
 
 // ----- I/O implementation -----
 
-ostream& AliTRDmcmSim::text(ostream& os)
+ostream& AliTRDmcmSim::Text(ostream& os)
 {
   // manipulator to activate output in text format (default)
 
@@ -1889,7 +1889,7 @@ ostream& AliTRDmcmSim::text(ostream& os)
   return os;
 }
 
-ostream& AliTRDmcmSim::cfdat(ostream& os)
+ostream& AliTRDmcmSim::Cfdat(ostream& os)
 {
   // manipulator to activate output in CFDAT format 
   // to send to the FEE via SCSN
@@ -1898,7 +1898,7 @@ ostream& AliTRDmcmSim::cfdat(ostream& os)
   return os;
 }
 
-ostream& AliTRDmcmSim::raw(ostream& os)
+ostream& AliTRDmcmSim::Raw(ostream& os)
 {
   // manipulator to activate output as raw data dump
 
index 4964c71..6dd5890 100644 (file)
@@ -111,9 +111,9 @@ class AliTRDmcmSim : public TObject {
          void      Draw(Option_t* const option ="");         // draw data (ADC data, hits and tracklets)
 
   friend  std::ostream& operator<<(std::ostream &os, const AliTRDmcmSim &mcm); // data output using ostream (e.g. cout << mcm;)
-  static  ostream&  cfdat(ostream &os);                       // manipulator to activate cfdat output
-  static  ostream&  raw  (ostream &os);                       // manipulator to activate raw output
-  static  ostream&  text (ostream &os);                       // manipulator to activate text output
+  static  ostream&  Cfdat(ostream &os);                       // manipulator to activate cfdat output
+  static  ostream&  Raw  (ostream &os);                       // manipulator to activate raw output
+  static  ostream&  Text (ostream &os);                       // manipulator to activate text output
 
  protected:
          Bool_t    CheckInitialized() const;           // Check whether the class is initialized
index dbb23ef..9373343 100644 (file)
@@ -692,6 +692,8 @@ Bool_t AliTRDtrapConfig::Poke(Int_t addr, Int_t value, Int_t /* det */, Int_t /*
 
 Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value)
 {
+  // Set the content of the given DMEM address 
+
   if ( (addr >> 14) != 0x3) {
     AliError(Form("No DMEM address: 0x%08x", addr));
     return kFALSE;
@@ -711,6 +713,8 @@ Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value)
 
 Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value, Int_t det)
 {
+  // Set the content of the given DMEM address 
+
   for (Int_t iROB = 0; iROB < AliTRDfeeParam::GetNrobC1(); iROB++) {
     for (Int_t iMCM = 0; iMCM < fgkMaxMcm; iMCM++) {
       fDmem[det*AliTRDfeeParam::GetNrobC1()*fgkMaxMcm + iROB*fgkMaxMcm + iMCM]
@@ -723,6 +727,8 @@ Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value, Int_t det)
 
 Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value, Int_t det, Int_t rob, Int_t mcm)
 {
+  // Set the content of the given DMEM address 
+
   fDmem[det*AliTRDfeeParam::GetNrobC1()*fgkMaxMcm + rob*fgkMaxMcm + mcm]
     [addr - fgkDmemStartAddress] = value;
   return kTRUE;
@@ -731,6 +737,8 @@ Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, Int_t value, Int_t det, Int_t rob,
 
 Int_t AliTRDtrapConfig::GetDmem(Int_t addr, Int_t det, Int_t rob, Int_t mcm)
 {
+  // Set the content of the given DMEM address 
+
   return fDmem[det*AliTRDfeeParam::GetNrobC1()*fgkMaxMcm + rob*fgkMaxMcm + mcm]
     [addr - fgkDmemStartAddress];
 }
@@ -861,12 +869,17 @@ Bool_t  AliTRDtrapConfig::LoadConfig(Int_t det, TString filename)
 
 Bool_t AliTRDtrapConfig::ReadPackedConfig(UInt_t *data, Int_t size) 
 {
+  // Read the packed configuration from the passed memory block
+  //
+  // To be used to retrieve the TRAP configuration from the 
+  // configuration as sent in the raw data. 
+
   Int_t idx = 0;
   Int_t err = 0;
-  Int_t step, bwidth, nwords, exit_flag, bitcnt;
+  Int_t step, bwidth, nwords, exitFlag, bitcnt;
   
   UShort_t caddr;
-  UInt_t dat, msk, header, data_hi;
+  UInt_t dat, msk, header, dataHi;
   
   while (idx < size) {
     
@@ -885,12 +898,12 @@ Bool_t AliTRDtrapConfig::ReadPackedConfig(UInt_t *data, Int_t size)
       {
         if (header & 0x02) // check if > 16 bits                                                                                                                                                              
         {
-          data_hi = *data;
-          AliDebug(5, Form("read: 0x%08x", data_hi));
+          dataHi = *data;
+          AliDebug(5, Form("read: 0x%08x", dataHi));
           data++;
           idx++;
-          err += ((data_hi ^ (dat | 1)) & 0xFFFF) != 0;
-          dat = (data_hi & 0xFFFF0000) | dat;
+          err += ((dataHi ^ (dat | 1)) & 0xFFFF) != 0;
+          dat = (dataHi & 0xFFFF0000) | dat;
         }
         AliDebug(5, Form("addr=0x%04x (%s) data=0x%08x\n", caddr, GetRegName(GetRegByAddress(caddr)), dat));
         if ( ! Poke(caddr, dat, 0, 0, 0) )
@@ -914,9 +927,9 @@ Bool_t AliTRDtrapConfig::ReadPackedConfig(UInt_t *data, Int_t size)
       bwidth = ((header >>  3) & 0x001F) + 1;
       nwords =  (header >>  8) & 0x00FF;
       caddr  =  (header >> 16) & 0xFFFF;
-      exit_flag = (step == 0) || (step == 3) || (nwords == 0);
+      exitFlag = (step == 0) || (step == 3) || (nwords == 0);
       
-      if (exit_flag) 
+      if (exitFlag) 
         return err;
       
       switch (bwidth)
index a475310..5dca27c 100644 (file)
@@ -11,7 +11,6 @@
 
 #include <TObject.h>
 #include <TString.h>
-#include <TClonesArray.h>
 
 class AliTRDtrapConfig : public TObject
 {
@@ -531,8 +530,8 @@ class AliTRDtrapConfig : public TObject
   // DMEM
   static const Int_t fgkDmemStartAddress; // = 0xc000;  // start address in TRAP GIO
   static const Int_t fgkDmemWords = 0x400;          // number of words in DMEM
-  UInt_t fDmem[540*8*18][fgkDmemWords];
-  Bool_t fDmemValid[540*8*18][fgkDmemWords];
+  UInt_t fDmem[540*8*18][fgkDmemWords]; // DMEM storage
+  Bool_t fDmemValid[540*8*18][fgkDmemWords]; // DMEM valid flag storage
 
   AliTRDtrapConfig(); // private constructor due to singleton implementation