Fixes for Coverity warnings
authormasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 4 Feb 2011 13:57:54 +0000 (13:57 +0000)
committermasera <masera@f7af4fe6-9843-0410-8265-dc069ae4e863>
Fri, 4 Feb 2011 13:57:54 +0000 (13:57 +0000)
ITS/AliITSMisAligner.cxx
ITS/AliITSMisAligner.h
ITS/AliITSclustererV2.cxx
ITS/AliITSgeom.cxx
ITS/AliITSgeom.h
ITS/AliITSv11Geometry.cxx

index 8da227b..baca8d2 100644 (file)
@@ -93,61 +93,6 @@ AliITSMisAligner::AliITSMisAligner():
     }
 }
 
-//________________________________________________________________________
-AliITSMisAligner::AliITSMisAligner(const AliITSMisAligner &mAligner):
-    AliMisAligner(),
-    fRnd(mAligner.fRnd),
-    fInd(0),
-    fAlignObjArray(mAligner.fAlignObjArray),
-    fStrSPD("ITS/SPD"),
-    fStrSDD("ITS/SDD"),
-    fStrSSD("ITS/SSD"),
-    fStrStave("/Stave"),
-    fStrHalfStave("/HalfStave"),
-    fStrLadder("/Ladder"),
-    fStrSector("/Sector"),
-    fStrSensor("/Sensor"),
-    fUnifSPDSector(kFALSE),
-    fUnifSPDHS(kFALSE),
-    fUnifSDDLadder(kFALSE),
-    fUnifSSDLadder(kFALSE),
-    fUnifSPDLadder(kFALSE),
-    fUnifSDDModule(kFALSE),
-    fUnifSSDModule(kFALSE)
-{
-    //
-    // copy constructor
-    //
-
-  SetMisalType(mAligner.GetMisalType());
-}
-
-//________________________________________________________________________
-AliITSMisAligner &AliITSMisAligner::operator= (const AliITSMisAligner &mAligner)
-{
-    //
-    // assignment operator
-    //
-    fRnd = mAligner.fRnd;
-    fInd = 0;
-    fAlignObjArray = mAligner.fAlignObjArray;
-    fStrSPD = "ITS/SPD";
-    fStrSDD = "ITS/SDD";
-    fStrSSD = "ITS/SSD";
-    fStrStave = "/Stave";
-    fStrHalfStave = "/HalfStave";
-    fStrLadder = "/Ladder";
-    fStrSector = "/Sector";
-    fStrSensor = "/Sensor";
-    fUnifSPDSector = mAligner.fUnifSPDSector;
-    fUnifSPDHS = kFALSE;
-    fUnifSDDLadder = kFALSE;
-    fUnifSSDLadder = kFALSE;
-    fUnifSPDLadder = kFALSE;
-    fUnifSDDModule = kFALSE;
-    fUnifSSDModule = kFALSE;
-    return (*this);
-}
 
 //_______________________________________________________________________________________
 TClonesArray* AliITSMisAligner::MakeAlObjsArray() {
@@ -194,7 +139,7 @@ TClonesArray* AliITSMisAligner::MakeAlObjsArray() {
     // all SSD ladders as from survey, + shift and smearing in the "residual" and "full" case
     for(Int_t ii=0; ii<surveyArray->GetEntriesFast(); ii++)
     {
-       AliAlignObjParams* aop = dynamic_cast<AliAlignObjParams*> (surveyArray->UncheckedAt(ii));
+       AliAlignObjParams* aop = static_cast<AliAlignObjParams*> (surveyArray->UncheckedAt(ii));
        TString sName(aop->GetSymName());
 
        if(sName.Contains("SSD") && !sName.Contains("Sensor"))
@@ -231,7 +176,7 @@ TClonesArray* AliITSMisAligner::MakeAlObjsArray() {
     // all SSD modules
     for(Int_t ii=0; ii<surveyArray->GetEntriesFast(); ii++)
     {
-       AliAlignObjParams* aop = dynamic_cast<AliAlignObjParams*> (surveyArray->UncheckedAt(ii));
+       AliAlignObjParams* aop = static_cast<AliAlignObjParams*> (surveyArray->UncheckedAt(ii));
        TString sName(aop->GetSymName());
        if(sName.Contains("SSD") && sName.Contains("Sensor"))
        {
index 36044cb..928fadf 100644 (file)
@@ -20,8 +20,6 @@ class AliAlignObjParams;
 class AliITSMisAligner : public AliMisAligner{
     public:
        AliITSMisAligner();
-       AliITSMisAligner(const AliITSMisAligner &mAligner);
-       AliITSMisAligner &operator= (const AliITSMisAligner &mAligner);
        ~AliITSMisAligner() {};
 
        TClonesArray* MakeAlObjsArray();
@@ -191,6 +189,8 @@ class AliITSMisAligner : public AliMisAligner{
        TString      fStrSensor; // name of sensitive volume
 
     private:
+       AliITSMisAligner(const AliITSMisAligner &mAligner);
+       AliITSMisAligner &operator= (const AliITSMisAligner &mAligner);
        enum {kNLayers = 6}; // The number of layers.
        static const Int_t  fgkNLadders[kNLayers];  // Array of the number of ladders/layer(layer)
        static const Int_t  fgkNDetectors[kNLayers];// Array of the number of detector/ladder(layer)
index 10113e2..eed786e 100644 (file)
@@ -54,10 +54,8 @@ fHlSSD(0),
 fTanP(0),
 fTanN(0){
    //default constructor
-  for(Int_t i=0;i<260;i++){
-    fYSPD[i]=0.;
-    fZSPD[i]=0.;
-  }
+  for(Int_t i=0;i<260;i++)fYSPD[i]=0.;
+  for(Int_t i=0;i<170;i++)fZSPD[i]=0.;
   for(Int_t i=0;i<2200;i++){
     fYshift[i]=0.;
     fZshift[i]=0.;
index 37b63ef..2dffb6f 100644 (file)
@@ -23,7 +23,7 @@
 // version: 0.0.1                                                    //
 // Updated May 27 1999.                                              //
 // Added Cylindrical random and global based changes.                //
-// Added  function PrintComparison.                                  //
+//                                                                   //
 // Modified and added functions Feb. 7 2006                          //
 ///////////////////////////////////////////////////////////////////////
 
@@ -546,77 +546,7 @@ Int_t AliITSgeom::GetLastDet(Int_t dtype)const{
     Warning("GetLastDet","undefined detector type %d",dtype);
     return 0;
 }
-//______________________________________________________________________
-void AliITSgeom::PrintComparison(FILE *fp,AliITSgeom *other)const{
-    //     This function was primarily created for diagnostic reasons. It
-    // print to a file pointed to by the file pointer fp the difference
-    // between two AliITSgeom classes. The format of the file is basically,
-    // define d? to be the difference between the same element of the two
-    // classes. For example dfrx = this->GetGeomMatrix(i)->frx 
-    // - other->GetGeomMatrix(i)->frx.
-    // if(at least one of dfx0, dfy0, dfz0,dfrx,dfry,dfrz are non zero) then
-    // print layer ladder detector dfx0 dfy0 dfz0 dfrx dfry dfrz
-    // if(at least one of the 9 elements of dfr[] are non zero) then print
-    // layer ladder detector dfr[0] dfr[1] dfr[2]
-    //                       dfr[3] dfr[4] dfr[5]
-    //                       dfr[6] dfr[7] dfr[8]
-    // Only non zero values are printed to save space. The differences are
-    // typical written to a file because there are usually a lot of numbers
-    // printed out and it is usually easier to read them in some nice editor
-    // rather than zooming quickly past you on a screen. fprintf is used to
-    // do the printing. The fShapeIndex difference is not printed at this time.
-    // Inputs:
-    //    FILE *fp           A file pointer to an opened file for writing 
-    //                       in which the results of the comparison will 
-    //                       be written.
-    //    AliITSgeom *other  The other AliITSgeom class to which this one is
-    //                       being compared.
-    // Outputs:
-    //    none.
-    // Return:
-    //    none.
-    Int_t    i,j,idt[3],ido[3];
-    Double_t tt[3],to[3];  // translation
-    Double_t rt[3],ro[3];  // phi in radians
-    Double_t mt[3][3],mo[3][3]; // matrices
-    AliITSgeomMatrix *gt,*go;
-    Bool_t   t;
 
-    for(i=0;i<this->fNmodules;i++){
-        gt  =  this->GetGeomMatrix(i);
-        go  = other->GetGeomMatrix(i);
-        gt->GetIndex(idt);
-        go->GetIndex(ido);
-        t = kFALSE;
-        for(i=0;i<3;i++) t = t&&idt[i]!=ido[i];
-        if(t) fprintf(fp,"%4.4d %1.1d %2.2d %2.2d %1.1d %2.2d %2.2d\n",i,
-                      idt[0],idt[1],idt[2],ido[0],ido[1],ido[2]);
-        gt->GetTranslation(tt);
-        go->GetTranslation(to);
-        gt->GetAngles(rt);
-        go->GetAngles(ro);
-        t = kFALSE;
-        for(i=0;i<3;i++) t = t&&tt[i]!=to[i];
-        if(t) fprintf(fp,"%1.1d %2.2d %2.2d dTrans=%f %f %f drot=%f %f %f\n",
-                      idt[0],idt[1],idt[2],
-                      tt[0]-to[0],tt[1]-to[1],tt[2]-to[2],
-                      rt[0]-ro[0],rt[1]-ro[1],rt[2]-ro[2]);
-        t = kFALSE;
-        gt->GetMatrix(mt);
-        go->GetMatrix(mo);
-        for(i=0;i<3;i++)for(j=0;j<3;j++)  t = mt[i][j] != mo[i][j];
-        if(t){
-            fprintf(fp,"%1.1d %2.2d %2.2d dfr= %e %e %e\n",
-                    idt[0],idt[1],idt[2],
-                    mt[0][0]-mo[0][0],mt[0][1]-mo[0][1],mt[0][2]-mo[0][2]);
-            fprintf(fp,"        dfr= %e %e %e\n",
-                    mt[1][0]-mo[1][0],mt[1][1]-mo[1][1],mt[1][2]-mo[1][2]);
-            fprintf(fp,"        dfr= %e %e %e\n",
-                    mt[2][0]-mo[2][0],mt[2][1]-mo[2][1],mt[2][2]-mo[2][2]);
-        } // end if t
-    } // end for i
-    return;
-}
 //______________________________________________________________________
 void AliITSgeom::PrintData(FILE *fp,Int_t lay,Int_t lad,Int_t det)const{
     //     This function prints out the coordinate transformations for
index 30e861e..9456c5a 100644 (file)
@@ -729,9 +729,6 @@ class AliITSgeom : public TObject {
     // This converts the geometry
     void GeantToTracking(const AliITSgeom &source);
     //  Other routines.
-    // This routine prints, to a file, the difference between this class
-    // and "other".
-    void PrintComparison(FILE *fp,AliITSgeom *other)const;
     // This routine prints, to a file, the contents of this class.
     void PrintData(FILE *fp,Int_t lay,Int_t lad,Int_t det)const;
     // This function prints out this class in a single stream. This steam
index 1d4bfd0..0943cfb 100644 (file)
@@ -1533,15 +1533,15 @@ void AliITSv11Geometry::MakeFigure1(Double_t x0,Double_t y0,Double_t r0,
     TText *t = new TText();
     t->SetTextSize(0.02);
     Char_t txt[100];
-    sprintf(txt,"(x0=%5.2f,y0=%5.2f)",x0,y0);
+    snprintf(txt,99,"(x0=%5.2f,y0=%5.2f)",x0,y0);
     t->DrawText(x0,y0,txt);
-    sprintf(txt,"(x1=%5.2f,y1=%5.2f)",x1,y1);
+    snprintf(txt,99,"(x1=%5.2f,y1=%5.2f)",x1,y1);
     for(j=0;j<4;j++) {
         t->SetTextColor(j+1);
         t->DrawText(x1,y1,txt);
-        sprintf(txt,"r0=%5.2f",ra0[j]);
+        snprintf(txt,99,"r0=%5.2f",ra0[j]);
         t->DrawText(0.5*(x0+xa0[j]),0.5*(y0+ya0[j]),txt);
-        sprintf(txt,"r1=%5.2f",ra1[j]);
+        snprintf(txt,99,"r1=%5.2f",ra1[j]);
         t->DrawText(0.5*(x1+xa1[j]),0.5*(y1+ya1[j]),txt);
     } // end for j
 }