BRAT update - BrDetectorList

From: Christian Holm Christensen (cholm@hehi03.nbi.dk)
Date: Mon Aug 14 2000 - 13:39:30 EDT

  • Next message: hagel@comp.tamu.edu: "BrRun object"

    Hi BRAT'ers, 
    
    I've commit a few changes to BRAT, and tagged it VER-1-8-2. 
    
    Please notice, that the versdef variable in test/brat-config should
    contain THREE numbers, seperated by a period '.'. The numbers are (of
    course) <major version>.<minor version>.<revision>. Please update this
    variable everytime you do a tagged commit. 
    
    The real important change is:
    -----------------------------
    Following the advice of Fons Rademarkers, I've changed the type of 
    
      static const Int_t BrDetectorList::kNumberOfDetectors = 17;
    
    to 
    
      enum { BrDetectorList::kNumberOfDetectors = 17 };
    
    and 
    
      static const Char_t* BrDetectorList::kDetectorNames[];
    
    to 
    
      static const Char_t* BrDetectorList::kDetectorNames[kNumberOfDetectors];
    
    since there seemed to be some problem with TDataMember::GetMaxIndex(0)
    for BrDetectorList::kDetectorNames returned a negative large number
    (overflow!?), which in turn screwed up the HTML generation for
    BrDetectorList (BrDetectorList.html reach a size of 2 GB!). 
    
    For more on the problem, take a look at my mail to roottalk, and Fons'
    reply:
    
    	http://root.cern.ch/root/roottalk/roottalk00/2323.html
    
    That should help a lot. 
    
    Christian  -----------------------------------------------------------
    Holm Christensen                             Phone:  (+45) 35 35 96 91 
      Sankt Hansgade 23, 1. th.                  Office: (+45) 353  25 305 
      DK-2200 Copenhagen N                       Web:    www.nbi.dk/~cholm    
      Denmark                                    Email:       cholm@nbi.dk
    



    This archive was generated by hypermail 2b29 : Mon Aug 14 2000 - 13:40:22 EDT