Fixes for coverity: SELF_ASSIGN
authorhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 20 Jan 2012 17:19:03 +0000 (17:19 +0000)
committerhristov <hristov@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 20 Jan 2012 17:19:03 +0000 (17:19 +0000)
18 files changed:
ANALYSIS/AliAnalysisTaskSE.cxx
HLT/MUON/OnlineAnalysis/AliHLTMUONMansoTrackerFSMComponent.h
MUON/AliMUONAlignmentTask.cxx
MUON/AliMUONReAlignTask.cxx
OADB/AliOADBPhysicsSelection.cxx
PWG2/DIFFRACTIVE/AliAnalysisTaskDDMeson.cxx
PWG2/UNICOR/AliUnicorEventAliceESD.h
PWGPP/AliRelAlignerKalmanArray.cxx
STEER/CDB/AliBaseCalibViewer.cxx
STEER/STEER/AliMatrixSparse.cxx
STEER/STEER/AliRelAlignerKalman.cxx
STEER/STEERBase/AliMagF.cxx
STEER/STEERBase/AliTRDPIDParams.cxx
TOF/AliTOFChainSummaryData.cxx
TOF/AliTOFDRMSummaryData.cxx
TOF/AliTOFDecoderSummaryData.cxx
TOF/AliTOFLTMSummaryData.cxx
TOF/AliTOFTRMSummaryData.cxx

index d1a5ad3..ef6bd2e 100644 (file)
@@ -143,6 +143,9 @@ AliAnalysisTaskSE::AliAnalysisTaskSE(const AliAnalysisTaskSE& obj):
 AliAnalysisTaskSE& AliAnalysisTaskSE::operator=(const AliAnalysisTaskSE& other)
 {
 // Assignment
+  if(&other == this) return *this;
+  AliAnalysisTask::operator=(other);
+
     AliAnalysisTask::operator=(other);
     fDebug            = other.fDebug;
     fEntry            = other.fEntry;
index b63415c..3aeb2ed 100644 (file)
@@ -258,6 +258,7 @@ private:
                
                AliRecHitBlockInfo& operator = (const AliRecHitBlockInfo& obj)
                {
+                 if(&obj == this) return *this;
                        fCount = obj.fCount;
                        fData = obj.fData;
                        return *this;
index a154887..3fbbd6c 100644 (file)
@@ -160,6 +160,7 @@ AliMUONAlignmentTask::AliMUONAlignmentTask(const AliMUONAlignmentTask& other):
 AliMUONAlignmentTask& AliMUONAlignmentTask::operator=(const AliMUONAlignmentTask& other)
 {
   /// Assignment
+  if(&other == this) return *this;
   AliAnalysisTaskSE::operator=(other);
 
   fReadRecords = other.fReadRecords;
index 9f03b5b..f9a8210 100644 (file)
@@ -149,6 +149,7 @@ AliMUONReAlignTask::AliMUONReAlignTask(const AliMUONReAlignTask& obj)
 AliMUONReAlignTask& AliMUONReAlignTask::operator=(const AliMUONReAlignTask& other)
 {
   /// Assignment
+  if(&other == this) return *this;
   AliAnalysisTask::operator=(other);
   fESD = other.fESD;
   fClusterInfoTree = other.fClusterInfoTree;
index 870924e..6aa685c 100644 (file)
@@ -80,6 +80,7 @@ AliOADBPhysicsSelection::AliOADBPhysicsSelection(const AliOADBPhysicsSelection&
 
 AliOADBPhysicsSelection& AliOADBPhysicsSelection::operator=(const AliOADBPhysicsSelection& other)  {
   //Assignment operator
+  if(&other == this) return *this;
   TNamed::operator=(other);
 
   fCollTrigClasses = other.fCollTrigClasses;
index 476d5d1..cd991d6 100644 (file)
@@ -191,6 +191,9 @@ AliAnalysisTaskDDMeson & AliAnalysisTaskDDMeson::operator=(const AliAnalysisTask
   //
   // overload =
   //
+  if(&p == this) return *this;
+  AliAnalysisTaskSE::operator=(p);
+
   fESD = p.fESD;
 
   fnmass = p.fnmass;
index 3d6e8de..4e885c6 100644 (file)
@@ -25,7 +25,10 @@ class AliUnicorEventAliceESD : public AliUnicorEvent {
              //AliUnicorEventAliceESD(const AliUnicorEventAliceESD &ev): AliUnicorEvent(ev), fViper(ev.fViper), fESD(ev.fESD), fPhysicsSelection(ev.fPhysicsSelection){}
               AliUnicorEventAliceESD(const AliUnicorEventAliceESD &ev): AliUnicorEvent(ev), fViper(ev.fViper), fESD(ev.fESD) {}
   virtual     ~AliUnicorEventAliceESD();
-  AliUnicorEventAliceESD &operator=(const AliUnicorEventAliceESD &source) {fViper=source.fViper; fESD=source.fESD; return *this;}
+  AliUnicorEventAliceESD &operator=(const AliUnicorEventAliceESD &source) {
+    if(&source == this) return *this;
+    AliUnicorEvent::operator=(source);
+    fViper=source.fViper; fESD=source.fESD; return *this;}
   Double_t    Etamin() const {return -0.75;}
   Double_t    Etamax() const {return  0.75;}
   void        AttachTree(TTree *tr) {fESD->ReadFromTree(tr);}
index 096a6f5..23dcce7 100644 (file)
@@ -206,6 +206,9 @@ AliRelAlignerKalman* AliRelAlignerKalmanArray::GetAligner(AliESDEvent* event)
 AliRelAlignerKalmanArray& AliRelAlignerKalmanArray::operator=(const AliRelAlignerKalmanArray& in)
 {
   //assignment operator
+  if(&in == this) return *this;
+  TNamed::operator=(in);
+
   if (fSize!=in.fSize)
   {
     //if sizes different, delete curent and make a new one with new size
index 4b1dad9..231d3bc 100644 (file)
@@ -121,6 +121,10 @@ AliBaseCalibViewer & AliBaseCalibViewer::operator =(const AliBaseCalibViewer & p
    // assignment operator - dummy
    // not yet working!!!
    //
+  if(&param == this) return *this;
+  TObject::operator=(param);
+
+
    fTree = param.fTree;
    fTreeMustBeDeleted = param.fTreeMustBeDeleted;
    fListOfObjectsToBeDeleted = param.fListOfObjectsToBeDeleted;
index 87320ad..a9be619 100644 (file)
@@ -47,7 +47,9 @@ AliVectorSparse* AliMatrixSparse::GetRowAdd(Int_t ir)
 //___________________________________________________________
 AliMatrixSparse& AliMatrixSparse::operator=(const AliMatrixSparse& src)
 {
-  if (*this == src) return *this;
+  if (this == &src) return *this;
+  AliMatrixSq::operator=(src);
+
   Clear();
   fNcols = src.GetNCols();
   fNrows = src.GetNRows();
index ebed7a4..e4222fb 100644 (file)
@@ -199,7 +199,10 @@ AliRelAlignerKalman::AliRelAlignerKalman(const AliRelAlignerKalman& a):
 AliRelAlignerKalman& AliRelAlignerKalman::operator=(const AliRelAlignerKalman& a)
 {
   //assignment operator
-  fMagField=a.fMagField,
+  if(&a == this) return *this;
+  TObject::operator=(a);
+
+  fMagField=a.fMagField;
   fNMeasurementParams=a.fNMeasurementParams;
   *fPX = *a.fPX;
   *fPXcov = *a.fPXcov;
index adc092c..65dee8a 100644 (file)
@@ -233,9 +233,11 @@ Double_t AliMagF::GetBz(const Double_t *xyz) const
 //_______________________________________________________________________
 AliMagF& AliMagF::operator=(const AliMagF& src)
 {
-  if (this != &src && src.fMeasuredMap) { 
-    if (fMeasuredMap) delete fMeasuredMap;
-    fMeasuredMap = new AliMagWrapCheb(*src.fMeasuredMap);
+  if (this != &src) {
+    if (src.fMeasuredMap) { 
+      if (fMeasuredMap) delete fMeasuredMap;
+      fMeasuredMap = new AliMagWrapCheb(*src.fMeasuredMap);
+    }
     SetName(src.GetName());
     fSolenoid    = src.fSolenoid;
     fBeamType    = src.fBeamType;
index f0be60a..5f8e60c 100644 (file)
@@ -155,6 +155,8 @@ AliTRDPIDParams::AliTRDPIDThresholds &AliTRDPIDParams::AliTRDPIDThresholds::oper
   //
   // Assignment operator
   //
+  if(&ref == this) return *this;
+
   TObject::operator=(ref);
 
   fNTracklets = ref.fNTracklets;
index 01c0806..c401908 100644 (file)
@@ -54,7 +54,7 @@ AliTOFChainSummaryData::AliTOFChainSummaryData() :
 //_________________________________________________________________
 
 AliTOFChainSummaryData::AliTOFChainSummaryData(const AliTOFChainSummaryData &source) :
-  TObject(),
+  TObject(source),
   fHeader(source.fHeader),
   fTrailer(source.fTrailer),
   fChain(source.fChain),
@@ -80,6 +80,9 @@ AliTOFChainSummaryData &
 AliTOFChainSummaryData::operator = (const AliTOFChainSummaryData &source)
 {
   /* operator = */
+  if(&source == this) return *this;
+  TObject::operator=(source);
+
   fHeader = source.fHeader;
   fTrailer = source.fTrailer;
   fChain = source.fChain;
index 1424bf0..1c1ef5d 100644 (file)
@@ -64,7 +64,7 @@ AliTOFDRMSummaryData::AliTOFDRMSummaryData() :
 //_________________________________________________________________
 
 AliTOFDRMSummaryData::AliTOFDRMSummaryData(const AliTOFDRMSummaryData &source) :
-  TObject(),
+  TObject(source),
   fHeader(source.fHeader),
   fTrailer(source.fTrailer),
   fSlotID(source.fSlotID),
@@ -100,6 +100,9 @@ AliTOFDRMSummaryData &
 AliTOFDRMSummaryData::operator = (const AliTOFDRMSummaryData &source)
 {
   /* operator = */
+  if(&source == this) return *this;
+  TObject::operator=(source);
+
   fHeader = source.fHeader;
   fTrailer = source.fTrailer;
   fSlotID = source.fSlotID;
index b3f6ca4..f9c8c67 100644 (file)
@@ -56,7 +56,7 @@ AliTOFDecoderSummaryData::AliTOFDecoderSummaryData() :
 //_________________________________________________________________
 
 AliTOFDecoderSummaryData::AliTOFDecoderSummaryData(const AliTOFDecoderSummaryData &source) :
-  TObject(),
+  TObject(source),
   fRunNumber(source.fRunNumber),
   fEventNumber(source.fEventNumber),
   fEquipmentID(source.fEquipmentID),
@@ -84,6 +84,9 @@ AliTOFDecoderSummaryData &
 AliTOFDecoderSummaryData::operator = (const AliTOFDecoderSummaryData &source)
 {
   /* operator = */
+  if(&source == this) return *this;
+  TObject::operator=(source);
+
   fRunNumber = source.fRunNumber;
   fEventNumber = source.fEventNumber;
   fEquipmentID = source.fEquipmentID;
index a9504cc..1bb1501 100644 (file)
@@ -54,7 +54,7 @@ AliTOFLTMSummaryData::AliTOFLTMSummaryData() :
 //_________________________________________________________________
 
 AliTOFLTMSummaryData::AliTOFLTMSummaryData(const AliTOFLTMSummaryData &source) :
-  TObject(),
+  TObject(source),
   fHeader(source.fHeader),
   fTrailer(source.fTrailer),
   fSlotID(source.fSlotID),
@@ -80,6 +80,9 @@ AliTOFLTMSummaryData &
 AliTOFLTMSummaryData::operator = (const AliTOFLTMSummaryData &source)
 {
   /* operator = */
+  if(&source == this) return *this;
+  TObject::operator=(source);
+
   fHeader = source.fHeader;
   fTrailer = source.fTrailer;
   fSlotID = source.fSlotID;
index be3dd64..0925464 100644 (file)
@@ -51,7 +51,7 @@ AliTOFTRMSummaryData::AliTOFTRMSummaryData() :
 //_________________________________________________________________
 
 AliTOFTRMSummaryData::AliTOFTRMSummaryData(const AliTOFTRMSummaryData &source) :
-  TObject(),
+  TObject(source),
   fHeader(source.fHeader),
   fTrailer(source.fTrailer),
   fSlotID(source.fSlotID),
@@ -74,6 +74,9 @@ AliTOFTRMSummaryData &
 AliTOFTRMSummaryData::operator = (const AliTOFTRMSummaryData &source)
 {
   /* operator = */
+  if(&source == this) return *this;
+  TObject::operator=(source);
+
   fHeader = source.fHeader;
   fTrailer = source.fTrailer;
   fSlotID = source.fSlotID;