From: mtadel <mtadel@f7af4fe6-9843-0410-8265-dc069ae4e863>
Date: Tue, 17 Feb 2009 10:47:17 +0000 (+0000)
Subject: Fix some coding convention violations.
X-Git-Url: http://git.uio.no/git/?a=commitdiff_plain;h=ca8fb7ae92bb129d6d49ba33f64fc5b22946a3b1;p=u%2Fmrichter%2FAliRoot.git

Fix some coding convention violations.
---

diff --git a/EVE/EveHLT/AliEveHOMERSourceMap.cxx b/EVE/EveHLT/AliEveHOMERSourceMap.cxx
index c4ba32d6a1e..98f7704044f 100644
--- a/EVE/EveHLT/AliEveHOMERSourceMap.cxx
+++ b/EVE/EveHLT/AliEveHOMERSourceMap.cxx
@@ -15,17 +15,26 @@
 //______________________________________________________________________
 // AliEveHOMERSourceMap
 //
+// AliEveHOMERSourceMap is an abstract container for HLT HOMER sources,
+// see AliHLTHOMERSourceDesc.
+//
+// The concrete implementations AliEveHOMERSourceMapByDet and
+// AliEveHOMERSourceMapByType allow retrieval of HOMER sources in proper
+// order as required for their display in EVE object browser.
+// 
 
 ClassImp(AliEveHOMERSourceMap)
 
 AliEveHOMERSourceMap::AliEveHOMERSourceMap(ESourceGrouping_e grouping) :
   fGrouping(grouping)
 {
-
+  // Constructor.
 }
 
 AliEveHOMERSourceMap* AliEveHOMERSourceMap::Create(ESourceGrouping_e grouping)
 {
+  // Static constructor - instantiates appropriate sub-class.
+
   switch (grouping)
   {
     case kSG_ByDet:  return new AliEveHOMERSourceMapByDet(grouping);
@@ -36,6 +45,9 @@ AliEveHOMERSourceMap* AliEveHOMERSourceMap::Create(ESourceGrouping_e grouping)
 
 Int_t AliEveHOMERSourceMap::iterator::level()
 {
+  // Returns the depth in iteration:
+  // Det / Sub-Det / Sub-Sub-Det / Data-Type.
+
   const AliEveHOMERSource::SourceId& sid = id();
 
   Int_t lvl = 0;
@@ -48,6 +60,8 @@ Int_t AliEveHOMERSourceMap::iterator::level()
 
 void AliEveHOMERSourceMap::PrintXXX()
 {
+  // Print entries in the map.
+
   for (iterator i = begin(); i != end(); ++i)
   {
     printf("%*s%s [state=%d, handle=0x%lx] {ssdet='%s'}\n", 4*i.level(), "",
@@ -68,6 +82,8 @@ AliEveHOMERSourceMapByDet::AliEveHOMERSourceMapByDet(ESourceGrouping_e grouping)
 
 TString AliEveHOMERSourceMapByDet::iterator_imp::description() const
 {
+  // Return identifier string for current entry.
+
   const AliEveHOMERSource::SourceId& sid = id();
 
   if ( ! sid.fType.IsNull())  return sid.fType;
@@ -81,6 +97,9 @@ void AliEveHOMERSourceMapByDet::insert(AliEveHOMERSource::SourceId& sid,
 				       AliEveHOMERSource::SourceState& sst,
 				       Bool_t def_state)
 {
+  // Insert source-state for given source-id.
+  // Does nothing if the entry already exists.
+
   Map_i i = fMap.find(sid);
   if (i == fMap.end())
   {
@@ -90,8 +109,10 @@ void AliEveHOMERSourceMapByDet::insert(AliEveHOMERSource::SourceId& sid,
   }
 }
 
-void AliEveHOMERSourceMapByDet::FillMap(TList* handles, Bool_t def_state)
+void AliEveHOMERSourceMapByDet::FillMap(const TList* handles, Bool_t def_state)
 {
+  // Fill the map from the list of HOMER source handles.
+
   TIter next(handles);
   AliHLTHOMERSourceDesc* h;
   while ((h = (AliHLTHOMERSourceDesc*) next()))
@@ -122,10 +143,14 @@ void AliEveHOMERSourceMapByDet::FillMap(TList* handles, Bool_t def_state)
 AliEveHOMERSourceMapByType::AliEveHOMERSourceMapByType(ESourceGrouping_e grouping) :
   AliEveHOMERSourceMap(grouping),
   fMap()
-{}
+{
+  // Constructor.
+}
 
 TString AliEveHOMERSourceMapByType::iterator_imp::description() const
 {
+  // Return identifier string for current entry.
+
   const AliEveHOMERSource::SourceId& sid = id();
 
   if ( ! sid.fSSDet.IsNull()) return sid.fSSDet;
@@ -139,6 +164,9 @@ void AliEveHOMERSourceMapByType::insert(AliEveHOMERSource::SourceId& sid,
 					AliEveHOMERSource::SourceState& sst,
 					Bool_t def_state)
 {
+  // Insert source-state for given source-id.
+  // Does nothing if the entry already exists.
+
   Map_i i = fMap.find(sid);
   if (i == fMap.end())
   {
@@ -148,8 +176,10 @@ void AliEveHOMERSourceMapByType::insert(AliEveHOMERSource::SourceId& sid,
   }
 }
 
-void AliEveHOMERSourceMapByType::FillMap(TList* handles, Bool_t def_state)
+void AliEveHOMERSourceMapByType::FillMap(const TList* handles, Bool_t def_state)
 {
+  // Fill the map from the list of HOMER source handles.
+
   TIter next(handles);
   AliHLTHOMERSourceDesc* h;
   while ((h = (AliHLTHOMERSourceDesc*) next()))
diff --git a/EVE/EveHLT/AliEveHOMERSourceMap.h b/EVE/EveHLT/AliEveHOMERSourceMap.h
index 4d2573eb46e..325be61e1ad 100644
--- a/EVE/EveHLT/AliEveHOMERSourceMap.h
+++ b/EVE/EveHLT/AliEveHOMERSourceMap.h
@@ -16,6 +16,14 @@
 
 #include <map>
 
+//______________________________________________________________________________
+//
+// AliEveHOMERSourceMap is an abstract container for HLT HOMER sources.
+// The concrete implementations AliEveHOMERSourceMapByDet and
+// AliEveHOMERSourceMapByType allow retrieval of HOMER sources in proper
+// order as required for their display in EVE object browser.
+
+
 class AliEveHOMERSourceMap : public TNamed
 {
 protected:
@@ -79,7 +87,7 @@ public:
 
   static AliEveHOMERSourceMap* Create(ESourceGrouping_e grouping);
 
-  virtual void FillMap(TList* handles, Bool_t def_state) = 0;
+  virtual void FillMap(const TList* handles, Bool_t def_state) = 0;
 
   void PrintXXX();
 
@@ -102,7 +110,7 @@ public:
   AliEveHOMERSourceMapByDet(ESourceGrouping_e grouping);
   virtual ~AliEveHOMERSourceMapByDet() {}
 
-  virtual void FillMap(TList* handles, Bool_t def_state);
+  virtual void FillMap(const TList* handles, Bool_t def_state);
 
 protected:
   typedef std::map<AliEveHOMERSource::SourceId,
@@ -153,7 +161,7 @@ public:
   AliEveHOMERSourceMapByType(ESourceGrouping_e grouping);
   virtual ~AliEveHOMERSourceMapByType() {}
 
-  virtual void FillMap(TList* handles, Bool_t def_state);
+  virtual void FillMap(const TList* handles, Bool_t def_state);
 
 protected:
   typedef std::map<AliEveHOMERSource::SourceId,