Re: int64in record not sending monitors for larger value changes

["Johnson, Andrew N. via Tech-talk" <tech-talk at aps.anl.gov>]

Hello Kay,

On Jul 29, 2021, at 10:37 AM, Kasemir, Kay via Tech-talk <tech-talk at aps.anl.gov> wrote:

I'm trying to use the relatively new 64 bit records in EPICS base..
The output of 'camonitor sixtyfour', however, stops updating as the value grows beyond 0x87654321:

The reason is this section in int64inRecord.c:

/* DELTA calculates the absolute difference between its arguments
 * expressed as an unsigned 32-bit integer */
#define DELTA(last, val) \
    ((epicsUInt32) ((last) > (val) ? (last) - (val) : (val) - (last)))

It only checks for changes in the lower 32 bits of the value.
Maybe the motivation was to always perform an _unsigned_ comparison, but "UInt32" was copy/pasted instead of "UInt64"?

The int64{in,out} record types were created from the long{in,out} types, and it looks like it that particular 32 to 64 conversion was just missed.

Would you mind creating an issue or even a pull request in GitHub for the appropriate change, just so we don’t forget about it?

Thanks!

- Andrew

-- 

Complexity comes for free, simplicity you have to work for.