epics-modules / autosave

APS BCDA synApps module: autosave
https://epics-modules.github.io/autosave/
Other
8 stars 31 forks source link

do_manual_restore: fix erroneous error message #50

Closed mdavidsaver closed 1 year ago

mdavidsaver commented 1 year ago

Fix incorrect error check on ca_put() in fdbrestore(). I guess no one uses this function much...

anjohnson commented 1 year ago

For folks reading that don't realize this, the ECA_NORMAL macro gives the value 1, not 0.

I have seen working CA client code that used status *= ca_...(); to combine the status returns from a long string of CA API calls, then checked whether the final result was 1 or not at the end. Not that I'm recommending the technique, but it took advantage of that definition.