Eff C++ warning removal (Marian)
authormarian <marian@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 19 Mar 2008 16:20:16 +0000 (16:20 +0000)
committermarian <marian@f7af4fe6-9843-0410-8265-dc069ae4e863>
Wed, 19 Mar 2008 16:20:16 +0000 (16:20 +0000)
TPC/AliTPCCalibTCF.cxx
TPC/AliTPCCalibViewerGUI.cxx
TPC/AliTPCcalibDB.cxx
TPC/AliTPCcalibTracks.cxx

index 34d18d9..be2f734 100644 (file)
@@ -490,7 +490,7 @@ void AliTPCCalibTCF::TestTCFonRootFile(const char *nameFileIn, const char *nameF
   Int_t iHist = 0;
   
   for(Int_t i=0;i<lowKey-1;i++){++iHist; key = (TKey *) next();}
-  while (key = (TKey *) next()) { // loop over saved histograms
+  while ((key = (TKey *) next())) { // loop over saved histograms
     
     //  loading pulse to memory;
     printf("validating pulse %d out of %d\n",++iHist,nHist);
index f8041e7..68387ea 100644 (file)
@@ -1184,7 +1184,7 @@ AliTPCCalibViewerGUI::AliTPCCalibViewerGUI(const AliTPCCalibViewerGUI &c)
 }
 
 
-AliTPCCalibViewerGUI & AliTPCCalibViewerGUI::operator =(const AliTPCCalibViewerGUI & param) {
+AliTPCCalibViewerGUI & AliTPCCalibViewerGUI::operator =(const AliTPCCalibViewerGUI & /*param*/) {
    //
    // dummy assignment operator
    //
index c89fac7..79685ed 100644 (file)
@@ -378,7 +378,7 @@ void AliTPCcalibDB::CreateObjectList(const Char_t *filename, TObjArray *calibObj
    TIter nextLine(arrFileLine);
    
    TObjString *sObjLine=0x0;
-   while ( sObjLine = (TObjString*)nextLine() ){
+   while ( (sObjLine = (TObjString*)nextLine()) ){
       TString sLine(sObjLine->GetString());
       
       TObjArray *arrNextCol = sLine.Tokenize("\t");
index 05d3f9f..08d3e43 100644 (file)
@@ -102,50 +102,82 @@ ClassImp(AliTPCcalibTracks)
 
 
 AliTPCcalibTracks::AliTPCcalibTracks():
-  TNamed()  
+  TNamed(),
+  fClusterParam(0),
+  fDebugStream(0),
+  fROC(0),
+  fArrayAmpRow(0),
+  fArrayAmp(0), 
+  fArrayQDY(0), 
+  fArrayQDZ(0), 
+  fArrayQRMSY(0),
+  fArrayQRMSZ(0),
+  fArrayChargeVsDriftlength(0),
+  fcalPadRegionChargeVsDriftlength(0),
+  fDeltaY(0),
+  fDeltaZ(0),
+  fResolY(0),
+  fResolZ(0),
+  fRMSY(0),
+  fRMSZ(0),
+  fCuts(0),
+  fHclus(0),
+  fRejectedTracksHisto(0),
+  fHclusterPerPadrow(0),
+  fHclusterPerPadrowRaw(0),
+  fClusterCutHisto(0),
+  fCalPadClusterPerPad(0),
+  fCalPadClusterPerPadRaw(0),
+  fDebugLevel(0),       
+  fFitterLinY1(0),   //!
+  fFitterLinZ1(0),   //! 
+  fFitterLinY2(0),   //! 
+  fFitterLinZ2(0),   //!
+  fFitterParY(0),    //! 
+  fFitterParZ(0)    //!  
 { 
    // 
    // AliTPCcalibTracks default constructor
    //    
-   if (fDebugLevel > 0) cout << "AliTPCcalibTracks' default constructor called" << endl;
-   fClusterParam = 0;
-//    fDebugStream  = 0;
-   fROC          = 0;
-   fArrayAmpRow  = 0;
-   fArrayAmp     = 0; 
-   fArrayQDY     = 0; 
-   fArrayQDZ     = 0; 
-   fArrayQRMSY   = 0;
-   fArrayQRMSZ   = 0;
-   fArrayChargeVsDriftlength = 0;
-   fcalPadRegionChargeVsDriftlength = 0;
-   fDeltaY       = 0;
-   fDeltaZ       = 0;
-   fResolY       = 0;
-   fResolZ       = 0;
-   fRMSY         = 0;
-   fRMSZ         = 0;
-   fCuts         = 0;
-   fHclus        = 0;
-   fRejectedTracksHisto    = 0;
-   fHclusterPerPadrow      = 0;
-   fClusterCutHisto        = 0;
-   fHclusterPerPadrowRaw   = 0;
-   fCalPadClusterPerPadRaw = 0;
-   fCalPadClusterPerPad    = 0;
-   fDebugLevel   = 0;       
-   fFitterLinY1=0;   //!
-   fFitterLinZ1=0;   //! 
-   fFitterLinY2=0;   //! 
-   fFitterLinZ2=0;   //!
-   fFitterParY=0;    //! 
-   fFitterParZ=0;    //!
-  
-//    cout << "end of default constructor" << endl; // TO BE REMOVED
+   if (fDebugLevel > 0) cout << "AliTPCcalibTracks' default constructor called" << endl;  
 }   
 
 
-AliTPCcalibTracks::AliTPCcalibTracks(AliTPCcalibTracks* ct){
+AliTPCcalibTracks::AliTPCcalibTracks(AliTPCcalibTracks* ct):
+TNamed(),
+  fClusterParam(0),
+  fDebugStream(0),
+  fROC(0),
+  fArrayAmpRow(0),
+  fArrayAmp(0), 
+  fArrayQDY(0), 
+  fArrayQDZ(0), 
+  fArrayQRMSY(0),
+  fArrayQRMSZ(0),
+  fArrayChargeVsDriftlength(0),
+  fcalPadRegionChargeVsDriftlength(0),
+  fDeltaY(0),
+  fDeltaZ(0),
+  fResolY(0),
+  fResolZ(0),
+  fRMSY(0),
+  fRMSZ(0),
+  fCuts(0),
+  fHclus(0),
+  fRejectedTracksHisto(0),
+  fHclusterPerPadrow(0),
+  fHclusterPerPadrowRaw(0),
+  fClusterCutHisto(0),
+  fCalPadClusterPerPad(0),
+  fCalPadClusterPerPadRaw(0),
+  fDebugLevel(0),       
+  fFitterLinY1(0),   //!
+  fFitterLinZ1(0),   //! 
+  fFitterLinY2(0),   //! 
+  fFitterLinZ2(0),   //!
+  fFitterParY(0),    //! 
+  fFitterParZ(0)    //!  
+{
    // 
    // AliTPCcalibTracks copy constructor
    // 
@@ -215,8 +247,40 @@ AliTPCcalibTracks::AliTPCcalibTracks(AliTPCcalibTracks* ct){
 
 
 AliTPCcalibTracks::AliTPCcalibTracks(const Text_t *name, const Text_t *title, AliTPCClusterParam *clusterParam,  AliTPCcalibTracksCuts* cuts, Int_t logLevel) : 
-   TNamed(name, title),
-   fHclus(0) {
+  TNamed(name, title),
+  fClusterParam(0),
+  fDebugStream(0),
+  fROC(0),
+  fArrayAmpRow(0),
+  fArrayAmp(0), 
+  fArrayQDY(0), 
+  fArrayQDZ(0), 
+  fArrayQRMSY(0),
+  fArrayQRMSZ(0),
+  fArrayChargeVsDriftlength(0),
+  fcalPadRegionChargeVsDriftlength(0),
+  fDeltaY(0),
+  fDeltaZ(0),
+  fResolY(0),
+  fResolZ(0),
+  fRMSY(0),
+  fRMSZ(0),
+  fCuts(0),
+  fHclus(0),
+  fRejectedTracksHisto(0),
+  fHclusterPerPadrow(0),
+  fHclusterPerPadrowRaw(0),
+  fClusterCutHisto(0),
+  fCalPadClusterPerPad(0),
+  fCalPadClusterPerPadRaw(0),
+  fDebugLevel(0),       
+  fFitterLinY1(0),   //!
+  fFitterLinZ1(0),   //! 
+  fFitterLinY2(0),   //! 
+  fFitterLinZ2(0),   //!
+  fFitterParY(0),    //! 
+  fFitterParZ(0)    //!  
+ {
    // 
    // AliTPCcalibTracks constructor
    // specify 'name' and 'title' of your object
@@ -2502,40 +2566,6 @@ Long64_t AliTPCcalibTracks::Merge(TCollection *collectionList) {
       if (fDebugLevel > 5) cout << "filling lists, object " << counter << " added." << endl;
    }
    
-//    // reset data members
-//    if (fDebugLevel > 1) cout << "histogram's reset-functins are called... " << endl;
-//    fDeltaY->Reset();
-//    fDeltaZ->Reset();
-//    fHclus->Reset();
-//    fClusterCutHisto->Reset();
-//    fRejectedTracksHisto->Reset();
-//    fHclusterPerPadrow->Reset();
-//    fHclusterPerPadrowRaw->Reset();
-//    for (Int_t i = 0; i < fArrayAmpRow->GetEntriesFast(); i++ ) 
-//       ((TProfile*)(fArrayAmpRow->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayAmp->GetEntriesFast(); i++ ) 
-//       ((TProfile*)(fArrayAmp->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayChargeVsDriftlength->GetEntriesFast(); i++)
-//       ((TProfile*)(fArrayChargeVsDriftlength->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayQDY->GetEntriesFast(); i++)
-//       ((TH3F*)(fArrayQDY->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayQDZ->GetEntriesFast(); i++)
-//       ((TH3F*)(fArrayQDZ->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayQRMSY->GetEntriesFast(); i++)
-//       ((TH3F*)(fArrayQRMSY->At(i)))->Reset();
-//    for (Int_t i = 0; i < fArrayQRMSZ->GetEntriesFast(); i++)
-//       ((TH3F*)(fArrayQRMSZ->At(i)))->Reset();
-//    for (Int_t i = 0; i < fResolY->GetEntriesFast(); i++) {
-//       ((TH3F*)(fResolY->At(i)))->Reset();
-//       ((TH3F*)(fResolZ->At(i)))->Reset();
-//       ((TH3F*)(fRMSY->At(i)))->Reset();
-//       ((TH3F*)(fRMSZ->At(i)))->Reset();
-//    }
-//    for (UInt_t isec = 0; isec < 36; isec++){
-//       ((TProfile*)fcalPadRegionChargeVsDriftlength->GetObject(isec, 0))->Reset();
-//       ((TProfile*)fcalPadRegionChargeVsDriftlength->GetObject(isec, 1))->Reset();
-//       ((TProfile*)fcalPadRegionChargeVsDriftlength->GetObject(isec, 2))->Reset();
-//    } 
    
    // merge data members
    if (fDebugLevel > 0) cout << "histogram's merge-functins are called... " << endl;