Coverity fixes:
authorJochen Klein <jochen.klein@cern.ch>
Tue, 21 Jan 2014 08:48:48 +0000 (09:48 +0100)
committerJochen Klein <jochen.klein@cern.ch>
Tue, 21 Jan 2014 08:48:48 +0000 (09:48 +0100)
TRD/AliTRDgtuSim: - forward null checks
TRD/AliTRDtrackOnline: - self assignment

TRD/AliTRDgtuSim.cxx
TRD/AliTRDtrackOnline.cxx

index 107cc85..9e53d3d 100644 (file)
@@ -151,8 +151,7 @@ Bool_t AliTRDgtuSim::RunGTUFromTrackletFile(TString filename, Int_t event, Int_t
                       iLink, i-4, ((TObjString*) tokens->At(i))->GetString().Data(), trackletWord));
       AliTRDtrackletWord *tracklet = new (trklArray[trklArray.GetEntriesFast()])       AliTRDtrackletWord(trackletWord);
       AliTRDtrackletGTU   *trkl    = new (trklArrayGTU[trklArrayGTU.GetEntriesFast()]) AliTRDtrackletGTU(tracklet);
-      if (fTMU)
-       fTMU->AddTracklet(trkl, iLink);
+      fTMU->AddTracklet(trkl, iLink);
     }
     //
     delete tokens;
@@ -230,9 +229,7 @@ Bool_t AliTRDgtuSim::RunGTU(AliLoader *loader, AliESDEvent *esd, Int_t label, In
        }
        AliDebug(1, Form("adding tracklet: 0x%08x in sec %i stack %i link %i",
                         trkl->GetTrackletWord(), trkl->GetDetector() / 30, (trkl->GetDetector() % 30) / 6, trkl->GetHCId() % 12));
-       if (fTMU) {
-         fTMU->AddTracklet(trkl, iLink);
-       }
+       fTMU->AddTracklet(trkl, iLink);
     }
 
     if (pendingTracklets) {
index 43be589..ee65f24 100644 (file)
@@ -300,9 +300,11 @@ AliTRDtrackResiduals::AliTRDtrackResiduals(const AliTRDtrackResiduals &rhs) :
 
 AliTRDtrackResiduals& AliTRDtrackResiduals::operator=(const AliTRDtrackResiduals &rhs)
 {
-  ROOT::Math::IBaseFunctionMultiDim::operator=(rhs);
-  fTrack = rhs.fTrack;
-  fParam = rhs.fParam;
+  if (&rhs != this) {
+    ROOT::Math::IBaseFunctionMultiDim::operator=(rhs);
+    fTrack = rhs.fTrack;
+    fParam = rhs.fParam;
+  }
 
   return *this;
 }