Comment 2 for bug 1250023
Revision history for this message
| Dirk Zimoch (dirk.zimoch) wrote Re: [Bug 1250023] Re: alarmString.h implements alarm strings in header. | #2 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
5466644
demo site
(Get the code!)
On 11.11.2013 18:01, Andrew Johnson wrote:
> There is a branch that already does this, see lp:~anj/epics-base/alarm-strings
> http://
> I had not considered making this change on the 3.14 branch, but if you think it's worth doing and that it won't break existing code I can move it to there.
>
Hi Andrew,
Great, I did not know that. However your approach is not backward
compatible since it requires the user to include alarmString.h directly now.
I suggest to keep the declaration of the string variables in alarm.h. So
no user code needs to be modified. (You can test this with an unmodified
tool_lib.c)
I also would appreciate a patch for 3.14.12.
Dirk