minor coverity defects: self-assignment protection
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Nov 2011 09:57:05 +0000 (09:57 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 21 Nov 2011 09:57:05 +0000 (09:57 +0000)
HLT/TPCLib/AliHLTTPCClusterAccessHLTOUT.h
HLT/TPCLib/AliHLTTPCHWCFData.h
HLT/TPCLib/AliHLTTPCHWCFSpacePointContainer.h

index e87a6a3..d16c59d 100644 (file)
@@ -145,6 +145,7 @@ class AliHLTTPCClusterAccessHLTOUT : public TObject
       iterator(AliTPCclusterMIContainer* pData) : fClusterNo(-1), fData(pData), fCluster(NULL), fClusterId(fData?fData->GetClusterId(fClusterNo):kAliHLTVoidDataSpec), fRowOffset(0) {}
       iterator(const iterator& other) : fClusterNo(other.fClusterNo), fData(other.fData), fCluster(other.fCluster), fClusterId(other.fClusterId), fRowOffset(other.fRowOffset) {}
       iterator& operator=(const iterator& other) {
+       if (this==&other) return *this;
        fClusterNo=other.fClusterNo; fData=other.fData; fCluster=other.fCluster, fClusterId=other.fClusterId; fRowOffset=other.fRowOffset; return *this;
       }
       ~iterator() {}
index 39e4c60..e4c03c8 100644 (file)
@@ -191,8 +191,10 @@ class AliHLTTPCHWCFData : public AliHLTLogging {
       : fData(pData), fVersion(version), fElementSize(elementSize) {}
     iterator(const iterator& i)
       : fData(i.fData), fVersion(i.fVersion), fElementSize(i.fElementSize) {}
-    iterator& operator=(const iterator& i)
-      { fData=i.fData; fVersion=i.fVersion; fElementSize=i.fElementSize; return *this;}
+    iterator& operator=(const iterator& i) {
+      if (this==&i) return *this;
+      fData=i.fData; fVersion=i.fVersion; fElementSize=i.fElementSize; return *this;
+    }
     ~iterator() {fData=NULL;}
 
     bool operator==(const iterator& i) const  {return (fData!=NULL) && (fData==i.fData);}
index 6293cfb..700b81e 100644 (file)
@@ -149,8 +149,10 @@ class AliHLTTPCHWCFSpacePointContainer : public AliHLTSpacePointContainer
       : fDecoder(pDecoder), fGrid(pGrid), fId(id) {}
     AliHLTTPCHWCFSpacePointBlock(const AliHLTTPCHWCFSpacePointBlock& s) 
       : fDecoder(s.fDecoder), fGrid(s.fGrid), fId(s.fId) {}
-    AliHLTTPCHWCFSpacePointBlock& operator=(const AliHLTTPCHWCFSpacePointBlock& s) 
-    { fDecoder=s.fDecoder; fGrid=s.fGrid; fId=s.fId; return *this;}
+    AliHLTTPCHWCFSpacePointBlock& operator=(const AliHLTTPCHWCFSpacePointBlock& s) {
+      if (this==&s) return *this;
+      fDecoder=s.fDecoder; fGrid=s.fGrid; fId=s.fId; return *this;
+    }
     ~AliHLTTPCHWCFSpacePointBlock() {}
 
     int GetNofSpacepoints() const {return fDecoder?fDecoder->GetNumberOfClusters():0;}