Coverity fix

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

diff --git a/MUON/AliMUONTrackerDataCompareDialog.cxx b/MUON/AliMUONTrackerDataCompareDialog.cxx
index 9b54277ec1d86e3e64f58e9fe8acc1d3740e56d2..4f342866cf454ab9d25ea04d484d5da975da846f 100644 (file)
--- a/MUON/AliMUONTrackerDataCompareDialog.cxx
+++ b/MUON/AliMUONTrackerDataCompareDialog.cxx
@@ -32,13 +32,13 @@
 ///
 
 #include "AliLog.h"
-#include "AliMpConstants.h"
 #include "AliMUON2DMap.h"
 #include "AliMUONCalibParamND.h"
-#include "AliMUONPainterRegistry.h"
+#include "AliMUONPainterDataRegistry.h"
 #include "AliMUONTrackerData.h"
 #include "AliMUONTrackerDataWrapper.h"
 #include "AliMUONVTrackerData.h"
+#include "AliMpConstants.h"
 #include "AliMpDDLStore.h"
 #include "AliMpDetElement.h"
 #include "AliMpManuIterator.h"
@@ -106,7 +106,7 @@ fCancel(new TGTextButton(fButtonFrame,"Cancel"))
   
   SetCleanup(kDeepCleanup);
   
-  AliMUONPainterRegistry* reg = AliMUONPainterRegistry::Instance();
+  AliMUONPainterDataRegistry* reg = AliMUONPainterDataRegistry::Instance();
   
   for ( Int_t i = 0; i < reg->NumberOfDataSources(); ++i ) 
   {
@@ -205,18 +205,20 @@ AliMUONTrackerDataCompareDialog::CompareData(const char* d1name,
 {
   /// Compare two data sources
   
-  AliMUONPainterRegistry* reg = AliMUONPainterRegistry::Instance();
+  AliMUONPainterDataRegistry* reg = AliMUONPainterDataRegistry::Instance();
   
   AliMUONVTrackerData* d1 = reg->DataSource(d1name);
   if (!d1)
   {
     AliError(Form("Cannot find data source %s",d1name));
+    return;
   }
   
   AliMUONVTrackerData* d2 = reg->DataSource(d2name);
   if (!d2)
   {
     AliError(Form("Cannot find data source %s",d2name));
+    return;
   }
   
   Double_t (*difffunction)(Double_t,Double_t)=0x0;
@@ -243,13 +245,16 @@ AliMUONTrackerDataCompareDialog::CompareData(const char* d1name,
     suffix = "ARD";
   }
   
-  TString basename = fBasename->GetText(); 
-  
-  AliMUONVTrackerData* d = CompareData(*d1,*d2,Form("%s:%s",basename.Data(),suffix.Data()),difffunction);
-  
-  AliMUONVTrackerDataMaker* dw = new AliMUONTrackerDataWrapper(d);
+  if ( difffunction ) 
+  {
+    TString basename = fBasename->GetText(); 
   
-  AliMUONPainterRegistry::Instance()->Register(dw);
+    AliMUONVTrackerData* d = CompareData(*d1,*d2,Form("%s:%s",basename.Data(),suffix.Data()),difffunction);
+    
+    AliMUONVTrackerDataMaker* dw = new AliMUONTrackerDataWrapper(d);
+    
+    AliMUONPainterDataRegistry::Instance()->Register(dw);
+  }
 }
 
 //______________________________________________________________________________