coverity fix

[u/mrichter/AliRoot.git] / TPC / AliTPCBuffer.cxx

diff --git a/TPC/AliTPCBuffer.cxx b/TPC/AliTPCBuffer.cxx
index 4487b31d2eaae279055e141f7fd64e91cf018d66..fdbdb72a228af7e1a5bb87d2b9ab197783a1665e 100644 (file)
--- a/TPC/AliTPCBuffer.cxx
+++ b/TPC/AliTPCBuffer.cxx
@@ -20,27 +20,44 @@
 // This class is used by AliTPCDDL.C macro
 // Author: D.Favretto
 
-#include "Riostream.h"
-#include "TObjArray.h"
+#include <Riostream.h>
+#include <TObjArray.h>
 #include "AliTPCBuffer.h"
 #include "AliSimDigits.h"
 
 //#include "TFile.h"
 //#include "TTree.h"
 
+using std::ios;
+using std::ofstream;
+using std::endl;
 ClassImp(AliTPCBuffer)
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////
-AliTPCBuffer::AliTPCBuffer(const char* fileName){
+//___________________________________________________________
+  AliTPCBuffer::AliTPCBuffer():TObject(),
+    fVerbose(0),
+    fNumberOfDigits(0),
+    f()
+{
+  //
+  // default
+  //
+}
+//____________________________________________________________
+  AliTPCBuffer::AliTPCBuffer(const char* fileName):TObject(),
+    fVerbose(0),
+    fNumberOfDigits(0),
+    f()
+{
   // Constructor
 #ifndef __DECCXX
-  f.open("AliTPCDDL.dat",ios::binary|ios::out);
+  f.open(fileName,ios::binary|ios::out);
 #else
-  f.open("AliTPCDDL.dat",ios::out);
+  f.open(fileName,ios::out);
 #endif
   // fout=new TFile(fileName,"recreate");
   // tree=new TTree("tree","Values");
-  fNumberOfDigits=0;
-  fVerbose=0;
+
   remove("TPCdigits.txt");
 }
 
@@ -52,7 +69,11 @@ AliTPCBuffer::~AliTPCBuffer(){
   //delete fout;
 }
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////
-AliTPCBuffer::AliTPCBuffer(const AliTPCBuffer &source){
+AliTPCBuffer::AliTPCBuffer(const AliTPCBuffer &source):TObject(source),
+    fVerbose(0),
+    fNumberOfDigits(0),
+    f()
+{
   // Copy Constructor
   this->fVerbose=source.fVerbose;
   return;
@@ -60,7 +81,9 @@ AliTPCBuffer::AliTPCBuffer(const AliTPCBuffer &source){
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////
 AliTPCBuffer& AliTPCBuffer::operator=(const AliTPCBuffer &source){
   //Assigment operator
-  this->fVerbose=source.fVerbose;
+  if(this!=&source){
+    this->fVerbose=source.fVerbose;
+  }
   return *this;
 }
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -131,7 +154,7 @@ void AliTPCBuffer::WriteRowBinary(Int_t eth,AliSimDigits *digrow,Int_t minPad,In
   data.Sec=sec;
   data.SubSec=SubSec;
   data.Row=row;
-  digrow->First();
+  if (!digrow->First()) return;
   Int_t padID=-1;
   Int_t ddlNumber=0;
   ofstream ftxt;