Reverting commit from rev. 37415;
authorivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 2 Dec 2009 09:18:36 +0000 (09:18 +0000)
committerivana <ivana@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 2 Dec 2009 09:18:36 +0000 (09:18 +0000)
at the end the empty mapping objects are not requested.
See also bug report #54539

12 files changed:
MUON/mapping/AliMpCDB.cxx
MUON/mapping/AliMpCDB.h
MUON/mapping/AliMpDDLStore.cxx
MUON/mapping/AliMpDDLStore.h
MUON/mapping/AliMpExMap.cxx
MUON/mapping/AliMpExMap.h
MUON/mapping/AliMpManuStore.cxx
MUON/mapping/AliMpManuStore.h
MUON/mapping/AliMpSegmentation.cxx
MUON/mapping/AliMpSegmentation.h
MUON/mapping/AliMpStringObjMap.cxx
MUON/mapping/AliMpStringObjMap.h

index a1ad6b3..077198f 100644 (file)
@@ -400,10 +400,9 @@ Bool_t AliMpCDB::WriteMpRunData()
 }
 
 //______________________________________________________________________________
-Bool_t AliMpCDB::WriteMpSegmentation(Bool_t readData, Bool_t empty)
+Bool_t AliMpCDB::WriteMpSegmentation(Bool_t readData)
 {
-/// Write mapping segmentation in OCDB;
-/// if empty is true then a non-valid object of minimum size is written
+/// Write mapping segmentation in OCDB
 
   if ( ! readData && ! AliMpSegmentation::Instance() ) return false;
 
@@ -419,18 +418,17 @@ Bool_t AliMpCDB::WriteMpSegmentation(Bool_t readData, Bool_t empty)
 
   if ( readData ) {
     AliMpDataStreams dataStreams;
-    AliMpSegmentation::ReadData(dataStreams, false, empty);
-    AliMpDDLStore::ReadData(dataStreams, false, empty);
+    AliMpSegmentation::ReadData(dataStreams, false);
+    AliMpDDLStore::ReadData(dataStreams, false);
   }
   
   return cdbManager->Put(AliMpSegmentation::Instance(), id, cdbData);
 }
 
 //______________________________________________________________________________
-Bool_t AliMpCDB::WriteDDLStore(Bool_t readData, Bool_t empty)
+Bool_t AliMpCDB::WriteDDLStore(Bool_t readData)
 {
 /// Write mapping DDL store in OCDB
-/// if empty is true then a non-valid object of minimum size is written
 
   if ( ! readData && ! AliMpDDLStore::Instance() ) return false;
 
@@ -446,17 +444,16 @@ Bool_t AliMpCDB::WriteDDLStore(Bool_t readData, Bool_t empty)
 
   if ( readData ) {
     AliMpDataStreams dataStreams;
-    AliMpSegmentation::ReadData(dataStreams, false, empty);
-    AliMpDDLStore::ReadData(dataStreams, false, empty);
+    AliMpSegmentation::ReadData(dataStreams, false);
+    AliMpDDLStore::ReadData(dataStreams, false);
   }
   return cdbManager->Put(AliMpDDLStore::Instance(), id, cdbData);
 }   
 
 //______________________________________________________________________________
-Bool_t AliMpCDB::WriteManuStore(Bool_t readData, Bool_t empty)
+Bool_t AliMpCDB::WriteManuStore(Bool_t readData)
 {
 /// Write mapping Manu store in OCDB
-/// if empty is true then a non-valid object of minimum size is written
 
   if ( ! readData && ! AliMpManuStore::Instance() ) return false;
 
@@ -472,8 +469,8 @@ Bool_t AliMpCDB::WriteManuStore(Bool_t readData, Bool_t empty)
 
   if ( readData ) {
     AliMpDataStreams dataStreams;
-    AliMpSegmentation::ReadData(dataStreams, false, empty);
-    AliMpManuStore::ReadData(dataStreams, false, empty);
+    AliMpSegmentation::ReadData(dataStreams, false);
+    AliMpManuStore::ReadData(dataStreams, false);
   }
   return cdbManager->Put(AliMpManuStore::Instance(), id, cdbData);
 }   
index 8ab1e84..dff83f8 100644 (file)
@@ -61,9 +61,9 @@ class AliMpCDB : public  TObject {
   
     static Bool_t WriteMpData();
     static Bool_t WriteMpRunData();
-    static Bool_t WriteMpSegmentation(Bool_t readData = true, Bool_t empty = true);
-    static Bool_t WriteDDLStore(Bool_t readData= true, Bool_t empty = true);
-    static Bool_t WriteManuStore(Bool_t readData= true, Bool_t empty = true);
+    static Bool_t WriteMpSegmentation(Bool_t readData = true);
+    static Bool_t WriteDDLStore(Bool_t readData= true);
+    static Bool_t WriteManuStore(Bool_t readData= true);
     
     static Bool_t GenerateMpData(
                     const char* cdbpath = "local://$ALICE_ROOT/OCDB",
index f1eb73b..c234f45 100644 (file)
@@ -87,7 +87,7 @@ AliMpDDLStore* AliMpDDLStore::Instance(Bool_t warn)
 
 //______________________________________________________________________________
 AliMpDDLStore* AliMpDDLStore::ReadData(const AliMpDataStreams& dataStreams,
-                                       Bool_t warn, Bool_t empty) 
+                                       Bool_t warn) 
 {
     /// Load the DDL store from ASCII data files
     /// and return its instance
@@ -98,17 +98,10 @@ AliMpDDLStore* AliMpDDLStore::ReadData(const AliMpDataStreams& dataStreams,
         return fgInstance;
     }
 
-    if ( ! empty ) {
-      if ( dataStreams.GetReadFromFiles() )
-        AliInfoClass("Reading DDL Store from ASCII files.");
-
-      fgInstance = new AliMpDDLStore(dataStreams);
-    }
-    else {
-      AliInfoClass("Creating invalid DDL Store of minimum size.");
-      fgInstance = new AliMpDDLStore(dataStreams, true);
-    }   
+    if ( dataStreams.GetReadFromFiles() )
+      AliInfoClass("Reading DDL Store from ASCII files.");
 
+    fgInstance = new AliMpDDLStore(dataStreams);
     return fgInstance;
 }
 
@@ -131,7 +124,6 @@ AliMpDDLStore::AliMpDDLStore(const AliMpDataStreams& dataStreams)
   AliDebug(1,"");
   fDDLs.SetOwner(true);
   fBusPatches.SetOwner(true);
-  
   fBusPatches.SetSize(900);
 
   // Load segmentation & DE store data
@@ -149,25 +141,6 @@ AliMpDDLStore::AliMpDDLStore(const AliMpDataStreams& dataStreams)
 }
 
 //______________________________________________________________________________
-AliMpDDLStore::AliMpDDLStore(const AliMpDataStreams& dataStreams, Bool_t /*empty*/)
-        : TObject(),
-        fkDataStreams(dataStreams),
-        fDDLs(fgkNofDDLs+fgkNofTriggerDDLs), // FIXEME
-        fBusPatches(true),
-        fManuList12(),
-        fManuBridge2(),
-        fRegionalTrigger()
-{
-  /// The special constructor for creating a minimum size object
-  /// The argument minSize is used only to distinguish this ctor
-  /// from the standard one, its value is not used.
-  
-  AliDebug(1,"");
-  fDDLs.SetOwner(true);
-  fBusPatches.SetOwner(true);
-}
-
-//______________________________________________________________________________
 AliMpDDLStore::AliMpDDLStore(TRootIOCtor* ioCtor)
         : TObject(),
         fkDataStreams(ioCtor),
@@ -572,7 +545,6 @@ Bool_t AliMpDDLStore::ReadBusPatchSpecial()
         }
       }
     }
-    
     delete stringList;
   }
   
index 7ad2a45..d9645c2 100644 (file)
@@ -41,8 +41,7 @@ class AliMpDDLStore : public  TObject {
     // static access method
     static AliMpDDLStore* Instance(Bool_t warn = true); 
     static AliMpDDLStore* ReadData(const AliMpDataStreams& dataStreams,
-                                   Bool_t warn = true,
-                                   Bool_t empty = false);
+                                   Bool_t warn = true);
     
     // methods
     AliMpDDL*          GetDDL(Int_t ddlId, Bool_t warn = true) const;
@@ -75,7 +74,6 @@ class AliMpDDLStore : public  TObject {
     
   private:
     AliMpDDLStore(const AliMpDataStreams& dataStreams);
-    AliMpDDLStore(const AliMpDataStreams& dataStreams, Bool_t empty);
     /// Not implemented
     AliMpDDLStore();
     /// Not implemented
index 9f14af0..08b2955 100644 (file)
@@ -104,25 +104,6 @@ AliMpExMap::AliMpExMap()
 }
 
 //_____________________________________________________________________________
-AliMpExMap::AliMpExMap(Bool_t /*minSize*/) 
-  : TObject(),
-    fMap(0),
-    fObjects(0),
-    fKeys(0)
-{
-      /// Special constructor for creating an object with minimum size.
-      /// The argument minSize is used only to distinguish this ctor
-      /// from the standard one, its value is not used.
-      /// When setting the initial size to 0, Root will adapt it to a minimal
-      /// allowed size: 
-      /// - 5 for TExMap (fMap)
-      /// - TCollection::kInitCapacity for TObjArray (fObjects)
-      /// - 0 for TArrayL (fKeys)
-
-  fObjects.SetOwner(fgkDefaultOwnership);
-}
-
-//_____________________________________________________________________________
 AliMpExMap::AliMpExMap(TRootIOCtor*) 
   : TObject(),
     fMap(),
index 963e836..b965e05 100644 (file)
@@ -31,7 +31,6 @@ class AliMpExMap : public TObject
   
   public:
     AliMpExMap();
-    AliMpExMap(Bool_t minSize);
     AliMpExMap(TRootIOCtor* /*ioCtor*/);
     AliMpExMap(const AliMpExMap& rhs);
     AliMpExMap& operator=(const AliMpExMap& rhs);
index 3a71ed8..afbcc0f 100644 (file)
@@ -74,28 +74,21 @@ AliMpManuStore* AliMpManuStore::Instance(Bool_t warn)
 
 //______________________________________________________________________________
 AliMpManuStore* AliMpManuStore::ReadData(const AliMpDataStreams& dataStreams, 
-                                         Bool_t warn, Bool_t empty) 
+                                         Bool_t warn) 
 {
     /// Load the DDL store from ASCII data files
     /// and return its instance
 
     if ( fgInstance ) {
         if ( warn )
-            AliWarningClass("Manu Store has been already loaded");
+            AliWarningClass("DDL Store has been already loaded");
         return fgInstance;
     }
 
-    if ( ! empty ) {
-      if ( dataStreams.GetReadFromFiles() )
-        AliInfoClass("Reading Manu Store from ASCII files.");
+    if ( dataStreams.GetReadFromFiles() )
+      AliInfoClass("Reading Manu Store from ASCII files.");
 
-      fgInstance = new AliMpManuStore(dataStreams);
-    }
-    else {
-      AliInfoClass("Creating invalid Manu Store of minimum size.");
-      fgInstance = new AliMpManuStore(dataStreams, true);
-    }   
-        
+    fgInstance = new AliMpManuStore(dataStreams);
     return fgInstance;
 }
 
@@ -124,24 +117,8 @@ AliMpManuStore::AliMpManuStore(const AliMpDataStreams& dataStreams)
        << endl;
      return;
   }      
-  
-  ReadManuSerial();
-}
 
-//______________________________________________________________________________
-AliMpManuStore::AliMpManuStore(const AliMpDataStreams& dataStreams, Bool_t empty)
-: TObject(),
-  fkDataStreams(dataStreams),
-  fManuToSerialNbs(0),
-  fSerialNbToManus(0),
-  fNofManusInDE(0),
-  fNofManus(0)
-{  
-/// The special constructor for creating a minimum size object
-/// The argument minSize is used only to distinguish this ctor
-/// from the standard one, its value is not used.
-
-  AliDebug(1,"");
+  ReadManuSerial();
 }
 
 //______________________________________________________________________________
index 1eb2a14..30cf5ad 100644 (file)
@@ -35,8 +35,7 @@ class AliMpManuStore : public  TObject {
     // static access method
     static AliMpManuStore* Instance(Bool_t warn = true); 
     static AliMpManuStore* ReadData(const AliMpDataStreams& dataStreams,
-                                    Bool_t warn = true,
-                                    Bool_t empty = false);
+                                    Bool_t warn = true);
                                     
     static void SetWarnIfDoublon(Bool_t warn);                             
     
@@ -52,7 +51,6 @@ class AliMpManuStore : public  TObject {
 
   private:
     AliMpManuStore(const AliMpDataStreams& dataStreams);
-    AliMpManuStore(const AliMpDataStreams& dataStreams, Bool_t empty);
      /// Not implemented
     AliMpManuStore();
      /// Not implemented
index bfeac34..ce9b566 100644 (file)
@@ -82,12 +82,10 @@ AliMpSegmentation* AliMpSegmentation::Instance(Bool_t warn)
 
 //______________________________________________________________________________
 AliMpSegmentation* AliMpSegmentation::ReadData(const AliMpDataStreams& dataStreams,
-                                               Bool_t warn, Bool_t empty)
+                                               Bool_t warn)
 {
 /// Load the sementation from ASCII data files
-/// and return its instance. \n
-/// If empty parameter is true, then no data is loaded and 
-/// an empty object is created.
+/// and return its instance
 
   if ( fgInstance ) {
     if ( warn )
@@ -95,17 +93,10 @@ AliMpSegmentation* AliMpSegmentation::ReadData(const AliMpDataStreams& dataStrea
     return fgInstance;
   }  
   
-  if ( ! empty ) {
-    if ( dataStreams.GetReadFromFiles() )
-      AliInfoClass("Reading segmentation from ASCII files.");
+  if ( dataStreams.GetReadFromFiles() )
+    AliInfoClass("Reading segmentation from ASCII files.");
 
-    fgInstance = new AliMpSegmentation(dataStreams);
-  }  
-  else {
-    AliInfoClass("Creating invalid segmentation of minimum size.");
-    fgInstance = new AliMpSegmentation(dataStreams, empty);
-  }   
-    
+  fgInstance = new AliMpSegmentation(dataStreams);
   return fgInstance;
 }    
 
@@ -132,7 +123,7 @@ AliMpSegmentation::AliMpSegmentation(const AliMpDataStreams& dataStreams)
   if ( ! AliMpDEStore::Instance(false) )  
     AliMpDEStore::ReadData(dataStreams);
   fDetElements = AliMpDEStore::Instance();  
-  
+
   // Create mapping segmentations for all detection elements
   AliMpDEIterator it;
   for ( it.First(); ! it.IsDone(); it.Next() ) 
@@ -155,25 +146,6 @@ AliMpSegmentation::AliMpSegmentation(const AliMpDataStreams& dataStreams)
 }
 
 //______________________________________________________________________________
-AliMpSegmentation::AliMpSegmentation(const AliMpDataStreams& dataStreams, 
-                                     Bool_t /*empty*/)
-: TObject(),
-  fkDataStreams(dataStreams),
-  fDetElements(0x0),
-  fMpSegmentations(true, true),
-  fElCardsMap(true),
-  fSlatMotifMap(0x0)
-{  
-/// The special constructor for creating a minimum size object
-/// The argument minSize is used only to distinguish this ctor
-/// from the standard one, its value is not used.
-
-  AliDebug(1,"");
-  
-  fElCardsMap.SetOwner(kTRUE);
-}
-
-//______________________________________________________________________________
 AliMpSegmentation::AliMpSegmentation(TRootIOCtor* ioCtor)
 : TObject(),
   fkDataStreams(ioCtor),
index 6b21e01..914765f 100644 (file)
@@ -56,8 +56,7 @@ class AliMpSegmentation : public  TObject {
     // static methods
     static AliMpSegmentation* Instance(Bool_t warn = true);
     static AliMpSegmentation* ReadData(const AliMpDataStreams& dataStreams, 
-                                       Bool_t warn = true,
-                                       Bool_t empty = false);
+                                       Bool_t warn = true);
 
     // methods
     
@@ -99,7 +98,6 @@ class AliMpSegmentation : public  TObject {
   
   private:
     AliMpSegmentation(const AliMpDataStreams& dataStreams);
-    AliMpSegmentation(const AliMpDataStreams& dataStreams, Bool_t empty);
     /// Not implemented
     AliMpSegmentation(const AliMpSegmentation& rhs);
     /// Not implemented
index 334fcc0..fdd6eef 100644 (file)
@@ -52,25 +52,6 @@ AliMpStringObjMap::AliMpStringObjMap(Bool_t isOwner)
 }
 
 //______________________________________________________________________________
-AliMpStringObjMap::AliMpStringObjMap(Bool_t /*minSize*/, Bool_t isOwner)
- : TObject(),
-   fNofItems(0),
-   fFirstArray(0),
-   fSecondArray(0),
-   fCurrentIndex(0)
-{
-/// Special constructor to create an object of minimum size.
-/// The argument minSize is used only to distinguish this ctor
-/// from the standard one, its value is not used.
-/// When setting the initial size to 0, Root will adapt it to a minimal
-/// allowed size: 
-/// - TCollection::kInitCapacity for TObjArray (fObjects)
-
-  fFirstArray.SetOwner(true);
-  fSecondArray.SetOwner(isOwner);
-}
-
-//______________________________________________________________________________
 AliMpStringObjMap::~AliMpStringObjMap()
 {
 /// Destructor
index d796906..6639153 100644 (file)
@@ -22,7 +22,6 @@ class AliMpStringObjMap : public TObject
 {
   public:
     AliMpStringObjMap(Bool_t isOwner = false);
-    AliMpStringObjMap(Bool_t minSize, Bool_t isOwner);
     virtual ~AliMpStringObjMap();
     
     // methods