On Thu, 27 Jul 2000, Djamel Ouerdane wrote: > Hi Alv and others, > > Following your idea, I made the modifications in BrRawDataInput and tested > it in an application that is not yet public (I'm working on a set of > classes for offline analyses, I shall tell more about that later). > It works pretty fine at least for the hodoscopes. > > I suggest you or I commit that stuff into the CVS repository (but maybe > some deeper testing is needed?). > Concerning the applications that use the old methods, I guess it will be > up to their authors to update them. > > Any comments? I have committed a new version of BrRawDataInput to cure an unwanted feature that was present in the SetXXXOn and SetXXXOff methods in revision 1.29 and later of BrRawDataInput.h. Calling SetXXXOff() for one detector would switch off the decoding for all detectors in either FS, MRS or Global. The old Bool_t fXXXOn flags have been replaced by a BrDetectorList fDetectors member. I have added the following methods void SetOn (EBrDetectorBit b) void SetOn (EBrSectorBit m) void SetOn (EBrDetectorBit b, Bool_t v) void SetOn (EBrSectorBit m, Bool_t v) void SetOff(EBrDetectorBit b) void SetOff(EBrSectorBit m) Bool_t IsOn (EBrDetectorBit b) const Bool_t IsOn (EBrSectorBit m) const to switch on or off the decoding of one detector or a group of detectors, and to test if the decoding has been switched on for a detector or at least one in a group of detectors. The default is to decode all detectors. The old SetXXXOn methods have been changed to use the new SetOn method. Thus, there is no need to rewrite old applications. However, none of the applications in brat/test use BrRawDataInput::SetXXXOn/Off. I have also moved the calls to SetRICHOn/Off from the FFS to the BFS, and made the GetDataState method const. -Alv Kjetil
This archive was generated by hypermail 2b29 : Fri Jul 28 2000 - 17:41:31 EDT