Experimental Physics and
Industrial Control System

Andrew Johnson via Core-talk <core-talk at aps.anl.gov> · Mon, 08 Jun 2020 18:31:34 -0000

No, the previous version of that code at
1f726c876027894c78c1a18fe2f2de9eb04c288c still had strchr(pstr, ',') in
the condition.

My point is that both [] and [42] are completely legal record names
according to our documentation, so this merge would change the meaning
of them when used as an INP field value. Neither the double-quote nor
comma characters are legal in record names as documented, and the two
strchr() tests were to ensure that we only changed the meaning in cases
that weren't previously legal.

The commit you pointed to was made to also allow constant strings to be
specified using the DB-file syntax ["..."].

We can have a discussion over whether we want to change the requirements
for legal record names, but we should agree about doing that first.

-- 
You received this bug notification because you are a member of EPICS
Core Developers, which is subscribed to EPICS Base.
Matching subscriptions: epics-core-list-subscription
https://bugs.launchpad.net/bugs/1882520

Title:
  Constant arrary input links of size 1 don't work

Status in EPICS Base:
  In Progress

Bug description:
  Constant input links in array form with 1 element don't work and
  return an empty array.

  Example:
  field (INP, [42])

  As soon as a comma is present, it works:
  field (INP, [42,])

To manage notifications about this bug go to:
https://bugs.launchpad.net/epics-base/+bug/1882520/+subscriptions

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System