> 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.
> 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