Comment 7 for bug 1950300

Revision history for this message
mdavidsaver (mdavidsaver) wrote :

> EPICS 3.14.12 has it in src/gdd/aitConvert.cc.

Ok, this is enough for me wrt. compatibility.

> In 3.14, missing arguments instead of 0 were allowed as well: "#C S2" ...

The change to acceptable link syntax dates to 2014 [2]. It was understood and accepted at the time that making the parser stricter could break some existing databases.

> I can create a PR but need to know which way to go. ...

I would encourage you to push this small change through before trying for something larger.

I don't see the change in interpretation of eg. "#C010 S1 @" as octal to be a likely issue. Although it might be argued that it is less obvious as it doesn't result in an error.

[1] https://code.launchpad.net/~epics-core/epics-base/link-parsing/+merge/229252