Code review comment for lp:~epics-core/epics-base/epicsTime-status

Revision history for this message
mdavidsaver (mdavidsaver) wrote :

I like the idea of more specific errors, but I'd like to keep the definition of epicsTimeERROR to maintain source compatibility with external code (ie. the time provider in mrfioc2). Perhaps epicsTimeERROR can be an alias for an S_ error code (S_time_legacy?).

Otherwise no problem. I can make the change if Andrew approves a name.

review: Needs Fixing

« Back to merge proposal