MantisBT - Hall D Offline
View Issue Details
0000429Hall D OfflineGeneralpublic2014-12-03 16:102014-12-04 08:01
shepherd 
davidl 
lowminoralways
resolvedfixed 
0000429: type mismatch
I notice there is some type mismatch between the Df250PulseIntegral class and, say, the DFCALDigiHit class. (It probably exists in other DigiHit classes also.)

In DTranslationTable.h class there is this template:

template<class T>
void DTranslationTable::CopyDf250Info(T *h, const Df250PulseIntegral *pi, const Df250PulseTime *pt, const Df250PulsePedestal *pp) const
{
    /// Copy info from the fADC250 into a hit object.
    h->pulse_integral = pi->integral;

For the last assignment the RHS is of type int32_t, while the LHS is of type uint32_t. It turns out the RHS is initialized from data of type uint32_t in the constructor of the Df250PulseIntegral object. Therefore, I *think* this is harmless, but it would be good to correct it for consistency.
No tags attached.
Issue History
2014-12-03 16:10shepherdNew Issue
2014-12-04 08:01davidlNote Added: 0000621
2014-12-04 08:01davidlStatusnew => resolved
2014-12-04 08:01davidlResolutionopen => fixed
2014-12-04 08:01davidlAssigned To => davidl

Notes
(0000621)
davidl   
2014-12-04 08:01   
Types of both integral and pedestal members of Df250PulseIntegral and Df125PulseIntegral to be uint32_t in revision 16742.