Cleaning up memory in destructor

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

diff --git a/MUON/AliMUONPainterPlotSelector.cxx b/MUON/AliMUONPainterPlotSelector.cxx
index 832fd4c418c2d1487b4ca284982e26f3b3476213..8b7749f504f55533a21aed3cb845ff69346fbca3 100644 (file)
--- a/MUON/AliMUONPainterPlotSelector.cxx
+++ b/MUON/AliMUONPainterPlotSelector.cxx
@@ -30,7 +30,7 @@
 #include "AliMUONPainterGroup.h"
 #include "AliMUONPainterInterfaceHelper.h"
 #include "AliMUONPainterMatrix.h"
-#include "AliMUONPainterRegistry.h"
+#include "AliMUONPainterDataRegistry.h"
 #include "AliMUONVPainter.h"
 #include "AliMUONVTrackerData.h"
 #include "AliLog.h"
@@ -62,7 +62,7 @@ fCurrentDimension(-1)
 
   fDataSourceNames = new TGButtonGroup(this,"Sources");
       
-  AliMUONPainterRegistry* reg = AliMUONPainterRegistry::Instance();
+  AliMUONPainterDataRegistry* reg = AliMUONPainterDataRegistry::Instance();
   
   reg->Connect("DataSourceWasRegistered(AliMUONVTrackerData*)",
                "AliMUONPainterPlotSelector",
@@ -142,7 +142,7 @@ AliMUONPainterPlotSelector::CreateDimensionButtons(const char* dataSourceName)
   
   AliDebug(1,Form("Creating dimension buttons for dataSource %s",dataSourceName));
   
-  AliMUONVTrackerData* data = AliMUONPainterRegistry::Instance()->FindDataSource(dataSourceName);
+  AliMUONVTrackerData* data = AliMUONPainterDataRegistry::Instance()->DataSource(dataSourceName);
 
   TGButtonGroup* bg = new TGButtonGroup(this,0,3,5,0,dataSourceName);
   
@@ -208,6 +208,8 @@ AliMUONPainterPlotSelector::DataSourceWasChanged(const char* type,
                   ( ( data && dataIndex >= 0 ) ? data->DimensionName(dataIndex).Data() :
                     "")));
   
+       UpdateTypeButton();
+       
   Long_t param[] = { (Long_t)type, (Long_t)data,
     (Long_t)dataIndex };
   
@@ -246,14 +248,14 @@ AliMUONPainterPlotSelector::NumberOfEventsChanged()
 
   // find first the sender of the signal
   
-  AliMUONVTrackerData* data = reinterpret_cast<AliMUONVTrackerData*>(gTQSender);
-  
-  TGButton* button = AliMUONPainterInterfaceHelper::FindButtonByUserData(*fDataSourceNames,data);
-  
-  if (button)
-  {
-    button->SetToolTipText(Form("%d events",data->NumberOfEvents()),250);
-  }
+//  AliMUONVTrackerData* data = reinterpret_cast<AliMUONVTrackerData*>(gTQSender);
+//  
+//  TGButton* button = AliMUONPainterInterfaceHelper::FindButtonByUserData(*fDataSourceNames,data);
+//  
+//  if (button)
+//  {
+//    button->SetToolTipText(Form("%d events",data->NumberOfEvents()),250);
+//  }
 }
 
 //_____________________________________________________________________________
@@ -262,29 +264,33 @@ AliMUONPainterPlotSelector::DataSourceWasUnregistered(AliMUONVTrackerData* data)
 {
   /// A data source has been unregistered : remove it from the interface
   
-  AliDebug(1,Form("Unregistering %s",data->GetName()));
+  TGButton* button = AliMUONPainterInterfaceHelper::FindButtonByUserData(*fDataSourceNames,data);
+
+  TGButton* bd = AliMUONPainterInterfaceHelper::FindDownButton(*fDataSourceNames);
 
-  TGButton* button = AliMUONPainterInterfaceHelper::FindButtonByName(*fDataSourceNames,
-                                                                     data->GetName());
-  AliMUONVTrackerData* check = reinterpret_cast<AliMUONVTrackerData*>(button->GetUserData());
-  if ( !button || check != data )
+  if ( bd == button ) 
   {
-    AliError("Something is seriously wrong. Please check");
-    return;
+    // selected data source is the one we are removing...
+    // revert to "none" before actually removing it.
+    SourceButtonWasClicked(1);
   }
   
-  fDataSourceNames->Remove(button);
-  delete button;
-  button->DestroyWindow();
-  
-  fDataSourceNames->Show();
+  AliMUONPainterInterfaceHelper::RemoveButton(*fDataSourceNames,button);
+
+  // do not forget to re-connect things
+  fDataSourceNames->Connect("Clicked(Int_t)","AliMUONPainterPlotSelector",
+                            this,
+                            "SourceButtonWasClicked(Int_t)");
   
+
   TObject* o = fDimensionButtonMap->Remove(new TObjString(data->GetName()));
   
   if (!o)
   {
     AliError("Remove failed. Please check");    
   }
+
+  fDataSourceNames->Show();
   
   Layout();
 }
@@ -403,9 +409,8 @@ AliMUONPainterPlotSelector::SourceButtonWasClicked(Int_t id)
   TGButton* button = fDataSourceNames->GetButton(id);
   if ( !button ) 
   {
-    AliError(Form("Could not get DataSource button id=%d",id));
     StdoutToAliDebug(1,AliMUONPainterInterfaceHelper::Dump(*fDataSourceNames));
-    button->GetUserData(); // to trigger a crash so gdb is possible ;-)
+    AliFatal(Form("Could not get DataSource button id=%d",id));
   }
   
   AliMUONVTrackerData* data = reinterpret_cast<AliMUONVTrackerData*>(button->GetUserData());
@@ -548,9 +553,12 @@ AliMUONPainterPlotSelector::UpdateSourceButton()
   (AliMUONPainterInterfaceHelper::FindButtonByUserData(*fDataSourceNames,
                                                        fCurrentData));
   
-  AliMUONPainterInterfaceHelper::Select(*fDataSourceNames,button->GetTitle());
+  if ( button ) 
+  {
+    AliMUONPainterInterfaceHelper::Select(*fDataSourceNames,button->GetTitle());
   
-  RestoreDimensionButtons(button->GetTitle(),kFALSE);
+    RestoreDimensionButtons(button->GetTitle(),kFALSE);
+  }
   
   fDataSourceNames->Show();
 }
@@ -560,9 +568,43 @@ void
 AliMUONPainterPlotSelector::UpdateTypeButton()
 {
   /// Update the type buttons
-  
+       
   AliDebug(1,Form("fCurrentType=%s",fCurrentType.Data()));
   
+       if (!fCurrentData)
+  {
+    AliMUONPainterInterfaceHelper::SetState(*fTypes,kFALSE);
+  }
+       else
+       {
+               // disable levels that cannot be handled by this data
+               TGTextButton* padButton = static_cast<TGTextButton*>
+               (AliMUONPainterInterfaceHelper::FindButtonByName(*fTypes,"PAD"));
+               if (padButton) 
+               { 
+                       padButton->SetEnabled(fCurrentData->IsChannelLevelEnabled());
+               }
+               TGTextButton* manuButton = static_cast<TGTextButton*>
+               (AliMUONPainterInterfaceHelper::FindButtonByName(*fTypes,"MANU"));
+               if (manuButton) 
+               { 
+                       manuButton->SetEnabled(fCurrentData->IsManuLevelEnabled());
+               }
+               TGTextButton* busPatchButton = static_cast<TGTextButton*>
+               (AliMUONPainterInterfaceHelper::FindButtonByName(*fTypes,"BUSPATCH"));
+               if (busPatchButton) 
+               { 
+                       busPatchButton->SetEnabled(fCurrentData->IsBusPatchLevelEnabled());
+               }
+               TGTextButton* pcbButton = static_cast<TGTextButton*>
+               (AliMUONPainterInterfaceHelper::FindButtonByName(*fTypes,"PCB"));
+               if (pcbButton) 
+               { 
+                       pcbButton->SetEnabled(fCurrentData->IsPCBLevelEnabled());
+               }
+    
+       }
+       
   TGTextButton* button = static_cast<TGTextButton*>
   (AliMUONPainterInterfaceHelper::FindButtonByName(*fTypes,fCurrentType));
 
@@ -575,11 +617,7 @@ AliMUONPainterPlotSelector::UpdateTypeButton()
     AliMUONPainterInterfaceHelper::Unselect(*fTypes,"*");
   }
 
-  if (!fCurrentData)
-  {
-    AliMUONPainterInterfaceHelper::SetState(*fTypes,kFALSE);
-  }
-  
+       
   fTypes->Show();
 }